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


(DevTools :: Inspector, defect)

Not set


(firefox26 fixed, firefox27 fixed, firefox28 fixed, b2g-v1.2 fixed)

Firefox 28
Tracking Status
firefox26 --- fixed
firefox27 --- fixed
firefox28 --- fixed
b2g-v1.2 --- fixed


(Reporter: RyanVM, Assigned: bgrins)


(Blocks 1 open bug)


(Whiteboard: [qa-])


(1 file)

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.

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
I'm not sure if this message just began to reappear after fixes from Bug 932880 (specifically, 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
Assignee: nobody → bgrinstead
QA Contact: bgrinstead
Small change to bail out if inspector panel has been destroyed, by checking for this._destroyPromise, as onNewRoot does.

Pushed to try:
Attachment #826823 - Flags: review?(pbrosset)
There was something wrong with the last try push, here is another one:
Comment on attachment 826823 [details] [diff] [review]

Review of attachment 826823 [details] [diff] [review]:

Attachment #826823 - Flags: review?(pbrosset) → review+
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.
Closed: 9 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 28
Whiteboard: [qa-]
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.