Make browser_console_dead_objects.js correctly wait for the window to be destroyed instead of relying on the scheduling of the corresponding event

RESOLVED FIXED in Firefox 55

Status

RESOLVED FIXED
a year ago
4 months ago

People

(Reporter: Ehsan, Assigned: Ehsan)

Tracking

unspecified
Firefox 55

Firefox Tracking Flags

(firefox55 fixed)

Details

Attachments

(1 attachment)

Comment hidden (empty)
(Assignee)

Comment 1

a year ago
Created attachment 8874247 [details] [diff] [review]
Make browser_console_dead_objects.js correctly wait for the window to be destroyed instead of relying on the scheduling of the corresponding event
Attachment #8874247 - Flags: review?(kmaglione+bmo)
(Assignee)

Updated

a year ago
Assignee: nobody → ehsan
Blocks: 1361461
Comment on attachment 8874247 [details] [diff] [review]
Make browser_console_dead_objects.js correctly wait for the window to be destroyed instead of relying on the scheduling of the corresponding event

Review of attachment 8874247 [details] [diff] [review]:
-----------------------------------------------------------------

::: devtools/client/webconsole/test/browser_console_dead_objects.js
@@ +31,5 @@
>      Services.prefs.setBoolPref("devtools.chrome.enabled", true);
>      yield loadTab(TEST_URI);
> +    let dwu = content.window.QueryInterface(Components.interfaces.nsIInterfaceRequestor)
> +                            .getInterface(Components.interfaces.nsIDOMWindowUtils);
> +    let winID = dwu.outerWindowID;

Please use `tab.linkedBrowser.outerWindowID` for this, in case we want to support e10s mode in these tests at some point. Or the no-cpows-in-tests rule gets turned on for this directory.
Attachment #8874247 - Flags: review?(kmaglione+bmo) → review+

Comment 3

a year ago
Pushed by eakhgari@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/5dd5d16e7085
Make browser_console_dead_objects.js correctly wait for the window to be destroyed instead of relying on the scheduling of the corresponding event; r=kmag
https://hg.mozilla.org/mozilla-central/rev/5dd5d16e7085
Status: NEW → RESOLVED
Last Resolved: a year ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 55

Updated

4 months ago
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.