Closed
Bug 1237124
Opened 8 years ago
Closed 8 years ago
Intermittent browser_webconsole_bug_658368_time_methods.js | Uncaught exception - at :0 - Error: operation not possible on dead CPOW
Categories
(DevTools :: Console, defect, P3)
DevTools
Console
Tracking
(e10s+, firefox48 fixed)
RESOLVED
FIXED
Firefox 48
People
(Reporter: KWierso, Assigned: bgrins)
References
(Blocks 1 open bug)
Details
(Keywords: intermittent-failure, Whiteboard: [btpp-backlog])
Attachments
(1 file)
Comment hidden (Intermittent Failures Robot) |
Updated•8 years ago
|
tracking-e10s:
--- → ?
Updated•8 years ago
|
Blocks: e10s-tests
Updated•8 years ago
|
Comment 2•8 years ago
|
||
This one popped up again with a couple of failures in the past few weeks.
Priority: -- → P3
Whiteboard: [btpp-backlog]
Assignee | ||
Comment 3•8 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/41567/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/41567/
Attachment #8733118 -
Flags: review?(lclark)
Assignee | ||
Comment 4•8 years ago
|
||
Saw this come up on a try push, so going to fix it up. New try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=7e8b6f673d20
Assignee: nobody → bgrinstead
Status: NEW → ASSIGNED
Updated•8 years ago
|
Attachment #8733118 -
Flags: review?(lclark) → review+
Comment 5•8 years ago
|
||
Comment on attachment 8733118 [details] MozReview Request: Bug 1237124 - Remove CPOW usage in browser_webconsole_bug_658368_time_methods.js;r=linclark https://reviewboard.mozilla.org/r/41567/#review38201 ::: devtools/client/webconsole/test/browser_webconsole_bug_658368_time_methods.js:49 (Diff revision 1) > "bTimer was not started", false, true); > > // The next test makes sure that timers with the same name but in separate > // pages, do not contain the same value. > - content.location = TEST_URI3; > + ContentTask.spawn(gBrowser.selectedBrowser, TEST_URI3, function*(location) { > + content.location = location; Should we use BrowserTestUtils.loadURI here instead of content.location? I think you might have suggested that for one of my patches before. I'm fine with committing it as is, though.
Assignee | ||
Comment 6•8 years ago
|
||
(In reply to Lin Clark [:linclark] from comment #5) > Comment on attachment 8733118 [details] > MozReview Request: Bug 1237124 - Remove CPOW usage in > browser_webconsole_bug_658368_time_methods.js;r=linclark > > https://reviewboard.mozilla.org/r/41567/#review38201 > > ::: > devtools/client/webconsole/test/browser_webconsole_bug_658368_time_methods. > js:49 > (Diff revision 1) > > "bTimer was not started", false, true); > > > > // The next test makes sure that timers with the same name but in separate > > // pages, do not contain the same value. > > - content.location = TEST_URI3; > > + ContentTask.spawn(gBrowser.selectedBrowser, TEST_URI3, function*(location) { > > + content.location = location; > > Should we use BrowserTestUtils.loadURI here instead of content.location? I > think you might have suggested that for one of my patches before. I'm fine > with committing it as is, though. Right, good idea. Going to make that change, make sure it passes and then land
Comment 8•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/1f504df693e9
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox48:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•