Intermittent browser_notification_open_settings.js | Uncaught exception - at :0 - Error: operation not possible on dead CPOW

RESOLVED FIXED in Firefox 46

Status

()

Toolkit
Notifications and Alerts
RESOLVED FIXED
3 years ago
2 years ago

People

(Reporter: philor, Assigned: mrbkap)

Tracking

({intermittent-failure})

Trunk
mozilla47
intermittent-failure
Points:
---

Firefox Tracking Flags

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

Details

Attachments

(3 attachments)

Comment 1

3 years ago
8 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 5
* fx-team: 2
* mozilla-central: 1

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

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

Comment 2

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

Repository breakdown:
* mozilla-inbound: 4
* fx-team: 2
* b2g-inbound: 1

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

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

Comment 3

2 years ago
11 automation job failures were associated with this bug in the last 7 days.

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

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

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

Comment 4

2 years ago
8 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 3
* fx-team: 3
* try: 2

Platform breakdown:
* osx-10-6: 5
* linux32: 3

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

Comment 5

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

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

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

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1219097&startday=2015-11-30&endday=2015-12-06&tree=all

Comment 6

2 years ago
8 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 3
* b2g-inbound: 3
* try: 1
* fx-team: 1

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

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

Comment 7

2 years ago
6 automation job failures were associated with this bug in the last 7 days.

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

Platform breakdown:
* linux32: 3
* osx-10-6: 2
* windows7-32: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1219097&startday=2015-12-28&endday=2016-01-03&tree=all

Comment 8

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

Repository breakdown:
* try: 2
* mozilla-inbound: 1
* mozilla-central: 1
* b2g-inbound: 1

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

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

Updated

2 years ago
Assignee: nobody → mrbkap
(Assignee)

Comment 9

2 years ago
Created attachment 8722729 [details] [diff] [review]
Fix

These tests use CPOWs incorrectly, leading to races and spurious failures. We can't simply use ContentTask because we need to hold state (the notification object) across a few calls to the content process, so I stuck it on the content window.
Attachment #8722729 - Flags: review?(felipc)
(Assignee)

Comment 10

2 years ago
Created attachment 8722730 [details] [diff] [review]
Fix newlines

There were some DOS newlines in here.
Attachment #8722730 - Flags: review?(felipc)
(Assignee)

Updated

2 years ago
Blocks: 1239563
Attachment #8722729 - Flags: review?(felipc) → review+
Attachment #8722730 - Flags: review?(felipc) → review+
sorry had to back this out, this change here caused problems merging mozilla-central to mozilla-inbound 
warning: conflicts while merging browser/base/content/test/alerts/browser_notification_close.js! (edit, then use 'hg resolve --mark')
Flags: needinfo?(mrbkap)
(Assignee)

Comment 16

2 years ago
I have no idea how that happened. Sorry about that!
Flags: needinfo?(mrbkap)

Comment 17

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/9460b1c1cccd
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox47: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla47

Comment 18

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

Repository breakdown:
* fx-team: 3
* try: 1
* mozilla-inbound: 1

Platform breakdown:
* linux32: 5

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1219097&startday=2016-02-22&endday=2016-02-28&tree=all
status-firefox44: affected → wontfix
status-firefox45: --- → wontfix
status-firefox46: --- → affected
status-firefox-esr45: --- → unaffected
Whiteboard: [checkin-needed-beta]
has problems/conflict on beta:

grafting 330506:9460b1c1cccd "Bug 1219097 - Stop using incorrect CPOWs in these tests. r=felipe"
merging browser/base/content/test/alerts/browser_notification_close.js
merging browser/base/content/test/alerts/browser_notification_do_not_disturb.js
merging browser/base/content/test/alerts/browser_notification_open_settings.js
merging browser/base/content/test/alerts/browser_notification_remove_permission.js
merging browser/base/content/test/alerts/browser_notification_tab_switching.js
merging browser/base/content/test/alerts/head.js
warning: conflicts while merging browser/base/content/test/alerts/browser_notification_close.js! (edit, then use 'hg resolve --mark')
abort: unresolved conflicts, can't continue
(use hg resolve and hg graft --continue)
Flags: needinfo?(mrbkap)
(Assignee)

Comment 21

2 years ago
Hi Carsten, please see the new patch for checkin to beta.
Flags: needinfo?(mrbkap) → needinfo?(cbook)
The merged patch doesn't apply cleanly to beta.
Flags: needinfo?(cbook)

Comment 23

2 years ago
bugherderuplift
https://hg.mozilla.org/releases/mozilla-beta/rev/ad291653a065
status-firefox46: affected → fixed
Whiteboard: [checkin-needed-beta]
You need to log in before you can comment on or make changes to this bug.