Open Bug 1580601 Opened 4 months ago Updated 2 months ago

Fix usage of nsIDocShellTreeItem in nsDocShell::InternalLoad


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




Fission Milestone M6


(Reporter: djvj, Unassigned)


(Blocks 1 open bug)


(Whiteboard: [rm-docshell-tree-item:hard])

Two uses, the first is more significant.

Here, InternalLoad gets the parent docshell (any type), and if it exists, gets the document from it and calls TryCancelFrameLoaderInitialization() on it.

This has to be checked for OOP parent, an if so forwarded to the appropriate process.

This usage is a much simpler fix. It occurs within #ifdef DEBUG code, retrieves the parent and asserts that it is null. Just needs to be changed to use BrowsingContext tree instead to check existence of a (same-type) parent.

Fission Milestone: --- → M5
Priority: -- → P2
Whiteboard: [rm-docshell-tree-item:hard]
Fission Milestone: M5 → Future

Kannan says replacing nsIDocShellTreeItem calls should block enabling Fission in Nightly (M6).

Fission Milestone: Future → M6
You need to log in before you can comment on or make changes to this bug.