1212035 - Intermittent browser_notification_open_settings.js | leaked 1 window(s) until shutdown [url = about:preferences#content]

Status: RESOLVED FIXED

(Firefox :: Settings UI, defect)

Description

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://treeherder.mozilla.org/logviewer.html#?job_id=15250531&repo=mozilla-inbound

Comment 1

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

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?

Comment 2

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

Sorry, that link should've been https://treeherder.mozilla.org/#/jobs?repo=mozilla-central&revision=3edc8d4a1e19

Comment 3

[Lina Butler [:lina]]

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.

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

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

Comment 5

[Intermittent Failures Robot]

64 automation job failures were associated with this bug yesterday.

Repository breakdown:
* fx-team: 47
* mozilla-central: 12
* mozilla-inbound: 5

Platform breakdown:
* osx-10-10: 64

For more details, see:
http://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1212035&startday=2015-10-06&endday=2015-10-06&tree=all

Comment 6

[Matthew N. [:MattN]]

(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.

Comment 7

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/36082e2aea9c

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/36082e2aea9c

Comment 9

[Intermittent Failures Robot]

72 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* fx-team: 48
* mozilla-central: 14
* mozilla-inbound: 10

Platform breakdown:
* osx-10-10: 72

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1212035&startday=2015-10-05&endday=2015-10-11&tree=all