1307015 - Intermittent browser/base/content/test/general/browser_bug623893.js | uncaught exception - Error: operation not possible on dead CPOW at getSessionHistory@resource://gre/modules/RemoteAddonsParent.jsm:890:3

Status: RESOLVED FIXED

(Firefox :: General, defect)

Description

[Treeherder Bug Filer]

Filed by: philringnalda [at] gmail.com

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

https://archive.mozilla.org/pub/firefox/tinderbox-builds/mozilla-inbound-macosx64-debug/1475430775/mozilla-inbound_yosemite_r7-debug_test-mochitest-e10s-browser-chrome-7-bm135-tests1-macosx-build315.txt.gz

Comment 1

[Intermittent Failures Robot]

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

Repository breakdown:
* mozilla-inbound: 2
* autoland: 2
* fx-team: 1

Platform breakdown:
* linux64: 5

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1307015&startday=2016-10-10&endday=2016-10-16&tree=all

Comment 2

[Intermittent Failures Robot]

5 failures in 715 pushes (0.007 failures/push) were associated with this bug in the last 7 days.  

Repository breakdown:
* mozilla-central: 3
* mozilla-inbound: 1
* autoland: 1

Platform breakdown:
* linux32: 3
* windows7-32-vm: 1
* linux64: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1307015&startday=2016-11-14&endday=2016-11-20&tree=all

Comment 3

[Ryan VanderMeulen (PTO, back July 27)]

Just saw this go by on Beta. Hadn't seen a dead CPOW failure in awhile!

Comment 4

[Blake Kaplan (:mrbkap) (inactive)]

Attachment: Bug 1307015 - Use addEventListener(..., { once: true }).

I didn't realize this existed! This lets us simplify some things.

Review commit: https://reviewboard.mozilla.org/r/113602/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/113602/

Comment 5

[Blake Kaplan (:mrbkap) (inactive)]

Attachment: Bug 1307015 - Remove CPOWs from browser_bug623893.js

It took me a little bit to understand what the test was doing. It was actually
using the default tab to navigate to three pages. Then, it duplicated it
immediatedly closed that tab. Finally, it duplicated the original tab, then
duplicated the duplicate before closing both of those tabs.

Review commit: https://reviewboard.mozilla.org/r/113604/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/113604/

Comment 6

[:Felipe Gomes (needinfo for replies!)]

Comment on attachment 8838790 [details]
Bug 1307015 - Use addEventListener(..., { once: true }).

https://reviewboard.mozilla.org/r/113602/#review115430

Doesn't this make the checkFn unusable? If there's a custom function to check for the wait then the event will be removed when it shouldn't.

If there are no or few users of checkFn we could remove it from here and make a more specific version where it's supported.

Comment 7

[:Felipe Gomes (needinfo for replies!)]

Comment on attachment 8838791 [details]
Bug 1307015 - Remove CPOWs from browser_bug623893.js

https://reviewboard.mozilla.org/r/113604/#review115434

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/1b501700e807
Remove CPOWs from browser_bug623893.js r=felipe

Comment 9

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/1b501700e807

Comment 10

[Ryan VanderMeulen (PTO, back July 27)]

https://hg.mozilla.org/releases/mozilla-aurora/rev/bfb5c94e4015

Comment 11

[Ryan VanderMeulen (PTO, back July 27)]

https://hg.mozilla.org/releases/mozilla-beta/rev/60618c6f74d2

Comment 12

[Ryan VanderMeulen (PTO, back July 27)]

https://hg.mozilla.org/releases/mozilla-esr52/rev/60618c6f74d2