Closed
Bug 1226157
Opened 9 years ago
Closed 9 years ago
Opening the debugger breaks on listWorker request and ends up breaking the debugger and the inspector
Categories
(DevTools Graveyard :: WebIDE, defect)
DevTools Graveyard
WebIDE
Tracking
(firefox45 fixed)
RESOLVED
FIXED
Firefox 45
Tracking | Status | |
---|---|---|
firefox45 | --- | fixed |
People
(Reporter: ochameau, Assigned: ochameau)
References
Details
Attachments
(1 file)
901 bytes,
patch
|
fabrice
:
review+
|
Details | Diff | Splinter Review |
We get the following exception: onPacket threw an exception: Error: Server did not specify an actor, dropping packet: {"error":"unknownError","message":"error occurred while processing 'listWorkers: TypeError: Ci.nsIWorkerDebugger is undefined\nStack: BTA_onListWorkers@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/server/actors/webbrowser.js:1100 For some reason, Ci.nsIWorkerDebugger is undefined on b2g... Then Debugger panel is completely broken, which is sad. But what is even more sad is that the inspector is also broken and what is even worse is that you can't restore to a working state unless you reopen webide :/
Assignee | ||
Comment 1•9 years ago
|
||
Oh. One mitigation point, you have to set devtools.debugger.workers to true via about:config or toolbox options to get this exception.
Assignee | ||
Comment 2•9 years ago
|
||
The issue at the end is quite simple, we miss the related xpt file on b2g to have these interfaces defined.
Attachment #8689492 -
Flags: review?(fabrice)
Assignee | ||
Comment 3•9 years ago
|
||
Yi-Fan, could you confirm that you have the workers debugging enabled? If not... there is something else broken that we should figure out.
Comment 4•9 years ago
|
||
Comment on attachment 8689492 [details] [diff] [review] Ship nsIWorkerDebugger* idl on b2g Review of attachment 8689492 [details] [diff] [review]: ----------------------------------------------------------------- r=e but please make the same change in https://mxr.mozilla.org/mozilla-central/source/mobile/android/b2gdroid/installer/package-manifest.in
Attachment #8689492 -
Flags: review?(fabrice) → review+
Assignee | ||
Comment 5•9 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/437a1938d6be57d64de1933cb1589b37ba9264cc Bug 1226157 - Ship nsIWorkerDebugger* idl on b2g. r=fabrice
Comment 6•9 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/437a1938d6be
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox45:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 45
Updated•6 years ago
|
Product: Firefox → DevTools
Updated•4 years ago
|
Product: DevTools → DevTools Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•