Intermittent browser_async_window_flushing.js | We should not have added the window to the closed windows array - Got 1, expected 0

RESOLVED FIXED in Firefox 46

Status

()

RESOLVED FIXED
3 years ago
2 years ago

People

(Reporter: philor, Assigned: mconley)

Tracking

({intermittent-failure})

Trunk
Firefox 47
intermittent-failure
Points:
---

Firefox Tracking Flags

(firefox45 wontfix, firefox46 fixed, firefox47 fixed, firefox-esr45 fixed)

Details

Attachments

(2 attachments)

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

Repository breakdown:
* mozilla-inbound: 4
* try: 1
* mozilla-central: 1
* fx-team: 1

Platform breakdown:
* linux64: 3
* linux32: 3
* osx-10-6: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1231422&startday=2015-12-07&endday=2015-12-13&tree=all
19 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 6
* fx-team: 5
* b2g-inbound: 3
* mozilla-central: 2
* mozilla-aurora: 2
* try: 1

Platform breakdown:
* linux64: 7
* linux32: 6
* windows8-64: 5
* osx-10-6: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1231422&startday=2015-12-14&endday=2015-12-20&tree=all
30 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 12
* fx-team: 6
* mozilla-aurora: 5
* try: 4
* b2g-inbound: 2
* mozilla-central: 1

Platform breakdown:
* linux64: 14
* linux32: 13
* osx-10-6: 2
* windows7-32: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1231422&startday=2015-12-21&endday=2015-12-27&tree=all
22 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* fx-team: 7
* mozilla-inbound: 6
* try: 3
* mozilla-central: 2
* mozilla-aurora: 2
* b2g-inbound: 2

Platform breakdown:
* linux64: 10
* osx-10-6: 7
* linux32: 3
* windowsxp: 1
* windows8-64: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1231422&startday=2015-12-28&endday=2016-01-03&tree=all
29 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 16
* fx-team: 6
* b2g-inbound: 3
* mozilla-aurora: 2
* try: 1
* mozilla-central: 1

Platform breakdown:
* linux64: 16
* linux32: 5
* osx-10-6: 3
* windows8-64: 2
* osx-10-10: 2
* windows7-32: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1231422&startday=2016-01-04&endday=2016-01-10&tree=all
30 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 13
* try: 6
* mozilla-aurora: 5
* mozilla-central: 2
* fx-team: 2
* b2g-inbound: 2

Platform breakdown:
* linux64: 16
* osx-10-6: 6
* windows8-64: 3
* windowsxp: 2
* linux32: 2
* windows7-32: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1231422&startday=2016-01-11&endday=2016-01-17&tree=all
Mike, possibly related to your recent sessionstore work?
status-firefox46: --- → affected
Flags: needinfo?(mconley)
(Assignee)

Comment 8

