Closed Bug 1502330 Opened 6 years ago Closed 6 years ago

Change nsGlobalWindowOuter::SetOpenerWindow to only handle the dynamic case

Categories

(Core :: DOM: Core & HTML, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
mozilla65
Tracking Status
firefox65 --- fixed

People

(Reporter: farre, Assigned: farre)

References

Details

Attachments

(1 file)

This depends on the possibility of creating browsing contexts with an opener directly
Assignee: nobody → afarre
Depends on: 1502328
Priority: -- → P3
Blocks: 1505838
Blocks: 1509016
By replacing nsWebBrowser's implementation of the nsIBaseWindow.initWindow and nsIBaseWindow.create with a new static nsWebBrowser::Create method we make it possible to pass arguments directly when creating an nsWebBrowser, for example the opener BrowsingContext. As a bonus we can do away with nsWebBrowser::mInitInfo!
Attachment #9026916 - Attachment description: Bug 1502330 - Create BrowsingContext with passed opener. r=qdot → Bug 1502330 - Create BrowsingContext with passed opener.
Status: NEW → ASSIGNED
Try looked good, and I could explain all oranges except three Android JIT which must be unrelated, so I'll go ahead and queue this for landing.
Pushed by afarre@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/cb9dec83210d Create BrowsingContext with passed opener. r=qdot
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
Depends on: 1510382
Depends on: 1510928
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: