Bug 1779645 Part 1 - Switch CSS named page fragmentation to happen at reflow instead of frame construction r=dholbert

This also adds a small post-processing step for frame-construction to ensure
that replaced frames (or more generally frames with content but no children)
have the auto page-name set on them.

When page-name value tracking is switched to be lazy, this post-processing step
will likely be redundant and can be removed.

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

Bug 1788790 - Using TimeDelta instead of TimeStamp to allow for negative values when given non spec ntp;r=pehrsons

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

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

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

Make it a bit easier to read and less prone to race conditions. Remove a
setTimeout referencing bug 103197 which I'm pretty sure it's not an
issue.

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

Depends on D156241

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

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

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

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

Bug 1790331 - Clarify why pre-loading of oleaut32.dll is necessary. r=bobowen,media-playback-reviewers,alwu

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

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

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

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

I also cleaned up browser_aboutwelcome_multistage_mr.js so that each task is
idempotent and does not pollute the state of other tasks (e.g., by leaving
prefs set).

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

What we are doing:
- Disabling the view/main-fenix and the cpu-memory jobs

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

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

Bug 1786927 - Consolidate the way we trigger autocomplete popup in tests. r=credential-management-reviewers,sgalich

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

We enable the sanitizing, which will omit the value in the
content process and trigger event telemetry - but we won't
crash the process.

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

Bug 1789421 - Generate crash reports for content process shutdown hangs only when we can confirm that the process did not shut down in time r=jstutte

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

dsb -> dce20589cb2b50184b90c2ba268deb7ae7607e9c
el -> 93b03feb3781aa3bfb368a22e4141c158a75b0c8
fi -> ee708cf4cb4b69da19503a153ed364915c188328
fr -> 10590b4850974ae25902260b3470dde958a7e709
he -> 7b779cfedacb2ee09158ebad8c092d54ddc879d6
hsb -> 88061c0806a5ee259f381b85bc7ffb5bc9da6363
hu -> f518d16bfae6e0e025d5345e5e42f48b29c76d8e
ia -> 75edcbb9ccc4540cbebc9b3280aa32e2616a72c0
kab -> 68833491b3ad425d8d58fb878e4700b2e899fb50
ko -> 095264e2a58972eeb8869c619ab8c68ad03b65b7
lo -> b4a40798001e3c9728db045423f142ebc7402008
nb-NO -> 511b77203d9996bfd7b1cc8b30a6cfa59dfea4b7
nn-NO -> 26300c62e29acffe00ff37672de5ad9de0026f1b
pt-BR -> ef938c665534c91cf42d981c331ab56a30b73629
pt-PT -> c21930309ca6b97bfb1fd2533e32fc870dcfdbe2
rm -> 3d9909e64c0bd82629ee71f9ee50a9260dadb84b
ro -> 810ae99530cb5b0174f02c582a47ca44793fca83
ru -> 9f7799b135bd485ae90f4cfbb3a8bfb1a73e826b
sk -> b142a5722b24b87013f2dc390b92c6c4d7546971
sl -> 12862f3a06eac20a666b2c86e368676ae6f71398
te -> 3474c18c64036e42981b78e31581182080415331
tg -> 782228966a3f0f80ea33b2437801d4380f659320
zh-CN -> a56db11b90878da1fee63443d2393af299cc1334
zh-TW -> 3230bc1cd94d3b8434d3a1cdfdcd70f716ea2da2

In the recent refactor of this (https://bugzilla.mozilla.org/show_bug.cgi?id=1782295) I managed to break our ability to create shortcuts if they don't already existed. This was caused by two things:
1) An errant `NS_SUCCEEDED` call remained after the call it was checking moved elsewhere (oops)
2) The taskbar pinning code was not updated to set the shortcut file path (required after the refactor).

This patches fixes both of those issues. Note that all the directory service interaction is done on the main thread, because the actual function doing the work is on a background thread, and the directory service does not work there.

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

This was an oversight during the original implementation in bug 1761291. I updated the code that creates shortcuts there, but not the code that looks at existing ones.

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

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

In days of old, it was safe to use -brand-short-name for this, as it always lined up with the installer's concept of BrandShortName. This changed when https://bugzilla.mozilla.org/show_bug.cgi?id=1378834 landed -- which started using "Firefox Nightly" for the nightly branding BrandShortName in the installer (but maintained "Nightly" as the in-product -brand-short-name). Changing one or the other of these is not really a viable option:

- Changing the installer's BrandShortName for Nightly to match -brand-short-name would cause shortcuts to simply be called "Nightly" -- which is the opposite of what the aforementioned bug wanted
- Changing -brand-short-name would cause a number of in-product strings to start using "Firefox Nightly" instead of "Nightly" -- which is probably undesirable for a few reasons, not the least of which is possible l10n implications

For these reasons, and the relatively short timeline I have to fix this, I'm taking the simplest and easiest path of introducing a new string specifically for in-product shortcut names, which lines up with the installer's values for BrandShortName. (We perhaps should also separate this out in the installer -- but it's unnecessary here, so I did not go to the trouble.)

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

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

Bug 1788601 - [devtools] Fix ESLint consistent-return failures in devtools/client/framework. r=ochameau.

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

Bug 1788594 - [devtools] Fix ESLint consistent-return failures in devtools/client/debugger/. r=bomsy.

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

Backed out changeset 8de3f54dc3ca (bug 1781434) for causing mochitest failures on test_dialogfocus.xhtml . CLOSED TREE

Backed out changeset 09fd858a96c9 (bug 1783497) for causing Xpcshell failures on test_ext_cookies_partitionKey.js

Bug 1789883 - Add telemetry to know often users click in the secondary toolbar r=TravisLong,pdfjs-reviewers,marco

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

We had zero tests for this, and when I implemented the optimization to
avoid reframing in most cases I apparently swapped the arguments ;_;

Fix <input type=search> which suffered from the same issue, but we don't
ship by default.

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

The default constructor for RemoteImageHolder assigns default values to
all of its members. gfx::ColorDepth does not define a default value. This
change makes it so RemoteImageHolder provides a default value.

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

When D155902 merged and landed, it undid a recent change in
RemoteVideoDecoderParent::ProcessDecodedData without causing a compiler
error. This fixes up the merge to what was intended.

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

Make it a bit easier to read and less prone to race conditions. Remove a
setTimeout referencing bug 103197 which I'm pretty sure it's not an
issue.

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

Drive-by, but this should be more correct, since I'm not sure how this
code would handle mSVGView otherwise, which is in effect on the relevant
page.

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

Backed out changeset aeb32fa2f7d5 (bug 1761089) for causing failures on IPCBlobUtils.cpp. CLOSED TREE

Looks like this is not needed and making a web-compat issue.

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

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

This is manually run and requires a full build. MOZ_OBJDIR needs to be set to point to the object directory.

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