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

Depends on D108345

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

Also move MOZ_MUST_USE before function declarations' specifiers and return type. While clang and gcc's __attribute__((warn_unused_result)) can appear before, between, or after function specifiers and return types, the [[nodiscard]] attribute must precede the function specifiers.

Depends on D108344

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

Bug 1698706 - Always add PIC flags to C{,XX}FLAGS. r=firefox-build-system-reviewers,andi,sheehan,mhentges

The build always uses them anyways, but configure tests don't, and some
of them fail as a consequence in some cases (example: when UBsan is
enabled, all AC_LANG_CPLUSPLUS + AC_CHECK_FUNCS tests fail). We also had
similar problems in the past, such as bug 1513605.

By adding the PIC flags to CFLAGS and CXXFLAGS, we ensure old-configure
tests use the flags as well.

While here, we also remove DSO_CFLAGS, which was always empty, and we
stop passing -fPIC to wasm compiles, because it has no effect there.

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

Bug 1697950 - Partial support for mach bootstrap on arm64 mac. r=firefox-build-system-reviewers,sheehan,mhentges

This is enough to make the instructions on the Google doc unnecessary.

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

Bug 1698546. Don't allow double tap to zoom on mac when context menus are open in the same situations that we don't allow pinch zooming. r=mstange

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

Bug 1698706 - Always add PIC flags to C{,XX}FLAGS. r=firefox-build-system-reviewers,andi,sheehan,mhentges

The build always uses them anyways, but configure tests don't, and some
of them fail as a consequence in some cases (example: when UBsan is
enabled, all AC_LANG_CPLUSPLUS + AC_CHECK_FUNCS tests fail). We also had
similar problems in the past, such as bug 1513605.

By adding the PIC flags to CFLAGS and CXXFLAGS, we ensure old-configure
tests use the flags as well.

While here, we also remove DSO_CFLAGS, which was always empty, and we
stop passing -fPIC to wasm compiles, because it has no effect there.

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

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

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

Bug 1698719 - Remove aSkipFrames argument to both FramePointerStackWalk and MozStackWalkThread. r=gsvelto,gerald

In the case of FramePointerStackwalk, the caller gives a pointer to the
top-most frame to walk from. There isn't really a reason to give a
number of frames to skip, as the right frame pointer could be given in
the first place if that was really necessary. And in practice, it's
hasn't been used so far.

In the case of MozStackWalkThread, the caller presumably doesn't know
what the thread the stack is being walked for is doing, and it would be
a guesswork to pass a valid number of frames to skip. In practice, it's
also not used.

The aSkipFrames is already a footgun on MozStackWalk (and we're going to
change that in bug 1515229), we don't need to keep a footgun on these
other stack walking methods.

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

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 two calls that I'm migrating are inside of
EndSwapDocShellsForViews, which gets called when a tab is dragged between
windows.  I'm annotating these two calls as OK, because:

- the first call is about maintaining the NS_FRAME_IN_POPUP state, which is
  used for things like the menulist-dropdown popup.  This bit doesn't need to
  be propagated across process boundaries.
- the second call is about propagating a "needs-paint" notification up to
  ancestor documents. I think we already handle paint invalidation for
  cross-process iframes properly, independent of the explicit invalidation that
  we're doing here.

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

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 XUL code
(specifically for the rendering of XUL trees); and we only expect to be
rendering XUL in a single process: Firefox's parent process. So this code
should be entirely OK with the fact that this API doesn't cross process
boundaries.

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

Bug 1698534 - Go back to properly doing exact matching for quick suggest, and rewrite flatten logic. r=daleharvey

The matching logic can be simplified since we want exact matches and not prefix
matches. There's no need to confirm that the query string is in a candidate
result's list of keywords because by its nature the process of looking up the
query in the tree confirms that the query is equal to some keyword. If the
lookup finds a result, then necessarily the query matches a keyword.

I changed lookup from recursive to iterative. It might be a little faster and
it's not really harder to understand.

I also moved the full keyword computation out of the keyword tree and into
UrlbarQuickSuggest. It was a little hacky how KeywordTree accessed
`UrlbarQuickSuggest._results` during its lookup, and it forced the test to have
to set up `UrlbarQuickSuggest._results`. And big picture, it's not the keyword
tree's job to compute the full keyword/suggestion.

While working on all of this, `test_flatten` started failing when it calls
`basicChecks` because the flattened tree wasn't correct, so I rewrote and
simplified the flatten routine. It's no longer necessary for `RESULT_KEY` to be
in a map all by itself. Instead, it's treated like an ordinary char in the
context of flattening, so now it's more like a sentinel or suffix than a key.

Finally, I removed the "test1" and "test2" data in the test because I added them
when I thought they tested bug 1697678, but they don't actually. I don't think
they're testing anything useful.

Depends on D108683

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

Replace with Maybe<YUVColorSpace> where still needed.

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

Bug 1691153 - Using the blob's principal for BlobURLRegistrationData creation in ContentParent::TransmitBlobURLsForPrincipal. r=asuth

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

Expose adapter limits, hook up spirv-cross

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

Bug 1698969 - Add some colors to HeadlessLookAndFeelGTK so that some widgets render properly in headless mode. r=mstange

Colors match Adwaita (the default GTK theme).

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

This matches pre-existing behavior.

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

There's no anonymous textfield since a long time ago.

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

It broke with bug 1698043 because we removed one wrapper.

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

The root cause of this bug is that we're now accidentally using the
native theme for the spinners because GetNumberControlFrameForSpinButton
broke with my changes.

These pseudos are not exposed to content, and there's just one user of
this pseudo-class (the print.css one).

So just disable native appearance explicitly and remove that check.

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

Backed out changeset f82846b236e3 (bug 1697863)
Backed out changeset 64166dc7f85f (bug 1697863)

Careless of me to not have tested this more thoroughly. I guess this is hidpi
only because automation has aScale = 1.0, which is why automation didn't catch
this? That's sad :(

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