Closed
Bug 945663
Opened 11 years ago
Closed 3 years ago
Sometimes, the browser toolbox is not loaded (only a black toolbar and the close button). Error: closeButton is null
Categories
(DevTools :: Framework, defect)
Tracking
(Not tracked)
RESOLVED
INACTIVE
People
(Reporter: paul, Unassigned)
References
(Blocks 1 open bug)
Details
Attachments
(1 obsolete file)
This happens because, in toolbox.js, domReady in toolbox.open() is called too early. Probably related to bug 936892.
Comment 1•11 years ago
|
||
I had this happen for a bit and then it went away. I wonder if the XUL cache or its friends are involved.
Reporter | ||
Comment 2•11 years ago
|
||
(In reply to :Gijs Kruitbosch from comment #1)
> I had this happen for a bit and then it went away. I wonder if the XUL cache
> or its friends are involved.
This is very likely.
Assignee: nobody → paul
Comment 3•10 years ago
|
||
I get the same problem, but I see the following error in the console of the Browser Toolbox process:
[Exception... "Component returned failure code: 0x804b000d (NS_ERROR_CONNECTION_REFUSED) [nsIInputStream.available]" nsresult: "0x804b000d (NS_ERROR_CONNECTION_REFUSED)" location: "JS frame :: resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/security/socket.js :: onInputStreamReady :: line 300" data: no]
This is after starting the main process with
./mach run --jsdebugger
then closing the Browser Toolbox, and opening another process with basically the same command that runs from ToolboxProcess.jsm [0]:
./mach run --no-remote --foreground --profile /tmp/chrome-debugging -chrome chrome://browser/content/devtools/framework/toolbox-process-window.xul --purgecaches --jsconsole
[0]: https://dxr.mozilla.org/mozilla-central/source/browser/devtools/framework/ToolboxProcess.jsm#208
Comment 4•10 years ago
|
||
With this patch, it at least prints out the error message in the toolbox instead of showing a blank page. Here is a way I can reliably trigger an error connecting:
1) ./mach run --jsdebugger
2) Close the browser toolbox
3) ./mach run --profile /tmp/foobar -chrome chrome://browser/content/devtools/framework/toolbox-process-window.xul --purgeches --jsconsole
4) You should see the error message from Comment 3 in the opened window
Attachment #8578930 -
Flags: review?(jryans)
Comment on attachment 8578930 [details] [diff] [review]
browser-toolbox-error-message.patch
I am okay with adding this error display, but I think we should move this to a separate bug, because it does not solve the issue originally reported, and your case is different anyway. Maybe this bug can depend on a bug with your error patch, to aid debugging the original issue? I think it makes sense, but let me know if it seems like too much bookkeeping work...
This bug is about using the normal Browser Toolbox from the menu item, but getting the blank window. Your steps change the assumptions of the Browser Toolbox: it stops the server when the toolbox (that was launched via its menu item) is closed. So, the second toolbox you start for the terminal has no server to connect to.
To run Browser Toolbox manually like you're doing here, I will typically start the server manually with "listen 6080" in GCLI. That way the server is manually managed, and won't be terminated when the Browser Toolbox closes.
I just tried this now, and it does get around the nsIInputStream exception your reported. However, it gives a different error, which does not appear in your error display on shut down:
Handler function threw an exception: [Exception... "Component returned failure code: 0x80004002 (NS_NOINTERFACE) [nsIInterfaceRequestor.getInterface]" nsresult: "0x80004002 (NS_NOINTERFACE)" location: "JS frame :: resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/actors/webbrowser.js :: TabActor.prototype.window :: line 676" data: no]
Stack: TabActor.prototype.window@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/actors/webbrowser.js:676:0
TabActor.prototype._windowReady@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/actors/webbrowser.js:1389:8
TabActor.prototype._changeTopLevelDocument/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/actors/webbrowser.js:1354:6
makeInfallible/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/DevToolsUtils.js:82:13
Line: 676, column: 0
Attachment #8578930 -
Flags: review?(jryans)
Updated•10 years ago
|
Attachment #8578930 -
Attachment is obsolete: true
Comment 6•10 years ago
|
||
(In reply to J. Ryan Stinnett [:jryans] from comment #5)
> Comment on attachment 8578930 [details] [diff] [review]
> browser-toolbox-error-message.patch
>
> I am okay with adding this error display, but I think we should move this to
> a separate bug, because it does not solve the issue originally reported, and
> your case is different anyway. Maybe this bug can depend on a bug with your
> error patch, to aid debugging the original issue? I think it makes sense,
> but let me know if it seems like too much bookkeeping work...
>
Filed bug 1144392 for this.
> This bug is about using the normal Browser Toolbox from the menu item, but
> getting the blank window. Your steps change the assumptions of the Browser
> Toolbox: it stops the server when the toolbox (that was launched via its
> menu item) is closed. So, the second toolbox you start for the terminal has
> no server to connect to.
I actually had the same problem when just closing and reopening the browser toolbox with the menu but I knew it didn't reproduce for other people. But will need to investigate this problem further when manually listening from the first process.
Updated•10 years ago
|
Reporter | ||
Updated•9 years ago
|
Assignee: paul → nobody
Comment 7•7 years ago
|
||
Are we getting any update on this asap? I am really not able to debug any addon due to same error comes up while clicking debug button on addon. I have also found that now I don't see any window prompting me to accept connection to ip and port address.
Comment 8•7 years ago
|
||
(In reply to akanshgulati2009 from comment #7)
> Are we getting any update on this asap? I am really not able to debug any
> addon due to same error comes up while clicking debug button on addon. I
> have also found that now I don't see any window prompting me to accept
> connection to ip and port address.
Since you aren't seeing the prompt, this could be the same issue as originally reported in https://bugzilla.mozilla.org/show_bug.cgi?id=1438962#c0. We landed some changes that print out diagnostics to help debug this issue, so if you can test with the most recent Nightly build and report back the log you see that would be helpful (it should show up in the addon toolbox window after ~20s if the toolbox fails to open).
Comment 9•7 years ago
|
||
@Brian G(In reply to Brian Grinstead [:bgrins] from comment #8)
> (In reply to akanshgulati2009 from comment #7)
> > Are we getting any update on this asap? I am really not able to debug any
> > addon due to same error comes up while clicking debug button on addon. I
> > have also found that now I don't see any window prompting me to accept
> > connection to ip and port address.
>
> Since you aren't seeing the prompt, this could be the same issue as
> originally reported in
> https://bugzilla.mozilla.org/show_bug.cgi?id=1438962#c0. We landed some
> changes that print out diagnostics to help debug this issue, so if you can
> test with the most recent Nightly build and report back the log you see that
> would be helpful (it should show up in the addon toolbox window after ~20s
> if the toolbox fails to open).
Thanks man. I installed Nightly in a freshly created new user on Mac OS and debugging of addon started working with and without prompt window of connection.
Updated•6 years ago
|
Product: Firefox → DevTools
Updated•3 years ago
|
Blocks: browser-toolbox
Comment 10•3 years ago
|
||
The error message is now shown and there is bug 1146414 reported for the nsIInputStream.available
error message.
Closing.
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → INACTIVE
You need to log in
before you can comment on or make changes to this bug.
Description
•