3 years ago
(In reply to Ryan VanderMeulen [:RyanVM] from comment #7)
> Mike, possibly related to your recent sessionstore work?

Definitely, yes. I'll see if I can give this some time this weekend.
(Assignee)

Updated

3 years ago
Assignee: nobody → mconley
(Assignee)

Comment 10

3 years ago
Oh cripes, I think I've figured this one out.

The thing that's supposed to listen for the TIMEOUT_DISABLED_PREF change in content-sessionStore.js doesn't work - it's comparing the pref string to topic, when it should compare it to data:

https://dxr.mozilla.org/mozilla-central/rev/c5da92c5b4906369dee83629f81d647226ac1038/browser/components/sessionstore/content/content-sessionStore.js#713

Luckily, that pref is just used for testing code.

Anyhow, this explains why this is intermittent - usually the test works quickly enough such that the message queue doesn't get flushed by the timer in the child, but periodically things line up just right so that the timer goes off in the right pocket of time, and blam.

Patch coming up.
(Assignee)

Comment 12

3 years ago
Created attachment 8711144 [details]
MozReview Request: Bug 1231422 - Fix busted pref observer for browser.sessionstore.debug.no_auto_updates. r?billm

Review commit: https://reviewboard.mozilla.org/r/32003/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/32003/
Attachment #8711144 - Flags: review?(wmccloskey)
16 automation job failures were associated with this bug yesterday.

Repository breakdown:
* mozilla-inbound: 7
* mozilla-central: 2
* mozilla-aurora: 2
* fx-team: 2
* b2g-inbound: 2
* try: 1

Platform breakdown:
* linux64: 11
* linux32: 3
* windowsxp: 1
* osx-10-6: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1231422&startday=2016-01-22&endday=2016-01-22&tree=all
53 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 21
* fx-team: 10
* mozilla-aurora: 7
* try: 6
* b2g-inbound: 5
* mozilla-central: 4

Platform breakdown:
* linux64: 33
* linux32: 7
* osx-10-6: 6
* windowsxp: 2
* windows8-64: 2
* windows7-32: 2
* osx-10-10: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1231422&startday=2016-01-18&endday=2016-01-24&tree=all
18 automation job failures were associated with this bug yesterday.

Repository breakdown:
* mozilla-inbound: 7
* mozilla-aurora: 4
* mozilla-central: 3
* fx-team: 2
* b2g-inbound: 2

Platform breakdown:
* linux64: 15
* osx-10-10: 2
* windows8-64: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1231422&startday=2016-01-25&endday=2016-01-25&tree=all
Comment on attachment 8711144 [details]
MozReview Request: Bug 1231422 - Fix busted pref observer for browser.sessionstore.debug.no_auto_updates. r?billm

https://reviewboard.mozilla.org/r/32003/#review29101

Oops.
Attachment #8711144 - Flags: review?(wmccloskey) → review+
(Assignee)

Updated

3 years ago
Flags: needinfo?(mconley)

Comment 18

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/98ccd385e6fd
Status: NEW → RESOLVED
Last Resolved: 3 years ago
status-firefox47: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 47
(Reporter)

Comment 19

3 years ago
Apparently that's not all of the problem.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
85 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 46
* try: 12
* fx-team: 10
* mozilla-central: 9
* mozilla-aurora: 5
* b2g-inbound: 2
* mozilla-beta: 1

Platform breakdown:
* linux64: 73
* osx-10-10: 5
* linux32: 4
* windows8-64: 2
* osx-10-6: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1231422&startday=2016-01-25&endday=2016-01-31&tree=all
(Assignee)

Updated

3 years ago
Depends on: 1245212
(Assignee)

Comment 22

3 years ago
Pretty sure I've figured this one out for real this time - see bug 1245212, and this try build: https://treeherder.mozilla.org/#/jobs?repo=try&revision=03c1b1945fdb
20 automation job failures were associated with this bug yesterday.

Repository breakdown:
* fx-team: 8
* mozilla-aurora: 5
* mozilla-inbound: 4
* try: 1
* mozilla-central: 1
* mozilla-beta: 1

Platform breakdown:
* linux64: 18
* windowsxp: 1
* windows8-64: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1231422&startday=2016-02-04&endday=2016-02-04&tree=all
21 automation job failures were associated with this bug yesterday.

Repository breakdown:
* fx-team: 10
* mozilla-inbound: 9
* try: 1
* mozilla-aurora: 1

Platform breakdown:
* linux64: 20
* linux32: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1231422&startday=2016-02-05&endday=2016-02-05&tree=all
108 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 47
* fx-team: 34
* try: 11
* mozilla-aurora: 10
* mozilla-central: 3
* mozilla-beta: 3

Platform breakdown:
* linux64: 95
* linux32: 5
* windows8-64: 4
* osx-10-6: 2
* windowsxp: 1
* osx-10-10: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1231422&startday=2016-02-01&endday=2016-02-07&tree=all
19 automation job failures were associated with this bug yesterday.

Repository breakdown:
* mozilla-inbound: 9
* fx-team: 4
* try: 2
* mozilla-central: 2
* mozilla-beta: 1
* mozilla-aurora: 1

Platform breakdown:
* linux64: 14
* windows8-64: 2
* osx-10-6: 2
* linux32: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1231422&startday=2016-02-08&endday=2016-02-08&tree=all
(Reporter)

Comment 27

3 years ago
And just in time, this has put on a burst of speed lately and is very close to being the top failure.
(Assignee)

Comment 28

3 years ago
Bug 1245212 just landed on fx-team. Let's see if this goes away now.
15 automation job failures were associated with this bug yesterday.

Repository breakdown:
* mozilla-inbound: 9
* mozilla-aurora: 4
* try: 2

Platform breakdown:
* linux64: 11
* windowsxp: 1
* osx-10-6: 1
* osx-10-10: 1
* linux32: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1231422&startday=2016-02-10&endday=2016-02-10&tree=all
54 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 23
* mozilla-aurora: 15
* fx-team: 6
* try: 4
* mozilla-central: 3
* mozilla-beta: 3

Platform breakdown:
* linux64: 42
* osx-10-6: 4
* linux32: 3
* windows8-64: 2
* windowsxp: 1
* windows7-32: 1
* osx-10-10: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1231422&startday=2016-02-08&endday=2016-02-14&tree=all
7 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-aurora: 5
* mozilla-beta: 2

Platform breakdown:
* linux64: 4
* linux32: 2
* osx-10-6: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1231422&startday=2016-02-22&endday=2016-02-28&tree=all
25 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-aurora: 16
* mozilla-beta: 5
* mozilla-release: 2
* mozilla-esr45: 2

Platform breakdown:
* linux32: 15
* linux64: 8
* osx-10-6: 2

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1231422&startday=2016-02-29&endday=2016-03-06&tree=all
Looks like not :(
Flags: needinfo?(mconley)
(Assignee)

Comment 34

3 years ago
Here's the good news:

Bug 1245212 seems to have fixed the orange on mozilla-central and mozilla-inbound (and since the recent uplift, I would wager it's had an impact on mozilla-aurora for the past few days).

If you look at the time that bug 1245212 landed, and when it was merged into mozilla-inbound after having landed in fx-team, you can see an immediate drop-off in occurrences.

The problem is that bug 1245212 was never uplifted to the aurora at the time (which is currently at beta).

So we will continue to see this orange on beta, release and esr45 until bug 1245212 gets merged in.

RyanVM: According to the tree rules, I can uplift fixes without requesting approval if they're test only using a=test-only... however, the code I'm touching here is _not_ within a test. It's just code that the test uses. Should I request approval for it then? And I suppose I should also do this for esr45?
Flags: needinfo?(mconley) → needinfo?(ryanvm)
If the code is only used by tests, I think a=test-only is fine. If it's used in shipping code as well, you should probably request formal approval. Thanks!
Flags: needinfo?(ryanvm)
(Assignee)

Comment 36

3 years ago
The patch in Bug 1245212 just landed for ESR 45 and beta, so this should be gone once and for all.
Status: REOPENED → RESOLVED
Last Resolved: 3 years ago3 years ago
Resolution: --- → FIXED
status-firefox45: affected → wontfix
status-firefox46: affected → fixed
status-firefox-esr45: --- → fixed
7 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-beta: 5
* try: 1
* mozilla-release: 1

Platform breakdown:
* osx-10-6: 3
* linux64: 2
* linux32: 2

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1231422&startday=2016-03-14&endday=2016-03-20&tree=all
You need to log in before you can comment on or make changes to this bug.