As far as the remote debugging protocol is concerned, chrome debugging is mostly a global thread actor that has visibility to all JS running in the application process. We should add such capability to Firefox OS in order to help debugging Firefox itself. The patch in bug 817580 adds a special "tab" target that corresponds to the shell.xul window, but having the client connect to the Remote Process target would be more natural and less of a hack.
Filter on BLACKEAGLE2.
Priority: -- → P3
Bug 817580 won't be adding the chrome window as a debuggee after all, so this bug should offer it from the chromeDebuger actor, as we do in desktop Firefox.
Assignee: nobody → past
Status: NEW → ASSIGNED
Priority: P3 → P2
Comment on attachment 716075 [details] [diff] [review] Patch v1 Review of attachment 716075 [details] [diff] [review]: ----------------------------------------------------------------- r=me with comment addressed ::: b2g/chrome/content/dbg-browser-actors.js @@ +66,1 @@ > let actor = this._tabActors.get(this.browser); Remove the let here.
(In reply to Fabrice Desré [:fabrice] from comment #4) > r=me with comment addressed > > ::: b2g/chrome/content/dbg-browser-actors.js > @@ +66,1 @@ > > let actor = this._tabActors.get(this.browser); > > Remove the let here. Good catch, thanks.
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 22
Fabrice, can we uplift this to the mozilla-b2g18 branch, so that it can be picked up by the simulator? If so, what flags should I set?
You need to ask for approval‑mozilla‑b2g18 in the patch details.
Comment on attachment 716149 [details] [diff] [review] Patch v2 NOTE: Please see https://wiki.mozilla.org/Release_Management/B2G_Landing to better understand the B2G approval process and landings. [Approval Request Comment] Bug caused by (feature/regressing bug #): new feature that we want to get into the Firefox OS Simulator User impact if declined: developers using the Firefox OS Simulator or b2g-desktop will not be able to connect to the "Main Process" target and have a working debugger. Currently their only choice for a working debugger is the "shell.xul" target, which unfortunately results in a broken web console due to a different bug. Testing completed: in m-i and m-c. Risk to taking this patch (and alternatives if risky): very low risk, the patch is only compiled for desktop b2g builds (not for devices) is a trivial port of the relevant desktop functionality, and will be used/triggered by developers only String or UUID changes made by this patch: none
Comment on attachment 716149 [details] [diff] [review] Patch v2 D'oh, apparently chrome debugging for desktop Firefox landed in Firefox 19, so there is nothing to do for 18.
You need to log in before you can comment on or make changes to this bug.