933214 - Using Profiler breaks Inspector in future App Manager toolboxes

Status: RESOLVED WORKSFORME

(DevTools Graveyard :: WebIDE, defect, P1)

Description

[Petruta Horea [:phorea], Desktop QA]

Mozilla/5.0 (X11; Linux x86_64; rv:28.0) Gecko/20100101 Firefox/28.0
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:28.0) Gecko/20100101 Firefox/28.0
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.8; rv:28.0) Gecko/20100101 Firefox/28.0
BuildID: 20131030030201

Steps To Reproduce:
1. Open an app
2. Select the Debug button
3. On toolbox select all the tabs and check that they contain data
4. Go to Console tab and close the toolbox
5. Reopen the toolbox and check again the tabs content

Actual Results:
The Inspector and Debugger tabs have no content. Please see the screencast: https://docs.google.com/file/d/0BzbDkQ2Zlg86ZklJaDZWZUNoUkk/edit?usp=sharing

Expected Results:
The tabs content is displayed.

Comment 1

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

This seems like a separate issue from bug 933212 to me.

Here is a more specific STR:

1. Connect to some device (like 1.2 simulator)
2. Start an app from Installed Apps
3. Debug this app
4. The toolbox starts on the Console tab
5. Click the Profiler tab
6. Close the toolbox
7. Debug the same app again
8. Click the Inspector tab
9. The Inspector is blank

Also, many fun errors are logged to the console:

console.error:
  Message: TypeError: v is undefined
  Stack:
    types.addActorType/type<.read@resource://gre/modules/commonjs/toolkit/loader.js -> file:///Users/jryans/projects/mozilla/gecko/toolkit/devtools/server//protocol.js:257
RetVal<.read@resource://gre/modules/commonjs/toolkit/loader.js -> file:///Users/jryans/projects/mozilla/gecko/toolkit/devtools/server//protocol.js:479
Response<.read@resource://gre/modules/commonjs/toolkit/loader.js -> file:///Users/jryans/projects/mozilla/gecko/toolkit/devtools/server//protocol.js:629
frontProto/</proto[name]/<@resource://gre/modules/commonjs/toolkit/loader.js -> file:///Users/jryans/projects/mozilla/gecko/toolkit/devtools/server//protocol.js:1156
resolve@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/core/promise.js:118
then@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/core/promise.js:43
resolve@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/core/promise.js:185
Front<.onPacket@resource://gre/modules/commonjs/toolkit/loader.js -> file:///Users/jryans/projects/mozilla/gecko/toolkit/devtools/server//protocol.js:1060
DC_onPacket/<@resource://gre/modules/devtools/dbg-client.jsm:664
resolve@resource://gre/modules/commonjs/sdk/core/promise.js:118
then@resource://gre/modules/commonjs/sdk/core/promise.js:43
then@resource://gre/modules/commonjs/sdk/core/promise.js:153
DC_onPacket@resource://gre/modules/devtools/dbg-client.jsm:705
DT__processIncoming/<@resource://gre/modules/devtools/dbg-client.jsm -> resource://gre/modules/devtools/server/transport.js:201
makeInfallible/<@resource://gre/modules/devtools/DevToolsUtils.jsm -> resource://gre/modules/devtools/DevToolsUtils.js:75

console.error:
  Message: Error: Unexpected packet from conn0.child1002:5, undefined
  Stack:
    Front<.onPacket@resource://gre/modules/commonjs/toolkit/loader.js -> file:///Users/jryans/projects/mozilla/gecko/toolkit/devtools/server//protocol.js:1051
DC_onPacket/<@resource://gre/modules/devtools/dbg-client.jsm:664
resolve@resource://gre/modules/commonjs/sdk/core/promise.js:118
then@resource://gre/modules/commonjs/sdk/core/promise.js:43
then@resource://gre/modules/commonjs/sdk/core/promise.js:153
DC_onPacket@resource://gre/modules/devtools/dbg-client.jsm:705
DT__processIncoming/<@resource://gre/modules/devtools/dbg-client.jsm -> resource://gre/modules/devtools/server/transport.js:201
makeInfallible/<@resource://gre/modules/devtools/DevToolsUtils.jsm -> resource://gre/modules/devtools/DevToolsUtils.js:75

Comment 2

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

I attempted to find a regression window for this, but it's been broken for the entire life of the App Manager.

Comment 3

[Panos Astithas (he/him) [:past] (please ni?)]

I remember seeing something similar, but this seems fixed now.

Comment 4

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

When I try to reproduce, I get the broken toolbox / gray screen of bug 958002, so I am not sure if this is fixed or not.

Comment 5

[Paul Rouget [:paul]]

(In reply to J. Ryan Stinnett [:jryans] from comment #4)
> When I try to reproduce, I get the broken toolbox / gray screen of bug
> 958002, so I am not sure if this is fixed or not.

Is this still happening?

Comment 6

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

(In reply to Paul Rouget [:paul] from comment #5)
> (In reply to J. Ryan Stinnett [:jryans] from comment #4)
> > When I try to reproduce, I get the broken toolbox / gray screen of bug
> > 958002, so I am not sure if this is fixed or not.
> 
> Is this still happening?

No, it now works for me too, as Panos said.  Not sure what fixed it though.

I did see an issue with Style Editor while testing this, so I'll file that separately if there's no dupe already.