1594280 - Fix usage of nsIDocShellTreeItem in nsDocShell::ValidateOrigin

Status: RESOLVED FIXED

(Core :: DOM: Navigation, defect, P2)

Description

[Kannan Vijayan [:djvj]]

https://searchfox.org/mozilla-central/rev/8b7aa8af652f87d39349067a5bc9c0256bf6dedc/docshell/base/nsDocShell.cpp#1082

The users of this bug have already changed to ensure that this only gets called when both arguments are in-process.

However, all of the tree-traversal code within this function needs to change to use BrowsingContext instead of nsIDocShellTreeItem to traverse.

Comment 1

[Chris Peterson [:cpeterson]]

Tentatively moving all bugs whose summaries mention "Fission" (or other Fission-related keywords) but are not assigned to a Fission Milestone to the "?" triage milestone.

This will generate a lot of bugmail, so you can filter your bugmail for the following UUID and delete them en masse:

0ee3c76a-bc79-4eb2-8d12-05dc0b68e732

Comment 2

[Tetsuharu OHZEKI [:tetsuharu] (UTC+9)]

Attachment: Bug 1594280 - Fix uses of nsIDocShellTreeItem in nsIDocShellTreeItem in nsDocShell::ValidateOrigin.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ea7d48958f2f
Fix uses of nsIDocShellTreeItem in nsIDocShellTreeItem in nsDocShell::ValidateOrigin. r=kmag

Comment 4

[Narcis Beleuzu [:NarcisB]]

Backed out for mochitest failures on nsDocShell.cpp

Backout link: https://hg.mozilla.org/integration/autoland/rev/a1dcad5b9e7658d99e720d982a1dfa1e992195ce
Log link: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=284969624&repo=autoland&lineNumber=10190

Comment 5

[Tetsuharu OHZEKI [:tetsuharu] (UTC+9)]

(In reply to Narcis Beleuzu [:NarcisB] from comment #4)

Backed out for mochitest failures on nsDocShell.cpp

Backout link: https://hg.mozilla.org/integration/autoland/rev/a1dcad5b9e7658d99e720d982a1dfa1e992195ce
Log link: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=284969624&repo=autoland&lineNumber=10190

The previous code checks aTarget->mDocShell guard in caller sides of this function.
After the above backouted patch, we change arguments of this to the style passing BrowsingContext but I forgot aTarget->mDocShell might be null. I'll fix it.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/bb770dc63b34
Fix uses of nsIDocShellTreeItem in nsIDocShellTreeItem in nsDocShell::ValidateOrigin. r=kmag

Comment 7

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/bb770dc63b34