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

Name of Singleton is commonly used in gecko.

Name of SharedGL is confusing. It looks like "shared context", but it is not. The GL context is actually a singleton GLContext that is used by all WebRender. Each window creates one EGLSurface for each window. EGLSurface is switched for each window rendering.

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

Bug 1642599 - [devtools] Enable all console messages server support everywhere. r=ochameau,devtools-backward-compat-reviewers.

When enabling the watcher support, we end up having duplicate platform messages as we
received them both from the content process and the parent process (the messages are cloned).

A couple test cases are added to ensure we handle all kind of messages properly.

Depends on D107648

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

We're getting those messages directly from the frame targets, so we can safely
ignore them in the content process.
A test case is added in a following patch in this queue.

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

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

The patch for Bug 1697453 made it so we weren't dispatching resources tied to
a target being destroyed. This revealed an issue when adding a test for webextension
popup messages, where the target is being destroyed before the browser console can
consume it (see following patches in this queue).
Since Bug 1697453 was only done to try to make `browser_toolbox_backward_forward_navigation.js`
pass when fission is enabled, and given it's still failing, let's revert this
patch for now and reassess if this is needed, and if so, if we shoul only do that
for some specific resources.

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

Bug 1642599 - [devtools] Don't use watchers in browser console/toolbox if devtools.browsertoolbox.fission is disabled. r=ochameau.

We bail out early in TargetList#hasTargetWatcherSupport and ResourceWatcher#hasResourceWatcherSupport
if we're in the browser toolbox or browser console and the pref is disabled.

Depends on D107410

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

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

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

Bug 1698543 - Rework handling of interaction between incremental barriers and rope flattening r=sfink

Currently we have a special path for performing barriers during rope flattening to take account of the fact that this overwrites the cell header word of ropes temporarily, making these cells untraceable.

The problem is that a rope can already be in the barrier buffer before flattening starts and so we need to check for this when processing the buffer.

The patch reworks use of temporary GC data to set one of the cell header flags when in use. This means we can check for it when processing the barrier buffer. It removes the special case function for barriers, which makes it slightly less efficient as we will now buffer all these ropes before skipping them later on. Hopefully that is not too deterimental to performance.

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

Bug 1699036 - Fix build error on (symbol 'profiler_is_locked_on_current_thread()' is multiply-defined) on platforms with MOZ_GECKO_PROFILER not defined, r=gerald. DONTBUILD

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

Backed out changeset 801dec9ebbd7 (bug 1697865) for causing frequent crashes on machines with dedicated graphics cards. a=backout

Bug 1696397: Move SpinEventLoopUntilOrShutdown to -Quit and move the current shutdown state logic from nsThreadManager to AppShutdown; r=kmag,xpcom-reviewers

The SpinEventLoopUntilOrShutdown is interrupted in the main process only at stage "xpcom-shutdown", which is too late. In a content process instead we interrupt it early enough, at "content-child-will-shutdown".

This patch removes the current observer logic in nsThreadManager in favor of a current shutdown phase state machinery in AppShutdown and renames SpinEventLoopUntilOrShutdown to SpinEventLoopUntilOrQuit. It also ensures that SpinEventLoopUntilOrQuit breaks early (at "quit-application" in the main process).

For a further possible cleanup around shutdown state in child and parent process please refer to bug 1697745.

We have no automated test for this edge case, but we have diagnostic telemetry that will tell us if it helps.

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

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

This test can oom on some underprovisioned 64-bit platforms like those in our CI,
so allow that as an outcome.

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

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

Bug 1697115 - Rationalize QuotaManager::DecreaseUsageForClient arguments; r=dom-storage-reviewers,sg

Note that the method DecreaseUsageForOrigin has been renamed to
DecreaseUsageForClient.

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

Bug 1697115 - Rationalize QuotaManager::NoteOriginDirectoryCreated arguments and return type; r=dom-storage-reviewers,sg

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

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

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

Bug 1697115 - Use FullOriginMetadata instead of OriginMetadata in QuotaManager::LoadQuota::LoadQuotaFromCache; r=dom-storage-reviewers,sg

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

Bug 1697115 - Rename mTimestamp to mLastAccessTime in FullOriginMetadata; r=dom-storage-reviewers,sg

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

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

MozPromiseHolder stores only a single pointer and so there is no advantage to
having an allocation and additional pointer indirection.

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

There is a change in behavior that the SourceListener::Stop() is now run off a
queued event, but timing is not urgent.

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

Denied() is called only from Observe(), which is main thread only.

The comment "Should happen *after* error runs for consistency, but may not
matter" was obsolete as of
https://hg.mozilla.org/mozilla-central/rev/46c936ce7efed56027d7770690f94ee31f415d7a#l3.634

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

MozPromise can be rejected from any thread.

"Do after the above runs, as it checks active window list" was obsolete as of
https://hg.mozilla.org/mozilla-central/rev/99426a6a77bf8ddfb334ed052eb18e2a9bc48122#l3.223

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

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

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

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

Bug 1697970 - Disable test_arraybufferinputstream_large.html on linux asan for permafailing. r=jstutte,necko-reviewers,valentin

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

Bug 1691402 - [marionette] Don't try to destroy a non-existent session. r=marionette-reviewers,jdescottes,jgraham

If there is no active WebDriver session the code
related to deleting a session, and resetting other
state should not be run again.

This could actually happen when the
"Marionette:Quit" command is called, which itself
destroys the session, and the follwing socket
connection drop tries to run it again.

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

Bug 1691402 - [marionette] Move current browsing context members to WebDriver session. r=marionette-reviewers,jdescottes,jgraham

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

Bug 1691402 - [marionette] Add session module for WebDriver session. r=marionette-reviewers,jdescottes,jgraham

This patch creates a new module for the WebDriver session
specific implementation. It no longer requires session data
like capabilities to be directly bound to the GeckoDriver class.

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

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

This patch doesn't change behavior; GetCrossDocParentFrameInProcess() is just a
wrapper for GetCrossDocParentFrame(), which is what we were calling before.

The "InProcess" version of this API (which we're migrating to in this patch) is
used to annotate GetCrossDocParentFrame() callsites that have been vetted as
being OK with the fact that this API returns null at the boundary of a
cross-origin iframe, if fission is enabled.

In this patch, the one call that I'm migrating is inside of
IsFrameDescendantOfAny(), which is a helper-function called by
TextOverflow::PruneDisplayListContents(), which is a function that we use to
traverse a display list and hide content that's been hidden via
"text-overflow:ellipsis" or "-webkit-line-clamp".

We're OK to migrate this call to the "InProcess"-flavored API, because it's
just looking back up the frame tree from the display items that we can reach in
our traversal, which are trivially all part of the same process.

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

Bug 1697991 - Enable Dark Mode support in the crash reporter client, and fix a case of black-on-black text. r=mac-reviewers,bradwerth

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