1583444 - Check remaining occurrences of window.top that might fail with content frame

Status: RESOLVED FIXED

(DevTools :: General, task, P1)

Description

[Julian Descottes [:jdescottes]]

After Bug 1539979, the DevTools toolbox runs in a content frame.
One of the consequences is that we can no longer reach the topmost chrome frame using window.top.

We fixed all the spots that were detected by test failures, but there are probably other areas that are not tested but rely on window.top as well.
We recently had the following regression in the performance panel: https://bugzilla.mozilla.org/show_bug.cgi?id=1578373

The goal of this bug is to be more proactive and try to identify other potential regressions and fix them as soon as possible.

Comment 1

[Julian Descottes [:jdescottes]]

Attachment: Bug 1583444 - Stop using window.top in WebConsole codebase

Comment 2

[Julian Descottes [:jdescottes]]

Attachment: Bug 1583444 - Stop using win.top in devtools/shared/layout/utils.js

Depends on D46957

Comment 3

[Julian Descottes [:jdescottes]]

Attachment: Bug 1583444 - Reuse DevToolsUtils.getTopWindow in HTMLTooltip

Depends on D46958

Comment 4

[Julian Descottes [:jdescottes]]

Attachment: Bug 1583444 - Stop using win.top in DevTools tests

Depends on D46959
For those tests, using win.top was the same as using win, and did not have any impact on the asserts performed in the test.
We could have left them around, but for completeness I prefer to remove them as well.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b4dca3917f04
Stop using window.top in WebConsole codebase r=nchevobbe
https://hg.mozilla.org/integration/autoland/rev/6f09a19d3fa4
Stop using win.top in devtools/shared/layout/utils.js r=pbro
https://hg.mozilla.org/integration/autoland/rev/a7bbb75ce721
Reuse DevToolsUtils.getTopWindow in HTMLTooltip r=yzen
https://hg.mozilla.org/integration/autoland/rev/c62a28520dd6
Stop using win.top in DevTools tests r=nchevobbe

Comment 6

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/b4dca3917f04
https://hg.mozilla.org/mozilla-central/rev/6f09a19d3fa4
https://hg.mozilla.org/mozilla-central/rev/a7bbb75ce721
https://hg.mozilla.org/mozilla-central/rev/c62a28520dd6