Make test_nuke_webextension_wrappers.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

()

enhancement
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: Ehsan, Assigned: Ehsan)

Tracking

unspecified
mozilla55
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox55 fixed)

Details

Attachments

(1 attachment)

Assignee

Description

2 years ago
No description provided.
Assignee

Updated

2 years ago
Assignee: nobody → ehsan
Blocks: 1361461
Assignee

Updated

2 years ago
Depends on: 1368286
Comment on attachment 8872105 [details] [diff] [review]
Make test_nuke_webextension_wrappers.js correctly wait for the window to be destroyed instead of relying on the scheduling of the corresponding event

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

::: js/xpconnect/tests/unit/test_nuke_webextension_wrappers.js
@@ +12,5 @@
> +      Services.obs.removeObserver(observer, "inner-window-destroyed");
> +      setTimeout(callback, 0);
> +    }
> +  };
> +  Services.obs.addObserver(observer, "inner-window-destroyed");

Same comments as the other window destroyed patch.
Attachment #8872105 - Flags: review?(kmaglione+bmo) → review+

Comment 3

2 years ago
Pushed by eakhgari@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/4747ae7509f1
Make test_nuke_webextension_wrappers.js correctly wait for the window to be destroyed instead of relying on the scheduling of the corresponding event; r=kmag

Comment 4

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/4747ae7509f1
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.