Closed Bug 1253510 Opened 8 years ago Closed 8 years ago

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

Categories

(DevTools :: Console, defect, P1)

Unspecified
Linux
defect

Tracking

(firefox47 affected, firefox48 fixed, firefox49 fixed)

RESOLVED FIXED
Firefox 49
Tracking Status
firefox47 --- affected
firefox48 --- fixed
firefox49 --- fixed

People

(Reporter: RyanVM, Assigned: bgrins)

Details

(Keywords: intermittent-failure, Whiteboard: [btpp-backlog])

Attachments

(1 file)

Hit while testing ASAN e10s on Try. I suspect it'd affect debug builds as well if we ran e10s tests on them.

https://treeherder.mozilla.org/logviewer.html#?job_id=17571229&repo=try

 14:06:30     INFO -  378 INFO Waiting for messages...
 14:06:30     INFO -  379 INFO TEST-PASS | devtools/client/webconsole/test/browser_webconsole_clickable_urls.js | matched rule: JS eval output: undefined -
 14:06:30     INFO -  380 INFO Clicking
 14:06:30     INFO -  381 INFO TEST-PASS | devtools/client/webconsole/test/browser_webconsole_clickable_urls.js | the message body -
 14:06:30     INFO -  382 INFO TEST-UNEXPECTED-FAIL | devtools/client/webconsole/test/browser_webconsole_clickable_urls.js | Uncaught exception - at :0 - Error: operation not possible on dead CPOW
 14:06:30     INFO -  Stack trace:
 14:06:30     INFO -      Handler.prototype.process@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:937:23
 14:06:30     INFO -      this.PromiseWalker.walkerLoop@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:816:7
 14:06:30     INFO -      Promise*this.PromiseWalker.scheduleWalkerLoop@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:747:11
 14:06:30     INFO -      this.PromiseWalker.schedulePromise@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:779:7
 14:06:30     INFO -      this.PromiseWalker.completePromise@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:714:7
 14:06:30     INFO -      maybeDone@chrome://mochitests/content/browser/devtools/client/webconsole/test/head.js:1359:7
 14:06:30     INFO -      onMessagesAdded@chrome://mochitests/content/browser/devtools/client/webconsole/test/head.js:1340:15
 14:06:30     INFO -      EventEmitter_emit@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/event-emitter.js:147:11
 14:06:30     INFO -      WebConsoleFrame.prototype._flushMessageQueue@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/client/webconsole/webconsole.js:2073:7
 14:06:30     INFO -      Handler.prototype.process@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:937:23
 14:06:30     INFO -      this.PromiseWalker.walkerLoop@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:816:7
 14:06:30     INFO -      Promise*this.PromiseWalker.scheduleWalkerLoop@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:747:11
 14:06:30     INFO -      this.PromiseWalker.schedulePromise@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:779:7
 14:06:30     INFO -      this.PromiseWalker.completePromise@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:714:7
 14:06:30     INFO -      checkOutputForInputs@chrome://mochitests/content/browser/devtools/client/webconsole/test/head.js:1628:10
 14:06:30     INFO -      @chrome://mochitests/content/browser/devtools/client/webconsole/test/browser_webconsole_clickable_urls.js:102:9
 14:06:30     INFO -      Tester_execTest@chrome://mochikit/content/browser-test.js:786:9
 14:06:30     INFO -      Tester.prototype.nextTest</<@chrome://mochikit/content/browser-test.js:706:7
 14:06:30     INFO -      SimpleTest.waitForFocus/waitForFocusInner/focusedOrLoaded/<@chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:741:59
 14:06:30     INFO -      Tester_execTest@chrome://mochikit/content/browser-test.js:786:9
 14:06:30     INFO -      Tester.prototype.nextTest</<@chrome://mochikit/content/browser-test.js:706:7
 14:06:30     INFO -      SimpleTest.waitForFocus/waitForFocusInner/focusedOrLoaded/<@chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:741:59
 14:06:30     INFO -  383 INFO Leaving test bound
 14:06:30     INFO -  MEMORY STAT | vsize 20973076MB | residentFast 765MB
14:06:30 INFO - 384 INFO TEST-OK | devtools/client/webconsole/test/browser_webconsole_clickable_urls.js | took 17860ms
1 failure total.
Priority: -- → P3
Whiteboard: [btpp-backlog]
Assignee: nobody → bgrinstead
Status: NEW → ASSIGNED
Priority: P3 → P1
Comment on attachment 8751925 [details]
MozReview Request: Bug 1253510 - Don't use CPOW inside the checkOutputForInputs webconsole helper function;r=pbro

https://reviewboard.mozilla.org/r/52315/#review49351

::: devtools/client/webconsole/test/head.js:1575
(Diff revision 1)
>      let tab = event.target;
>      let browser = gBrowser.getBrowserForTab(tab);
> -    loadBrowser(browser).then(() => {
> -      let uri = content.location.href;
> +
> +    Task.spawn(function* () {
> +      yield loadBrowser(browser);
> +      let uri = yield ContentTask.spawn(gBrowser.selectedBrowser, {}, function* () {

If there's only one tab, you could use the `browser` variable defined just above instead of `gBrowser.selectedBrowser`.
Attachment #8751925 - Flags: review?(pbrosset) → review+
Alright, going to switch to the browser variable
https://hg.mozilla.org/mozilla-central/rev/9658aa8c037d
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 49
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.