Closed
Bug 1212035
Opened 9 years ago
Closed 9 years ago
Intermittent browser_notification_open_settings.js | leaked 1 window(s) until shutdown [url = about:preferences#content]
Categories
(Firefox :: Settings UI, defect)
Firefox
Settings UI
Tracking
()
RESOLVED
FIXED
Firefox 44
Tracking | Status | |
---|---|---|
firefox44 | --- | fixed |
People
(Reporter: KWierso, Assigned: lina)
References
Details
(Keywords: intermittent-failure)
Reporter | ||
Comment 1•9 years ago
|
||
The first instance I've seen was in this merge from fx-team to m-c: https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&fromchange=f0865d36454c&group_state=expanded&filter-searchStr=OS%20X%2010.10%20debug%20Mochitest%20Mochitest%20Browser%20Chrome%20M%28bc3%29&tochange=2722b65059df
As a guess, bug 1192458 or bug 1208295?
Reporter | ||
Comment 2•9 years ago
|
||
Sorry, that link should've been https://treeherder.mozilla.org/#/jobs?repo=mozilla-central&revision=3edc8d4a1e19
Reporter | ||
Updated•9 years ago
|
Flags: needinfo?(kcambridge)
Flags: needinfo?(MattN+bmo)
Assignee | ||
Comment 3•9 years ago
|
||
Does that mean we aren't waiting for the window to close? In https://dxr.mozilla.org/mozilla-central/source/browser/base/content/test/alerts/browser_notification_open_settings.js, should the last line be `yield BrowserTestUtils.removeTab(tab)`? Looks like it returns a promise.
I can look more tonight, unless :MattN beats me to it.
Reporter | ||
Comment 4•9 years ago
|
||
Okay, the first instance on fx-team was on the push for bug 1208295:
https://treeherder.mozilla.org/#/jobs?repo=fx-team&fromchange=a5abfa8e6310&group_state=expanded&filter-searchStr=10.10%20debug%20bc3&tochange=e799bbf371a6
Flags: needinfo?(kcambridge)
Comment hidden (Intermittent Failures Robot) |
Comment 6•9 years ago
|
||
(In reply to Kit Cambridge [:kitcambridge] (PTO from 2015-10-06 to 2015-10-16) from comment #3)
> Does that mean we aren't waiting for the window to close?
Yep, you're right!
> In
> https://dxr.mozilla.org/mozilla-central/source/browser/base/content/test/alerts/browser_notification_open_settings.js, should the last line be `yield
> BrowserTestUtils.removeTab(tab)`? Looks like it returns a promise.
Yep, I'll push a patch with you as the author.
Assignee: nobody → kcambridge
Status: NEW → ASSIGNED
No longer depends on: 1192458
Flags: needinfo?(MattN+bmo)
Updated•9 years ago
|
Comment 8•9 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox44:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 44
Comment hidden (Intermittent Failures Robot) |
You need to log in
before you can comment on or make changes to this bug.
Description
•