1493628 - BrowserWindowTracker assumes Browser:Init messages only ever come from foreground windows

Status: RESOLVED FIXED

(Firefox :: General, enhancement, P1)

Description

[:Gijs (he/him)]

(Spotted when looking at about:blank creation stuff forced by BrowserWindowTracker getting the outerWindowID on a parent-process browser when creating a new window, which will call contentWindow.windowUtils.outerWindowID, which forces creation of the about:blank window)


Browser:Init messages are sent when browser-child.js loads, for any frame.

The code in BrowserWindowTracker checks that the browser is the selected browser in that window, but if so it'll unconditionally update the 'current toplevel window' outerWindowID. This is stored by netwerk/ code in a single parent process static var, so messing with it in this way seems... wrong.

We should also check that the window is the toplevel window.

Comment 1

[:Gijs (he/him)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=66bd31a97bf6b21774b839cc5bdeac4a1db75f2d

Comment 2

[:Gijs (he/him)]

Attachment: Bug 1493628 - only update the 'current' outerWindowID when the browser window is the foreground window, r?dao

Comment 3

[:Gijs (he/him)]

It seems it also makes this assumption for all the other callers that update the 'current' outerWindowID, e.g. tab selects in background windows or opening minimized windows will also throw off this code...

Comment 4

[Dão Gottwald [::dao]]

Comment on attachment 9012136 [details]
Bug 1493628 - only update the 'current' outerWindowID when the browser window is the foreground window, r?dao

Dão Gottwald [::dao] has approved the revision.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/513ebcc0f395
only update the 'current' outerWindowID when the browser window is the foreground window, r=dao

Comment 6

[Stefan Hindli [:stefan_hindli]]

https://hg.mozilla.org/mozilla-central/rev/513ebcc0f395