Open Bug 1597491 Opened 2 months ago Updated 1 month ago

Fix uses of nsXULPopupManager::MayShowPopup in layout/xul/nsXULPopupManager.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:simple])

In file layout/xul/nsXULPopupManager.cpp

Tests whether it is OK to show a popup or not, given an nsMenuPopupFrame.

Gets BaseWindow by QueryInterfacing PresContext->DocShell.

Gets root-window for in-process root.

It seems like this would be running in chrome context all the time, so out-of-process documents and windows are not an issue.

Change code to using BrowsingContext to navigate tree.

Assert that all relevant BrowsingContext items (and windows) are in-process.

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.