Closed
Bug 934487
Opened 9 years ago
Closed 9 years ago
"TypeError: this.markup is undefined: InspectorPanel_onNewRoot/</<@resource://gre/modules/commonjs/toolkit/loader.js -> resource:///modules/devtools/inspector/inspector-panel.js:361" in all mochitest-bc runs
Categories
(DevTools :: Inspector, defect)
DevTools
Inspector
Tracking
(firefox26 fixed, firefox27 fixed, firefox28 fixed, b2g-v1.2 fixed)
RESOLVED
FIXED
Firefox 28
People
(Reporter: RyanVM, Assigned: bgrins)
References
(Blocks 1 open bug)
Details
(Whiteboard: [qa-])
Attachments
(1 file)
1.07 KB,
patch
|
pbro
:
review+
|
Details | Diff | Splinter Review |
Having stared at a LOT of mochitest-bc logs lately (as all of the above CCed also have), I've seen this staring at me and figured it should be filed. It affects 26/27/28. https://tbpl.mozilla.org/php/getParsedLog.php?id=30079315&tree=Mozilla-Inbound Windows 7 32-bit mozilla-inbound debug test mochitest-browser-chrome on 2013-11-04 06:10:37 PST for push fec854fceba6 slave: t-w732-ix-063 07:01:59 INFO - TEST-PASS | chrome://mochitests/content/browser/browser/devtools/inspector/test/browser_inspector_select_last_selected.js | Node re-selected after navigation 07:01:59 INFO - [Parent 3252] WARNING: No inner window available!: file e:/builds/moz2_slave/m-in-w32-d-0000000000000000000/build/dom/base/nsGlobalWindow.cpp, line 9281 07:01:59 INFO - TypeError: this.markup is undefined: InspectorPanel_onNewRoot/</<@resource://gre/modules/commonjs/toolkit/loader.js -> resource:///modules/devtools/inspector/inspector-panel.js:361 07:01:59 INFO - @resource://gre/modules/commonjs/toolkit/loader.js -> resource:///modules/devtools/shared/event-emitter.js:63 07:01:59 INFO - EventEmitter_emit@resource://gre/modules/commonjs/toolkit/loader.js -> resource:///modules/devtools/shared/event-emitter.js:110 07:01:59 INFO - InspectorPanel__onMarkupFrameLoad@resource://gre/modules/commonjs/toolkit/loader.js -> resource:///modules/devtools/inspector/inspector-panel.js:676 07:01:59 INFO - InspectorPanel_initMarkupPanel_onload@resource://gre/modules/commonjs/toolkit/loader.js -> resource:///modules/devtools/inspector/inspector-panel.js:658
Assignee | ||
Comment 1•9 years ago
|
||
I'm not sure if this message just began to reappear after fixes from Bug 932880 (specifically https://bugzilla.mozilla.org/attachment.cgi?id=825590&action=diff#a/browser/devtools/inspector/test/browser_inspector_select_last_selected.js_sec1), or if this has been around since before that. I know I've seen this message in the past, but I'm not sure if it was on every b-c run. I removed a bunch of executeSoons in that test to fix some of the leaks - I will take a look at it.
QA Contact: bgrinstead
Updated•9 years ago
|
Assignee: nobody → bgrinstead
QA Contact: bgrinstead
Assignee | ||
Comment 2•9 years ago
|
||
Small change to bail out if inspector panel has been destroyed, by checking for this._destroyPromise, as onNewRoot does. Pushed to try: https://tbpl.mozilla.org/?tree=Try&rev=6598a4c8d6c3
Attachment #826823 -
Flags: review?(pbrosset)
Assignee | ||
Comment 3•9 years ago
|
||
There was something wrong with the last try push, here is another one: https://tbpl.mozilla.org/?tree=Try&rev=8ce73af51454
Comment 4•9 years ago
|
||
Comment on attachment 826823 [details] [diff] [review] markup-undefined.patch Review of attachment 826823 [details] [diff] [review]: ----------------------------------------------------------------- LGTM
Attachment #826823 -
Flags: review?(pbrosset) → review+
Comment 5•9 years ago
|
||
I think we'll have to improve how we handle these things: - when a test ends abruptly without waiting for a server-side response for instance - when the toolbox is closed - when a page is navigated away. For now, we don't have that many nullchecks or early return statements as you did here, but we may have to add more and more as we fix bugs, and although it doesn't look very good, I don't see any other way, given that most of our code is async and relying on client-server communication. On the server-side inspector for instance, there are still many scenarios where, when a request for a given node comes, that node has already been removed from the DOM.
Assignee | ||
Comment 6•9 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/20fb98a374ce https://tbpl.mozilla.org/?tree=Fx-Team&rev=20fb98a374ce
Status: NEW → ASSIGNED
Whiteboard: [fixed-in-fx-team]
Reporter | ||
Comment 7•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/20fb98a374ce
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 28
Reporter | ||
Comment 8•9 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/58a25d20bbad https://hg.mozilla.org/releases/mozilla-beta/rev/ca693fb76f9b
Reporter | ||
Comment 9•9 years ago
|
||
https://hg.mozilla.org/releases/mozilla-b2g26_v1_2/rev/ca693fb76f9b
status-b2g-v1.2:
--- → fixed
Updated•9 years ago
|
Whiteboard: [qa-]
Updated•4 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•