Open Bug 1597427 Opened 2 months ago Updated 2 months ago

Fix uses of nsFrameLoader::CheckForRecursiveLoad in dom/base/nsFrameLoader.cpp

Categories

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

defect

Tracking

()

Fission Milestone M6

People

(Reporter: djvj, Unassigned)

References

(Blocks 1 open bug)

Details

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

In file dom/base/nsFrameLoader.cpp

Checks ItemType on DocShell. Change to use BrowsingContext.

Walks up tree to check for depth. Change to use BrowsingContext.

Checks URI up chain to try to find cases of recursive nested loads. This should be factored out into a separate method that only gets called on frame-loaders whose entire trees are in-process.

For frame-loaders where any ancestor docshell is out-of-process, this must have been checked prior to (or checked after) the call to this method.

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

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