Closed Bug 1515433 Opened 5 years ago Closed 3 years ago

Consolidate URI loading logic in nsWindowWatcher::OpenWindowInternal

Categories

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

Product:
Core
Component:
DOM: Navigation
Type:
enhancement

Tracking

()

Status:
RESOLVED WORKSFORME

People

(Reporter: qdot, Unassigned)

References

Details

OpenWindowInternal possibly takes a URI, and a LoadState object, and in some cases, combines them. However, we now allow the LoadState object to contain the URI, meaning we have 2 places we could possibly receive a URI from (though we currently should only pay attention to the URI passed as an argument to OpenWindowInternal). See if there is a way to clean this logic up some so everything operates on LoadState objects.
Priority: -- → P3
Assignee: kyle → nobody

nsDocShell has been changed a lot (and replaced by BrowsingContext at a number of places) to make it Fission compatible via other bugs. So, closing all of this bug list as WFM.

Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.