Differential Revision: https://phabricator.services.mozilla.com/D46145

--HG--
extra : moz-landing-system : lando

Bug 1581377 - Let display:flex not override the hidden attribute on the one off search button container. r=harry

Differential Revision: https://phabricator.services.mozilla.com/D46124

--HG--
extra : moz-landing-system : lando

Bug 1579420 - Fix remaining ESLint no-async-promise-executor issues in browser/ and toolkit/. r=mossop

Differential Revision: https://phabricator.services.mozilla.com/D45010

--HG--
extra : moz-landing-system : lando

Differential Revision: https://phabricator.services.mozilla.com/D45009

--HG--
extra : moz-landing-system : lando

Differential Revision: https://phabricator.services.mozilla.com/D45500

--HG--
extra : moz-landing-system : lando

Differential Revision: https://phabricator.services.mozilla.com/D46130

--HG--
extra : moz-landing-system : lando

Differential Revision: https://phabricator.services.mozilla.com/D45785

--HG--
extra : moz-landing-system : lando

The thread calls into some windows APIs that may sometimes block for a long time. Since we're shutting down anyway, it is OK to sometimes leak this thread, similar to what we do for DNS resolver threads.

The patch converts nsNotifyAddrListener::mThread to a nsIThreadPool (with a max of 1 thread) so that we may call shutdownWithTimeout on it.

Differential Revision: https://phabricator.services.mozilla.com/D46002

--HG--
extra : moz-landing-system : lando

The Telemetry::AccumulateTimeDelta function reports in milliseconds but I had
assumed seconds when I added the hostname obfuscation ICE duration histogram.
This change adjusts the "high" value accordingly.

With the telemetry we've collected to date, 60% of the values are below 600
milliseconds, so I think that setting a high value of 30 seconds should be
sufficient even if there is a long tail. Since this does not change the data
being collected, no additional data-review is required, but it does require
renaming the key.

Differential Revision: https://phabricator.services.mozilla.com/D46045

--HG--
extra : moz-landing-system : lando

Depends on D45996

Differential Revision: https://phabricator.services.mozilla.com/D45997

--HG--
extra : moz-landing-system : lando

Flushing on mprotect is a lot simpler and it's easier to reason about
correctness. The next patch will remove the AutoFlushICache infrastructure.

