Bug 1704572 - Use BrowserTestUtils.is_visible rather than checking the bounding client rect in browser_contextmenu_linkopen.js. r=Gijs

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

Bug 1704572 - Correctly wait for popup state changes in browser/components/places/tests/browser/browser_check_correct_controllers.js. r=Gijs

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

Bug 1704572 - Use openMenu(true) instead of openPopup() for opening the 'this frame' submenu in browser/base/content/test/contextMenu/browser_contextmenu_iframe.js. r=Gijs

openMenu(true) works on both native and non-native context menus. openPopup() does not work on native context menus.

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

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

Bug 1686315 - Fix up the last few tests to pass with content prompt subdialogs enabled or disabled. r=Gijs

Critically, this makes sure SubDialog.abort calls SubDialog.close even if its contents haven't finished
loading. This prevents a shutdown leak.

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

Batch 2 of 2.

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

Batch 1 of 2.

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

Bug 1704576 - Make the CacheIR plain object op handle failure by calling into the VM rather than failing the stub r=jandem

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

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

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

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

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

Backed out changeset 86f0423265b7 (bug 1695963) for failures on browser_ext_management_themes.js. CLOSED TREE

Bug 1405708 - Fallback -moz-buttonhoverface to the regular button background if transparent. r=stransky

It seems we get transparent in Adwaita otherwise, which causes this.

This is fixed on nightly via the non-native theme, which spoofs these
colors for contrast, but seems worth fixing anyways.

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

This will cause nsXULPopupManager to change the state to "open" before the
popupshown event fires. This is consistent with non-native menus.

This was caught by dom/html/test/browser_content_contextmenu_userinput.js.

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

Bug 1704494 - Enable the ascent-, descent-, and line-gap-override descriptors on all channels. r=emilio

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

Glean is throwing an error if a string metric is `None`.
Since `get_cpu_brand()` can return `None`, only update telemetry if it's
set.

Depends on D110847

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

Bug 1703055: Don't include global site packages in python path r=firefox-build-system-reviewers,glandium,sheehan

To avoid erroneously importing a package from the system, we remove
system site package.
This is another positive step towards further insulating the Firefox
build from the state of the environment.

This may cause failures if some mach commands have been sneakily
importing system packages instead of properly vendoring them.

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

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

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

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

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

Bug 1702689 - Fix browsertime_benchmark.js to collect all pagecycle results r=perftest-reviewers,sparky

Fixed browsertime benchmark script to correctly run the tests, fix python code to collect all the pagecycles data and disable unity-webgl test running on windows7.

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

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

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

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

This change converts some heap-unclassified memory to js-content. These show
up in DMD reports now that Set/Map are used in the browser itself.

As a side-effect, this adds more precise tracking to UBI system for tenured
objects. The nursery objects don't have the expected size but this is
pre-existing similar to typed objects. Due to this mismatch it is difficult
to write automated tests for this.

Also mark the Map / Set types as using delayed metadata notification so that
reserved slots can be fully initialized before debugger is notified. This
only delays things to a few function calls later.

Depends on D111753

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

The `Zone::regExps_` field is a UniquePtr so we should use the including-this
variant of memory reporters for it. This was previously heap-unclassified.

Depends on D111752

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

This moves some memory accounting from heap-unclassified to being part of the
runtime-atoms accounting like `commonNames`.

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

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

Bug 1698541 - [devtools] Stop emitting now unused tabDetached event. r=nchevobbe,devtools-backward-compat-reviewers

Now that we are destroying the target via target-destroyed-form, sent via the watcher actor
we no longer need this legacy event.

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

This better highlight that may only be the transport that is closed and not the whole client.
But at the end, both should follow the same lifecycle and be destroyed altogether.

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

This will better highlight what is closing. Not the DevToolsClient, but just the transport class.

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

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

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

Bug 1704092 - Disable chrome/chromium tests on mac 10.15 for perma-failures. r=perftest-reviewers,Bebe

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

Bug 1704474 - Remove pin/unpin page action context menu items. r=adw,fluent-reviewers,extension-reviewers,flod,zombie

The bug calls for these items to be hidden with JS, but they were going to be removed anyways post-Proton. The removal of some subtests in browser/base/content/test/pageActions tests is consistent with [this comment](https://searchfox.org/mozilla-central/rev/d9f6cded535d202a9ade4a530e653e659bcb5bbd/browser/base/content/test/pageActions/browser.ini#7), which says that were are removing that test coverage post-Proton anyways.

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

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

This patch tries to address the issue that legacy CJK extensions have various
extended variants where the core of the encoding is compatible but the edges
are incompatible. Without this patch, we reject e.g. Big5 if it has a single
character from the UAO extension or a single Windows end-user-defined character.

Likewise for the other legacy CJK encodings.

This patch tolerates:

* All Big5 extensions (the motivating part of this patch).
* Windows EUDC for EUC-KR.
* Classic Mac OS extensions to Shift_JIS, EUC-KR, GBK, and Big5 to the
  extent practical considering conflicting definitions of what constitutes
  a lead byte in the Encoding Standard but a single-byte extension in
  Classic Mac OS.
* JIS X 0213 / 2004 extensions to Shift_JIS and EUC-JP. (It's unclear if
  these have actual deployment.)

Tolerating means that the occurrence of an extension character doesn't
disqualify a candidate but only applies a penalty to the pending score.
If there is enough other convincing content, it should be able to overcome
the penalty.

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

Bug 1697769 - Apply implicit pointer capture only for pointer events that are generated from the touch events; r=smaug

Pointer capture would override the capturing content set in PresShell for mouse
event, it isn't affect touch event because dispatching touch event would take
the capturing content in PresShell into account, i.e.
https://searchfox.org/mozilla-central/rev/fa48ebee58d59fa846919c3e2c3122b08db57c9c/layout/base/PresShell.cpp#8712-8719

Touch input isn't always generating a touch event, but a compabitility mouse
event with the touch as inputsource, e.g. chrome popup window on Windows platform,
see https://searchfox.org/mozilla-central/rev/3de2db87f3c9001ae478318d47a2ca3427574382/widget/windows/nsWindow.cpp#8130-8134
In such case, we don't apply implicit pointer capture to prevent the capturing
content set in PresShell being overried which could cause the drag to scroll
doesn't work.

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