Closed Bug 1253510 Opened 10 years ago Closed 9 years ago

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

Categories

(DevTools :: Console, defect, P1)

Product:
DevTools
Component:
Console
Platform:
Unspecified
Linux
Type:
defect

Tracking

(firefox47 affected, firefox48 fixed, firefox49 fixed)

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

People

(Reporter: RyanVM, Assigned: bgrins)

Details

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

Attachments

(1 file)

MozReview Request: Bug 1253510 - Don't use CPOW inside the checkOutputForInputs webconsole helper function;r=pbro
58 bytes, text/x-review-board-request
pbro
: review+
Details
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: 9 years ago
status-firefox49: --- → fixed
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.

Attachment

General

Creator:
Created:
Updated:
Size: