Otherwise, it changes the move-to-rect inputs, which can change the
output as well, making us move the anchor all the way to the right.

You could, I guess, consider this a mutter bug of sorts, because it
feels weird that you pass it an anchor that has been a `move-to-rect`
output and you get another rect as an output.

But also, it's kinda silly that we're doing that to begin with, so avoid
it by telling the popup frame whether it's been positioned / moved by
move-to-rect (and keeping the anchor in that case).

The reason this works on my setup without "Large text" is just dumb luck
(the front-end computes a max-height for the panel that is small enough
to fit on the screen).

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

Backed out 10 changesets (bug 1787794) for causing build bustages on BytecodeEmitter.cpp CLOSED TREE

Backed out changeset 2a4095902192 (bug 1787794)
Backed out changeset ee76c50ba4b7 (bug 1787794)
Backed out changeset 8fbfb862f801 (bug 1787794)
Backed out changeset cd0dd29a5c65 (bug 1787794)
Backed out changeset 71436fb16132 (bug 1787794)
Backed out changeset 4abc0092802a (bug 1787794)
Backed out changeset 39525ff15d29 (bug 1787794)
Backed out changeset b495ed555e0d (bug 1787794)
Backed out changeset f5e5764cc4d3 (bug 1787794)
Backed out changeset 5e9c66da7409 (bug 1787794)

There isn't a good reason to keep a simple `while` loop in a separate function.

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

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

`TokenStreamPosition` no longer contains GC things.

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

Instead of branching around `parenFreeArrow` in multiple places, handle the
paren-free arrow function case in a separate if-block. This also simplifies
the `Token::Modifier` handling, because we can always pass a constant.

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

`emitDestructuringLHSRef()` and `emitSetOrInitializeDestructuring()` no longer
need to inspect `target` to handle spread and assignment expressions when we
directly pass the unpacked parse nodes.

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

Similar to part 3.

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

BigInt property keys are parsed as synthetic computed properties, so these
cases no longer apply.

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

The property is most likely a normal property key and not a number, so let's
handle the normal property key case first.

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

Directly pass `lhs` to `emitSetOrInitializeDestructuring()`.

Part 6 will further align Object and Array destructuring code.

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

This shaves off one byte in byte code size:

`dis(function(){ [...a] = [] })`

Before:
00092:  27  True                        # [] [][Symbol.iterator]().next [][Symbol.iterator]() [] true
00093:  27  Unpick 1                    # [] [][Symbol.iterator]().next [][Symbol.iterator]() true []
00095:  27  TryDestructuring            # [] [][Symbol.iterator]().next [][Symbol.iterator]() true []

After:
00092:   1  True                        # [] [][Symbol.iterator]().next [][Symbol.iterator]() [] true
00093:   1  Swap                        # [] [][Symbol.iterator]().next [][Symbol.iterator]() true []
00094:   1  TryDestructuring            # [] [][Symbol.iterator]().next [][Symbol.iterator]() true []

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

`CanReuseScriptForClone()` is now only used for an assertion.

Depends on D155838

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

Drive-by changes:
- Use `needsPrototypeProperty()` in `fun_enumerate`, because it's more
  efficient and more closely follows `fun_resolve`.
- Remove unused function `fromShapeFieldPointer()`.

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

Bug 1784244 - do not surface sync errors in Firefox View if we previously synced successfully, r=sfoster

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

Bug 1786465 - Introduce FileSystemManager intended to be referenced by FileSystemHandle; r=dom-storage-reviewers,jesup

This patch also splits the creation of the top level protocol and getting the
root handle into separate logical steps.

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

This has been added many years ago when introducing side hosts.
We should now consider this a builtin feature.

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

ar -> e147b5263af57e873edadeb2bf5a3f4d8d2b8ea9
hi-IN -> a819112781d03d71928892ecbde249b896e261a5
hsb -> 7912e365c809d3f27179cd7e25622819d78db256
nb-NO -> 42def68deae9b735225ffea30ee7b7c3d136dbdc
sat -> d2ce5f9aab7d15ed300b778cf305663a54d651ce

Bug 1787953 - Update the value of the epsilon used in SampledAPZCState::RemoveFractionalAsyncDelta() to 0.01 as well. r=tnikkel

This epsilon was split off from COORDINATE_EPSILON in bug 1787099,
but the original value used here (before bug 1659642) was 0.01
as well.

Depends on D155904

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

The previous change to 0.02 appears to have been unintentional,
and the value should be less than 1 app unit (1/60 of a pixel).

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

Bug 1704978 - Prevent opening context menu in the case the user did right click in the overscroll gutter on Windows. r=botond,handyman

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

The comment about NS_IF_RELEASE() is now useless since there's no
NS_IF_RELEASE() call.

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

Bug 1787964: Improve content-based-sizing and vertical centering of Firefox View's "Last Active" badge. r=Gijs

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

This is a first basic implementation that we might extend in the future
(for example by adding a way to select the container node when clicking
on the container name).

A test is added to ensure we display container queries as expected, even
when dealing with unknown containers, or with nested layer/media/container rules.

Depends on D155602

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

Bug 1787386 - The arrow from the “Skip this step” button has a black color when the browser's dark theme is enabled r=emcminn

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

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

This patch is the result of `mach esmify --convert toolkit/mozapps/extensions/internal/GMPProvider.jsm`.
Callsites were fixed manually, as well as occasional references in comments.

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

There were cases that the function was called when CompositorThreadHolder was not active

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

Backed out changeset c0865c4d8f2f (bug 1787290) for causing xpc failures in backgroundtasks/tests/xpcshell/test_backgroundtask_targeting.js CLOSED TREE

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

Bug 1787621 - Fix contrast on the about:addons Available Updates count badge, when in dark mode r=sfoster

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

Depends on D154851

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

Historically, we had configure.in in the tree, and generated configure
with autoconf via client.mk at build time, and we'd run that configure.
As a side effect, configure would be shipped in source tarballs, as with
traditional autoconf-driven software.

Bug 1671424 changed that, and made `mach build`/`mach configure` invoke
configure.py directly, without needing configure. As a side effect,
configure stopped being generated in source directories. An unexpected
side effect was that configure stopped being shipped in source tarballs.

Anyways, it has been long enough now that adding a configure file in the
tree shouldn't cause too many problems with people updating their trees
where a configure file might exist from those older builds.

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

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

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

Windows a11y clients retrieve MathML markup using ISimpleDOMNode::innerHTML.
We cache innerHTML to support this, but only on math elements and only on Windows.
sdnAccessible had to be modified to support RemoteAccessible and to use the cache for this method.

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

Bug 1787686: Expose the URL of a link as its value, as well as the value of text/image leaf descendants. r=morgan

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

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

Bug 1787321 - convert nsAHttpConnection::GetSecurityInfo into ::GetTLSSocketControl r=kershaw,necko-reviewers

This continues work started in bug 1784098.

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