Include inner/outer window IDs in PWindowGlobal

RESOLVED FIXED in Firefox 65

Status

()

enhancement
P2
normal
RESOLVED FIXED
7 months ago
2 months ago

People

(Reporter: Nika, Assigned: Nika)

Tracking

unspecified
mozilla65
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox65 fixed)

Details

Attachments

(1 attachment)

This should make it much easier to get a handle on one, as well as giving a per-global handle for Chrome JS.

It's also handy due to LoadInfo providing a windowID in the parent process.
Assignee

Comment 1

7 months ago
This will be useful as both an ID for PWindowGlobal, as well as a mechanism for
taking advantage of already synchronized information. As an example, LoadInfo
objects contain the inner window IDs of the window requesting the load, which
can now be used to obtain a reference to the corresponding WindowGlobalParent
in the parent process.

Depends on D9395
Priority: -- → P2

Comment 2

6 months ago
Pushed by nika@thelayzells.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a300e9ae7a01
Include innerWindowId/outerWindowId in PWindowGlobal, r=farre

Comment 4

6 months ago
Pushed by nika@thelayzells.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/3b5fffae2d2b
Include innerWindowId/outerWindowId in PWindowGlobal, r=farre
Assignee

Updated

6 months ago
Flags: needinfo?(nika)

Comment 6

6 months ago
Pushed by nika@thelayzells.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/622d4bf1c0ef
Include innerWindowId/outerWindowId in PWindowGlobal, r=farre
Assignee

Updated

6 months ago
Flags: needinfo?(nika)

Comment 7

6 months ago
Pushed by nika@thelayzells.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/f824e2415aa6
Include innerWindowId/outerWindowId in PWindowGlobal, r=farre

Comment 8

6 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/f824e2415aa6
Status: NEW → RESOLVED
Last Resolved: 6 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
Component: DOM → DOM: Core & HTML
Product: Core → Core
You need to log in before you can comment on or make changes to this bug.