Intermittent browser_devices_get_user_media.js | Unexpected Exception: Error: operation not possible on dead CPOW

RESOLVED FIXED in Firefox 50

Status

()

Firefox
General
P5
normal
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: RyanVM, Assigned: florian)

Tracking

(Blocks: 1 bug, {intermittent-failure})

Trunk
Firefox 52
Unspecified
Linux
intermittent-failure
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(e10s+, firefox48 wontfix, firefox49 wontfix, firefox50 fixed, firefox51 fixed, firefox52 fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

2 years ago
https://treeherder.mozilla.org/logviewer.html#?job_id=2180665&repo=mozilla-aurora

10:42:31     INFO -  518 INFO deny audio, allow video, request audio, expect denied
10:42:31     INFO -  519 INFO requesting devices
10:42:31     INFO -  520 INFO TEST-PASS | browser/base/content/test/general/browser_devices_get_user_media.js | received error: SecurityError: The operation is insecure. -
10:42:31     INFO -  521 INFO TEST-PASS | browser/base/content/test/general/browser_devices_get_user_media.js | expected notification recording-window-ended -
10:42:31     INFO -  522 INFO deny audio, allow video, request video, expect ok (video)
10:42:31     INFO -  523 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_devices_get_user_media.js | Unexpected Exception: Error: operation not possible on dead CPOW -
10:42:31     INFO -  Stack trace:
10:42:31     INFO -  chrome://mochitests/content/browser/browser/base/content/test/general/browser_devices_get_user_media.js:test/onload/<:714
10:42:31     INFO -  resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:Handler.prototype.process:940
10:42:31     INFO -  resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:this.PromiseWalker.walkerLoop:816

Updated

2 years ago
tracking-e10s: ? → +
Intermittent e10s test failure
Priority: -- → P5
(Reporter)

Comment 2

2 years ago
This one's still showing up in the list of regularly-hit dead CPOW oranges. Can you please take a look, Florian?

TEST-UNEXPECTED-FAIL | browser/base/content/test/webrtc/browser_devices_get_user_media.js | Unexpected Exception: Error: operation not possible on dead CPOW -
Stack trace:
   chrome://mochitests/content/browser/browser/base/content/test/webrtc/browser_devices_get_user_media.js:test/onload/<:546
   resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:Handler.prototype.process:940
   resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:this.PromiseWalker.walkerLoop:816
status-firefox48: affected → wontfix
status-firefox49: --- → wontfix
status-firefox50: --- → affected
status-firefox51: --- → affected
Flags: needinfo?(florian)
(In reply to Ryan VanderMeulen [:RyanVM] from comment #2)
> This one's still showing up in the list of regularly-hit dead CPOW oranges.

Where is this list? How many times have we hit this failure? The only link to a log here is 6 months old and the log isn't there anymore.
Flags: needinfo?(florian) → needinfo?(ryanvm)
https://hg.mozilla.org/integration/fx-team/rev/f4a0c92bc46c24ec17ac86e60ffc23f7bc64ab07
Bug 1258142 - add a Cu.reportError to the browser_devices_get_user_media_* tests to get stacks of exceptions in logs, rs=Gijs over IRC.
Keywords: leave-open
2 new failures since we landed the patch for better debugging info, and the 2 failures have the same stack for the CPOW:
promiseMessage@chrome://mochitests/content/browser/browser/base/content/test/webrtc/head.js:251:3
usePerm@chrome://mochitests/content/browser/browser/base/content/test/webrtc/browser_devices_get_user_media.js:287:15

It's unclear to me why this call of promiseMessage sometimes involves a dead CPOW but lots of previous calls to it worked before in the same test run... so I guess I can just remove that CPOW and forget about it :-).
Assignee: nobody → florian
Keywords: leave-open
Created attachment 8793358 [details] [diff] [review]
Patch
Attachment #8793358 - Flags: review?(gijskruitbosch+bugs)

Comment 10

2 years ago
Comment on attachment 8793358 [details] [diff] [review]
Patch

Review of attachment 8793358 [details] [diff] [review]:
-----------------------------------------------------------------

::: browser/base/content/test/webrtc/head.js
@@ +245,5 @@
>    });
>  }
>  
>  function promiseMessage(aMessage, aAction) {
> +  let promise = new Promise((resolve, reject) => {

Can we now also remove any imports of Promise.jsm there might be? Or is this still used elsewhere?
Attachment #8793358 - Flags: review?(gijskruitbosch+bugs) → review+
(In reply to :Gijs Kruitbosch from comment #10)

> Can we now also remove any imports of Promise.jsm there might be? Or is this
> still used elsewhere?

I still see 6 Promise.defer(); calls in this head.js file.

Comment 13

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/6757959dbba2
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox52: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 52
(Reporter)

Comment 14

2 years ago
bugherderuplift
https://hg.mozilla.org/releases/mozilla-aurora/rev/ef567dcc1de7
status-firefox51: affected → fixed
Flags: in-testsuite+
(Reporter)

Comment 15

2 years ago
bugherderuplift
https://hg.mozilla.org/releases/mozilla-beta/rev/781b860baf29
status-firefox50: affected → fixed
You need to log in before you can comment on or make changes to this bug.