1327693 - Can't close devtools with opened inspector while tab is busy/loading

Status: RESOLVED WORKSFORME

(DevTools :: Inspector, defect, P3)

Description

[arni2033]

>>>   My Info:   Win7_64, Nightly 49, 32bit, ID 20160526082509
STR_1:
0. Open devtools -> inspector, close devtools
1. Open url http://forums.somethingawful.com/showthread.php?threadid=1170645%26userid=48835
  (you may need to enter the captcha to visit that url)
2. When the page starts loading, open devtools [inspector tab will be selected]
3. Wait 5 seconds
4. Click on close button in devtools

AR:  Step 2, Step 3 - inspector shows nothing. Step 4 - no visible action
ER:  Step 4 - devtools should close

Comment 1

[Julian Descottes [:jdescottes]]

This bug seems similar to Bug 1330573. 

It has the same stack trace, and it only occurs after https://hg.mozilla.org/mozilla-central/rev/b105a99a50cf (Bug 1245921).
I tried applying the patch from Bug 1328004, but this time it doesn't fix the issue.

Stack trace below, but there are several other errors, all linked to issues with the inspector being destroyed while loading.

> TypeError: this.inspector is null
> Stack trace:
> _getPageStyle@chrome://devtools/content/inspector/inspector.js:290:5
> Inspector.prototype.init<@chrome://devtools/content/inspector/inspector.js:130:11
> _run@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:311:39
> process@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:917:23
> walkerLoop@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:801:7
> Promise*scheduleWalkerLoop@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:734:11
> schedulePromise@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:765:7
> completePromise@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:702:7
> onPacket/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/protocol.js:1329:9
> DevTools RDP*request@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/protocol.js:1269:14
> generateRequestMethods/</frontProto[name]@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/protocol.js:1426:14
> initCssProperties<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/fronts/css-properties.js:237:16
> _run@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:311:39
> TaskImpl@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:273:3
> asyncFunction@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:247:14
> Inspector.prototype.init<@chrome://devtools/content/inspector/inspector.js:127:33
> _run@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:311:39
> TaskImpl@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:273:3
> asyncFunction@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:247:14
> open@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/client/inspector/panel.js:12:12
> onLoad@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/client/framework/toolbox.js:1437:19

Comment 2

[Patrick Brosset <:pbro>]

I don't believe this happens anymore. Closing.