Restore overwriting of this._sourceBrowser as it was before bug 1308621
to ensure we continue to use the same DOM to do print previewing (and
eventually printing).
Bug 1308621 already changed the enterPrintPreview code to rely on the
member _sourceBrowser variable, which will be updated to point to the
print preview browser if/when print preview is being reinitialized for
the same page (because one of the print settings changes). We need to
do this to avoid re-initializing off the original browser, which may
now have navigated or be displaying something else entirely.
This also updates the 'simplified' mode code to rely on the extant
_originalURL member to avoid displaying the page URL as about:blank
after a settings change.
MozReview-Commit-ID: DZ1kT7Mb0mS
--HG--
extra : rebase_source : cb79c835d6f8bcd67a7118de3a19b1cab85b6593
This was tricky because synthesizeMouse would compute the incorrect
coordinates if the requested event target was in a sub-frame. With this patch,
we deal correctly with sub-frames.
MozReview-Commit-ID: KpUKxFXKMrl
--HG--
extra : rebase_source : 08dbbd1abe04886ac4fe14fad11312ba0f63873b
Patch originally by Rahid Hasan. Updated by Akshendra Pratap Singh, jdm and Standard8.
MozReview-Commit-ID: F0edTCSsXzy
--HG--
extra : rebase_source : 7123a8162a4cdbcfb08f9a765d158e179c46b202
nsPrintingPromptServiceX doesn't use Objective-C, so we should change to C++ instead.
Also, nsPrintDialogServiceX::ShowPageSetup should have NS_OBJC_BEGIN_TRY_ABORT_BLOCK_NSRESULT.
MozReview-Commit-ID: 33LfNFmiUTQ
--HG--
rename : toolkit/components/printingui/mac/nsPrintingPromptServiceX.mm => toolkit/components/printingui/mac/nsPrintingPromptService.cpp
extra : rebase_source : 7ab7e3aeed81afc70d02bd262d87d6858380bb01
What you see first is the removal of the line `this._highlightAll = aHighlight;`, which is repeated in the `_setHighlightAll` method.
This line was put here initially to make the test_findbar_events.xul test pass but in fact makes it so that the pref is never set in `_setHighlightAll`!
In other words, we never actually persisted the 'Highlight All' state properly.
Reading further: the `_dispatchFindEvent` attaches some findbar state flags to the event details, including the value of `_highlightAll`.
Even though none of our consumers use it currently (haven't checked if TB does, though), you can cancel further execution of highlighting all ranges.
Since the `_setHighlightAll` doesn't do that kind of processing, but merely makes sure the internal state is up to snuff, is persisted properly and the buttons are updated, I moved it up to be invoked before dispatching the event.
MozReview-Commit-ID: 4BBy4FR1r5c
--HG--
extra : rebase_source : 204e77aaef3cd55886daeb2e0fdef84da1159c68
Info is useful to sandboxing. Logs when ServiceWorker is registered (i.e. each run), not just when it is installed.
--HG--
extra : rebase_source : e21a521a5015c722cfbf2f5a7a23d5cf39a88a18
Added |_shuttingDown| boolean flag, which is initialized to false and is set to true when Firefox shuts down.
This allows rejecting pings sent after shutdown by checking the value of this flag.
Also added a new test that shuts down Telemetry using |yield TelemetryController.testShutdown()| and then tries sending a ping to it.
--HG--
extra : rebase_source : 9cb901e9aa12560ab8bb1eadc86a45df47b75bca