Because mprotect is pretty expensive we've been moving away from code patching
in perf-sensitive cases so I don't expect any serious regressions from this.
(This patch could be optimized more, if needed, but it'd add more complexity.)

Differential Revision: https://phabricator.services.mozilla.com/D45996

--HG--
extra : moz-landing-system : lando

This restores the code for generating devicechange events that was
accidentally removed as part of updating the Windows video capture code
in Bug 1552755.

Differential Revision: https://phabricator.services.mozilla.com/D46033

--HG--
extra : moz-landing-system : lando

Bug 1580185. Clicking on clear button in the Search panel doesn't cancel the current search. r=Honza.

Differential Revision: https://phabricator.services.mozilla.com/D46047

--HG--
extra : moz-landing-system : lando

Differential Revision: https://phabricator.services.mozilla.com/D40100

--HG--
extra : moz-landing-system : lando

Bug 1580130 - Add telemetry to record presence of the networkID when the connectivity check has completed r=dragana

When the IPv4 check is completed, we get the network ID and report to telemetry if it was empty or not. This allows us to more accurately determine if we always have a networkID.

Depends on D45331

Differential Revision: https://phabricator.services.mozilla.com/D45332

--HG--
extra : moz-landing-system : lando

Since we sometimes coalesce SendEvents, calling it from there is not actually correct, and sometimes we fail to recompute the networkID when network changes happen.

Differential Revision: https://phabricator.services.mozilla.com/D45331

--HG--
extra : moz-landing-system : lando

Backed out changeset 281d238e6259 (bug 1579489)
Backed out changeset 764628963a3c (bug 1579489)
Backed out changeset 680a45ca89b7 (bug 1579489)

Differential Revision: https://phabricator.services.mozilla.com/D45922

--HG--
extra : moz-landing-system : lando

Bug 1561443 - Move _getErrorMessageFromCode from NetErrorChild.jsm to aboutNetError.js. r=fluent-reviewers,flod,johannh

Differential Revision: https://phabricator.services.mozilla.com/D36542

--HG--
extra : moz-landing-system : lando

Differential Revision: https://phabricator.services.mozilla.com/D45540

--HG--
extra : moz-landing-system : lando

Differential Revision: https://phabricator.services.mozilla.com/D45550

--HG--
extra : moz-landing-system : lando

Bug 1581071 - Update devtools metrics test to check that modules are not loaded twice r=Honza,ochameau

Check for duplicated entries in the loaders during the metrics tests

Differential Revision: https://phabricator.services.mozilla.com/D45833

--HG--
extra : moz-landing-system : lando

Bug 1579489 - Check the persistent permission checkbox by default for WebExtension WebRTC prompts. r=jib

This is done to offset the requirement that extensions have to get persistent permission before
they can start using WebRTC in popups.

Differential Revision: https://phabricator.services.mozilla.com/D45335

--HG--
extra : moz-landing-system : lando

Differential Revision: https://phabricator.services.mozilla.com/D45334

--HG--
extra : moz-landing-system : lando

The goal is to

- Avoid showing any permission prompt when calling getUserMedia in webextension popups,
  while still checking for persistent permissions.
- Never allow getUserMedia calls on background pages.

Differential Revision: https://phabricator.services.mozilla.com/D45333

--HG--
extra : moz-landing-system : lando

Bug 1570081 - Put the blob dirty rect in the same space as the drawing commands and visible rect. r=jrmuizel

Differential Revision: https://phabricator.services.mozilla.com/D43223

--HG--
extra : moz-landing-system : lando

Differential Revision: https://phabricator.services.mozilla.com/D43222

--HG--
extra : moz-landing-system : lando

Differential Revision: https://phabricator.services.mozilla.com/D41389

--HG--
extra : moz-landing-system : lando

Differential Revision: https://phabricator.services.mozilla.com/D41387

--HG--
extra : moz-landing-system : lando

Differential Revision: https://phabricator.services.mozilla.com/D39911

--HG--
extra : moz-landing-system : lando

Differential Revision: https://phabricator.services.mozilla.com/D46018

--HG--
extra : moz-landing-system : lando

Differential Revision: https://phabricator.services.mozilla.com/D45853

--HG--
rename : gfx/wr/webrender/src/display_list_flattener.rs => gfx/wr/webrender/src/scene_building.rs
extra : moz-landing-system : lando

Bug 1581058 - Rename scene_builder::SceneBuilder into scene_builder_thread::SceneBuilderThread. r=gw

Differential Revision: https://phabricator.services.mozilla.com/D45854

--HG--
rename : gfx/wr/webrender/src/scene_builder.rs => gfx/wr/webrender/src/scene_builder_thread.rs
extra : moz-landing-system : lando

Differential Revision: https://phabricator.services.mozilla.com/D45855

--HG--
extra : moz-landing-system : lando

Bug 1578468 - Skip fission mochitests and sw-e10s xpcshell tests for DevTools try presets r=ochameau

Depends on D45843

Differential Revision: https://phabricator.services.mozilla.com/D45844

--HG--
extra : moz-landing-system : lando

Differential Revision: https://phabricator.services.mozilla.com/D45843

--HG--
extra : moz-landing-system : lando

Differential Revision: https://phabricator.services.mozilla.com/D45835

--HG--
extra : moz-landing-system : lando

Bug 1540029 - part 2: Get rid of `TextEditRules::DocumentIsEmpty()` and `HTMLEditRules::DocumentIsEmpty()` r=m_kato

`TextEditRules::DocumentIsEmpty()` is a wrapper of `TextEditor::IsEmpty()` so
that we can get rid of it simply.

`HTMLEditRules::DocumentIsEmpty()` needs to change.  It's oddly checks only
`EditorBase::mPaddingBRElementForEmptyEditor` is `nullptr` or not.  However,
the editor may be completely empty.  And the result may be different from
`TextEditor::IsEmpty()` which is not overridden by `HTMLEditor`.  For partially
solving this issue, this patch makes `HTMLEditor` overrides `IsEmpty()` and
optimizes `TextEditor::IsEmpty()`.

With this change, the caller of `HTMLEditRules::DocumentIsEmpty()` may behave
differently in the only caller, `HTMLEditor::SelectEntireDocument()`.  And
unfortunately, its root called from `SelectAllCommand::DoCommand()`.  However,
it does just collapse `Selection` into the root element (`<body>` or
`Document.documentElement`) if it returns `true`.  Therefore, this change
must be safe since anyway `SelectionRefPtr()->SelectAllChildren()` with
the root element is exactly same as `SelectionRefPtr()->Collapse()` with
the empty root element if it's truly empty.

Differential Revision: https://phabricator.services.mozilla.com/D45784

--HG--
extra : moz-landing-system : lando

ServiceWorker mochitests fail for Fission is fixed by bug 1542415.
Remove the skip-if for tests

Differential Revision: https://phabricator.services.mozilla.com/D46030

--HG--
extra : moz-landing-system : lando

Bug 1580496 - Return NS_ERROR_FAILURE while getting nullptr PaymentRequestChild for PaymentRquest r=baku

PaymentRequestChild could be nullptr while PaymentRequest's owner is nullptr.

Differential Revision: https://phabricator.services.mozilla.com/D46012

--HG--
extra : moz-landing-system : lando