1500949 - Include inner/outer window IDs in PWindowGlobal

Status: RESOLVED FIXED

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

Description

[Nika Layzell [:nika] (ni? for response)]

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.

Comment 1

[Nika Layzell [:nika] (ni? for response)]

Attachment: Bug 1500949 - Include innerWindowId/outerWindowId in PWindowGlobal,

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

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a300e9ae7a01
Include innerWindowId/outerWindowId in PWindowGlobal, r=farre

Comment 3

[Arthur Iakab [arthur_iakab]]

Backed out 9 changesets (bug 1500948, bug 1500949, bug 1448426, bug 1487249, bug 1500950, bug 1500944) for causing talos crashes on ts_paint | application crashed [@ MOZ_CrashOOL(char const*, int, char const*)]

Backout revision https://hg.mozilla.org/integration/mozilla-inbound/rev/c12b84f575c3fe48d72f3c8c24a54b3dee1cc985

Failed push https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&resultStatus=testfailed%2Cbusted%2Cexception&classifiedState=unclassified&revision=429776feea6ba796345ef7e41b9c9d367335ef62

Failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=213326943&repo=mozilla-inbound
             https://treeherder.mozilla.org/logviewer.html#?job_id=213331977&repo=mozilla-inbound 

:Nika Layzell Cold you please take a look?

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/3b5fffae2d2b
Include innerWindowId/outerWindowId in PWindowGlobal, r=farre

Comment 5

[Arthur Iakab [arthur_iakab]]

Backed out 13 changesets (bug 1500948, bug 1509362, bug 1509591, bug 1448426, bug 1500949, bug 1487249, bug 1509930, bug 1500950, bug 1500944) for causing crashes and assertion failures on PBackgroundParent.cpp:696 CLOSED TREE

Backout revision https://hg.mozilla.org/integration/mozilla-inbound/rev/9bfe29337ffe3d93cd060077e2e999e72bb9b7cf

Failed push https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&resultStatus=testfailed%2Cbusted%2Cexception&classifiedState=unclassified&selectedJob=214255312&revision=c3fe435e473a463fbc22d4afa531bdedb757079c

Failure logs: https://treeherder.mozilla.org/logviewer.html#?job_id=214255312&repo=mozilla-inbound
              https://treeherder.mozilla.org/logviewer.html#?job_id=214255794&repo=mozilla-inbound
              https://treeherder.mozilla.org/logviewer.html#?job_id=214249038&repo=mozilla-inbound
              https://treeherder.mozilla.org/logviewer.html#?job_id=214249630&repo=mozilla-inbound

:Nika Layzell Could you please take a look?

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/622d4bf1c0ef
Include innerWindowId/outerWindowId in PWindowGlobal, r=farre

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/f824e2415aa6
Include innerWindowId/outerWindowId in PWindowGlobal, r=farre

Comment 8

[Stefan Hindli [:stefan_hindli]]

https://hg.mozilla.org/mozilla-central/rev/f824e2415aa6