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

Backed out 6 changesets (bug 1631593) for perma failures on browser_aboutCertError_exception. CLOSED TREE

Backed out changeset f176af77900a (bug 1631593)
Backed out changeset 74b6fa2a1409 (bug 1631593)
Backed out changeset 7fbc8e2c4a4a (bug 1631593)
Backed out changeset 77ff0aee23f6 (bug 1631593)
Backed out changeset 90bde53721ac (bug 1631593)
Backed out changeset ea777b1a9380 (bug 1631593)

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

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

This corrects the handling of user-scalable=no to first clamp scale (whether
specified or default) between min and max scales, then applies the resulting
value to initial-scale, min and max.

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

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

- Use MakeScopeExit to restore states.
- Use IgnoredErrorResult to suppress expressions.
- Do early return in the case where CloneNode fails.

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

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

Bug 1639570 - Implement nsICookieService::getCookieStringFromDocument - part 1 - implementation, r=ckerschb

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

Bug 1631405 - Move nsISecureBrowserUI to be owned by the canonical browsing context instead of docshell. r=nika,ckerschb,Gijs,webcompat-reviewers,twisniewski

This removes all docshell nsISecureBrowserUI and mixed content properties, and moves them into CanonicalBrowsingContext/WindowGlobalParent. It makes the mixed content blocker just compute the state for the current load, and then send the results to the parent process, where we update the security state accordingly.

I think we could in the future remove onSecurityChange entirely, and instead just fire an event to the <browser> element notifying it of changes to the queryable securityUI.

Unfortunately we have a lot of existing code that depends on specific ordering between onSecurityChange and onLocationChange, so I had to hook into the RemoteWebProgress implementation in BrowserParent to mimic the same timings.

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

Backed out changeset 9963cc0b23cb (bug 1631405)
Backed out changeset 469ac933ed7c (bug 1631405)
Backed out changeset 0c5f55864268 (bug 1631405)
Backed out changeset 20dcbcc2f3b8 (bug 1631405)

Bug 1631405 - Move nsISecureBrowserUI to be owned by the canonical browsing context instead of docshell. r=nika,ckerschb,Gijs,webcompat-reviewers,twisniewski

This removes all docshell nsISecureBrowserUI and mixed content properties, and moves them into CanonicalBrowsingContext/WindowGlobalParent. It makes the mixed content blocker just compute the state for the current load, and then send the results to the parent process, where we update the security state accordingly.

I think we could in the future remove onSecurityChange entirely, and instead just fire an event to the <browser> element notifying it of changes to the queryable securityUI.

Unfortunately we have a lot of existing code that depends on specific ordering between onSecurityChange and onLocationChange, so I had to hook into the RemoteWebProgress implementation in BrowserParent to mimic the same timings.

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

Bug 1625156 - Error page for HTTPS Only Mode. r=fluent-reviewers,ckerschb,nhnt11,flod,nika,johannh,mattwoodrow

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

Backed out changeset 8190c67514cc (bug 1620679)
Backed out changeset 4ca2912d7421 (bug 1620679)

This matches what the spec says, and what blink does.

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

Spec: https://html.spec.whatwg.org/multipage/interactive-elements.html#dialog-focusing-steps

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

IsInNativeAnonymousSubtree and IsRootOfNativeAnonymousSubtree are
exposed in nsINode, so we don't need to make sure to call them on
nsIContent objects.

Depends on D76681

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

In favor of the NativeAnonymous versions which they forward to.

Done automatically with:

  rg -l 'IsInAnonymousSubtree' | xargs sed -i 's/IsInAnonymousSubtree/IsInNativeAnonymousSubtree/g'

And removing the function definitions afterwards.

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

The only caller that passes that principal is the only caller that calls
into LoadSheet with a loader that has a document anyway.

And the principal we're passing is Document::NodePrincipal(), which is
what we'd end up using anyway, so the new code is exactly equivalent, as
far as I can tell.

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

This corrects the handling of user-scalable=no to first clamp scale (whether
specified or default) between min and max scales, then applies the resulting
value to initial-scale, min and max.

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

Bug 1637194 - P4. Inherit the first-party domain from the parent document when creating CookieJarSetting in document r=timhuang,baku

