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

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

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

Server-Timing was only being parsed with HTTPS. But this is overly
restrictive to developers, so it's better to restrict it to
secure origins which includes `http://localhost/`

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

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

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

Safari does this. This reduces the runtime in the example linked from
comment 0 quite a lot (40ms on a local opt build, from ~130ms on a
release nightly build).

I added a pref because there's a slight chance of performance
regressions on pages that do not use attribute selectors, as we're now
doing more unconditional work per element (adding the attributes to the
bloom filter). But the trade-off should be worth it, I think.

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

I need to use these from Rust to insert mapped attribute names in the
bloom filter, and this would save me handrolling even more bindings :)

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

Backed out changeset ba593c7436f6 (bug 1689730) for failures on test_http_server_timing.js. CLOSED TREE

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

Bug 1702467. Double tap zoom can make us zoom to a part of an element when we could fit the entire element at the same zoom. r=botond

If we double tap on an element that is narrower than the viewport at maximum we can get into a situation where we zoom on part (say the bottom half) of that element but we could easily fit the entire element.

This happens because the code that calculate the rect to zoom to (CalculateRectToZoomTo) doesn't know about the maximum zoom. So it proceeds as though we will fit the width of the element. Under that assumption we will have to cut off part of the element vertically, so the code centers the rect on the users tap point.

This ends up cutting off part of the element vertically when it is clear that the whole element can fit on screen, which is a pretty ugly result.

This is not my favourite patch. This seemed to make the most sense. Another option I considered was passing the tap point through to AsyncPanZoomController::ZoomToRect but I think this approach came out better: the calculation all happens in one place at one time.

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

Server-Timing was only being parsed with HTTPS. But this is overly
restrictive to developers, so it's better to restrict it to
secure origins which includes `http://localhost/`

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

Bug 1702179 - nsIndexedToHTML generated views - focus ring for directories only shows the right margin r=Gijs

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

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

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

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