Bug 1797840: Replace stray tab characters with spaces in IDL files throughout the tree. r=mccr8,necko-reviewers,credential-management-reviewers,smaug,sgalich,valentin

DONTBUILD because this is just whitespace cleanup.

I found the files to fix up here using this command:
    grep -r '   ' * 2>/dev/null | grep -v "other-licenses" | grep "idl:"

I replaced the tab characters with however many spaces seemed consistent with
the indentation in the surrounding code (and did some minor space-indentation
cleanup in contextual lines to preserve alignment, in a few cases).

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

Bug 1797148: Simplify checkVisibility API (on nsISelectionController and nsIFrame) into a single nsTextFrame::HasVisibleText method. r=masayuki

Before this patch, we had two `checkVisibilty` methods on the
nsISelectionController interface, backed by several layers of implementation,
ultimately backed by a single function on nsTextFrame (which didn't actually
do anything meaningful with any of the parameters).

As it turns out, this API only had one caller, in HTMLEditUtils.cpp.

This patch converts that caller to directly query nsTextFrame (if the given
node's primary frame is indeed a nsTextFrame).  The direct function-call is
renamed to HasVisibleText(), to be a bit clearer about it being text-specific
and also to avoid confusion with the (unrelated) recently-specified HTML
checkVisibility() API.

With these changes, we can remove the API from the nsISelectionController
interface and its implementations.

This patch also updates the HTMLEditUtils::IsInVisibleTextFrames documentation
(with s/all/any/) to reflect the reality of what the nsTextFrame impl actually
does.

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

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

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

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

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

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

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

Bug 1797755 - Part 5: Use a single initial mark stack size regardless of whether incremental GC is enabled r=sfink

Currently we initialize the mark stack to a different size depending on whether
or not incremental GC is enable. However, after the first GC we always shrink
it to the initial size when it is disabled.

This bug shows that there's no point having separate sizes so this patch
removes the option and uses a single size regardless.

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

Currently we have a GC parameter that allows setting a maximum mark stack
capacity. This is only ever used by test code, not in the browser. This
requires extra unnecessary work in release builds if we move to a different
stack representation as we won't be able to fold the comparison into the
current capacity check as we do now.

The patch makes this feature condtional on JS_GC_ZEAL.

Depends on D160526

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

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

Currently the way mark colors work for delayed marking doesn't align with
normal marking for gray marking of GC things that can only be marked black
(e.g. strings).

Normal marking keeps these on the gray mark stack, but when it comes to mark
them they are marked black. Currently OOM during marking pushes arenas
containing such GC things onto the delayed black marking list. This means that
gray marking can cause delayed black marking. This is surprising and an
unnecessarily additional complication.

The patch makes this work the same as normal marking. OOM during gray marking
always pushes arenas onto the gray marking list; arenas on the gray marking
list are marked black if they are a GC thing kind that cannot be marked gray.

Depends on D160524

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

Parallel marking will use one GCMarker per thread. The testing mark queue is
really a per-runtime data structure, so this patch moves it to the GCRuntime.

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

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

Bug 1791762 - CreateFileInputStream, CreateFileOutputStream and CreateFileInputOutputStream should return MovingNotNull<T> instead of NotNull<T>; r=dom-storage-reviewers,jesup

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

Bug 1797631 - Removing the code implemented for the new metric capture experiment ping. Reverts back to previous state.  chutten r=chutten

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

Bug 1794435 - disable browser/components/firefoxview/tests/browser/browser_tab_pickup_list.js on Linux debug r=intermittent-reviewers,jmaher DONTBUILD

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

Bug 1788720 - Expose private browsing flag in StorageController.getPermissions r=geckoview-reviewers,jonalmeida

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

Bug 1795954 - Implement the Initial Onboarding Feature in GeckoView Example app r=geckoview-reviewers,ohall,amejiamarmol

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

Bug 1510561 - Part 10: Apply `plugin:mozilla/require-jsdoc` to `browser/components/pagedata`. r=Standard8

Depends on D160634

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

Bug 1510561 - Part 9: Apply `plugin:mozilla/valid-jsdoc` to `browser/components/pagedata`. r=Standard8

Depends on D160633

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

The two rules here (`require-jsdoc` and `valid-jsdoc`) will likely be
removed in the not-to-distant future, so I am cleaning up their usage.

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

This prevents a11y from getting instantiated shortly after clipboard paste, in
order to prevent hangs with the Windows 11 suggested actions feature.

When combined with the previous patch, the behavior is the following:

 * For users with a11y already-enabled:

   * No hang (due to clipboard flush).
   * Quick actions menu is positioned at selection offset.

 * For users with a11y disabled (most):

   * No hang (due to no a11y instantiation + clipboard flush).
   * Quick actions menu is positioned at pointer (cursor) offset.

Co-Authored-By: Emilio Cobos Álvarez <emilio@crisal.io>

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

Depends on D160646

Bug 1774285 - On Windows 11 22H2, flush the Windows clipboard immediately after setting it. r=neildeakin


This works around a windows 11 suggested actions bug, see comment.

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

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

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

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

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

Update Feature Callout code to ensure styling is the same in browser chrome as it is when used in content pages such as Firefox View.

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

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

Bug 1769265 - Disabling updater for packages should not be Windows specific. r=bhearsum,settings-reviewers,application-update-reviewers,bytesized

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

We don't need mEvent to wait for IO completion. Instead, we just cancel
it immediately.

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

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

Bug 1794722 - Part 3. Remove all uses of GfxInfo::GetWebRenderEnabled. r=jrmuizel,webcompat-reviewers,twisniewski

This is always true.

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

Bug 1794722 - Part 2. Remove all uses of gfxVars::UseWebRender(). r=jrmuizel,media-playback-reviewers,alwu

This is always true.

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

WebRender is a mature feature. We should start blocking it on known bad
devices over allowlisting known good devices. This may enable WebRender
in a few more obscure places than we shipped before.

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

MANUAL PUSH: Orange fix CLOSED TREE

Backed out changeset 10883f7ad411 (bug 1104311) for causing wpt failures on data-uri.htm. CLOSED TREE

Backed out changeset 7cda151d97a5 (bug 1797755)
Backed out changeset 098569654f17 (bug 1797755)
Backed out changeset 6e5a935e576a (bug 1797755)
Backed out changeset eb109d0ce408 (bug 1797755)
Backed out changeset 186a012563fc (bug 1797755)

As per the discussion in https://github.com/whatwg/html/pull/8347

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