This fixes a issue that about:blank page can't get correct first-party
domain when fission is enabled.

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

Backed out changeset bb7c330d407f (bug 1637307) for wpt failures at layout/generic/ViewportFrame.cpp on s CLOSED TREE

Backed out 6 changesets (bug 1631593) for Assertion failure in xpcom/base/nsCycleCollector.cpp. CLOSED TREE

Backed out changeset 35b4cd4bb12a (bug 1631593)
Backed out changeset a20b9fb485fc (bug 1631593)
Backed out changeset 8ade55be1042 (bug 1631593)
Backed out changeset ee3255e55ab3 (bug 1631593)
Backed out changeset 7d5a3d1035dc (bug 1631593)
Backed out changeset 90972e768c33 (bug 1631593)

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

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

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

This patch completes the top layer requirement for showModal()
Spec: https://html.spec.whatwg.org/multipage/interactive-elements.html#dom-dialog-showmodal

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

Bug 1624269 - P2. Cache access granted result in the 3rd-party window instead of top-level window in fission mode. r=timhuang,baku

Before this patch, in non-fission mode, we cache storage access granted result
in the top-level window so we don't have to iterate all the browsing contexts
in the same tree while syncing the storage permission granted decision.

However, since we plan to rely on the current update mechanism to sync
mHasStorageAccess flag for different documents in the same tab (instead of using
the syncing mechanism of permission manager), we will eventually need to iterate
the browsing context tree to find all the documents to sync. Base on this,
we no longer have to maintain different method for fission and non-fission.

In this patch, we store the permission granted result in the inner
window instead of using permission key and store the key in the top-level
window.

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

Backed out 5 changesets (bug 1624269) for browser-chrome failures on browser_storageAccessWithHeuristics.js. CLOSED TREE

Backed out changeset 59cdba115447 (bug 1624269)
Backed out changeset 23b5c53f4be8 (bug 1624269)
Backed out changeset be697a5bc0fd (bug 1624269)
Backed out changeset 81420bca683c (bug 1624269)
Backed out changeset 599db5acefe1 (bug 1624269)

Bug 1624269 - P2. Cache access granted result in the 3rd-party window instead of top-level window in fission mode. r=timhuang,baku

Before this patch, in non-fission mode, we cache storage access granted result
in the top-level window so we don't have to iterate all the browsing contexts
in the same tree while syncing the storage permission granted decision.

However, since we plan to rely on the current update mechanism to sync
mHasStorageAccess flag for different documents in the same tab (instead of using
the syncing mechanism of permission manager), we will eventually need to iterate
the browsing context tree to find all the documents to sync. Base on this,
we no longer have to maintain different method for fission and non-fission.

In this patch, we store the permission granted result in the inner
window instead of using permission key and store the key in the top-level
window.

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

Local compiler warns about the return being unreached. Remove one of the
two copy-pasta links.

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

This also requires changing the EffectCompositor to allow animations in print
and print preview, and setting up a document timeline for the cloned document

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

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

Backed out changeset 5633cc2aaa9b (bug 1637487)
Backed out changeset c4f0c757cae0 (bug 1637487)
Backed out changeset 65bb1114df87 (bug 1637487)
Backed out changeset 1a64ef10e773 (bug 1637487)
Backed out changeset d1709ebb7f46 (bug 1637487)
Backed out changeset 286e5ee6517f (bug 1637487)
Backed out changeset 83362625c297 (bug 1637487)
Backed out changeset ab9df2f2d11f (bug 1637487)
Backed out changeset b83f49bc11b0 (bug 1637487)

Bug 1637487 - Store mixed content security flags as a flags word on Document instead of bools. r=ckerschb

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

Bug 1637487 - Store mixed content security flags as a flags word on Document instead of bools. r=ckerschb

Depends on D75025

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

Backed out changeset 4309266ff43d (bug 1629291) for frequent assertion failures on ErrorResult.h. CLOSED TREE

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

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

This patch persists the CrossOriginEmbedderPolicy on the Document and
WindowContext.
The WindowContext's embedder policy is set in
WindowGlobalActor::BaseInitializer by either adopting the policy of the HTTP
channel that loaded the document, or in WindowGlobalChild::OnNewDocument
by inheriting the one from the browsingContext's windowContext.

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