Closed
Bug 988408
Opened 10 years ago
Closed 10 years ago
toolbox.js - focusTool() - TypeError: this.doc is undefined
Categories
(DevTools :: Framework, defect)
DevTools
Framework
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 33
People
(Reporter: msucan, Assigned: past)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
While running webconsole tests I see the following exception: 1:58.75 TEST-PASS | chrome://mochitests/content/browser/browser/devtools/webconsole/test/browser_webconsole_js_input_expansion.js | height shrank to original size within 1px 1:58.75 ************************* 1:58.75 A coding exception was thrown in a Promise resolution callback. 1:58.75 1:58.75 Full message: TypeError: this.doc is undefined 1:58.75 See https://developer.mozilla.org/Mozilla/JavaScript_code_modules/Promise.jsm/Promise 1:58.75 Full stack: Toolbox.prototype.focusTool@resource://gre/modules/commonjs/toolkit/loader.js -> resource:///modules/devtools/framework/toolbox.js:872:1 1:58.75 Toolbox.prototype.selectTool/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource:///modules/devtools/framework/toolbox.js:858:7 1:58.75 Handler.prototype.process@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:707:11 1:58.75 this.PromiseWalker.walkerLoop@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:586:7 1:58.75 1:58.75 ************************* This is happening in several tests, not just the one mentioned here.
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → past
Assignee | ||
Comment 1•10 years ago
|
||
Attachment #8446623 -
Flags: review?(jwalker)
Assignee | ||
Comment 2•10 years ago
|
||
Hunh, bzexport seems to eat my comments lately, so here it goes: This fixes 13 test failures in devtools with the patch from bug 1016387 applied. The first problem (that affected many tests) was that when opening the toolbox we were not waiting for the tool panel to be selected before firing toolbox-ready. The second was that we weren't using the selectTool() promise in one case to serialize execution. I got to remove a bunch of test code that waited on tool-selected after opening the toolbox as a result. Try: https://tbpl.mozilla.org/?tree=Try&rev=de22d140c0f3
Updated•10 years ago
|
Attachment #8446623 -
Flags: review?(jwalker) → review+
Assignee | ||
Comment 3•10 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/afb676164612
Status: NEW → ASSIGNED
OS: Linux → All
Hardware: x86_64 → All
Whiteboard: [fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/afb676164612
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 33
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•