Open
Bug 899054
(dbg-addon)
Opened 11 years ago
Updated 2 years ago
[meta] Add-on debugging
Categories
(DevTools :: Debugger, task, P5)
Tracking
(a11y-review:requested)
NEW
a11y-review | requested |
People
(Reporter: ejpbruel, Unassigned)
References
(Depends on 11 open bugs)
Details
(Keywords: meta)
Attachments
(1 file)
(deleted),
application/octet-stream
|
Details |
Now that the remote browser debugger can attach to add-on scripts, it is possible to debug add-ons with it. Doing so is still cumbersome, however, since the browser debugger attaches to all globals regardless of whether they are part of a specific add-on or not. What we want is for the browser debugger to attach to a specific add-on, rather than the entire browser.
In terms of implementation, our proposal is to create a specialization of the chrome thread actor used by the debugger, which we will call an add-on thread actor (bug 898559). In contrast to the chrome thread actor, which adds *all* globals as debuggees, an add-on thread actor only adds globals that are part of a given add-on.
In addition, the root actor should be extended to respond to a request for a list of add-ons by sending a reply containing a list of add-on thread actors, one for each (active) add-on. Clients can then attach to/detach from these add-on thread actors on an individual basis.
The add-on thread actor must be able to tell which globals are part of a given add-on and which are not. To facilitate this, we propose implementing a metadata API (bug 898559) which allows globals to be tagged as such.
Once we've implemented add-on thread actors, we propose to extend the add-on manager UI with an option 'debug this add-on' for each active add-on. Clicking this option will cause the browser debugger to be started for a specific add-on (by passing it the add-on ID as a parameter).
Depends on: 898229
Updated•11 years ago
|
Priority: -- → P5
Comment 1•11 years ago
|
||
(In reply to Eddy Bruel [:ejpbruel] from comment #1)
> In terms of implementation, our proposal is to create a specialization of
> the chrome thread actor used by the debugger, which we will call an add-on
> thread actor (bug 898559). In contrast to the chrome thread actor, which
> adds *all* globals as debuggees, an add-on thread actor only adds globals
> that are part of a given add-on.
For anyone else reading and being confused like me: I think the bug number was meant to be bug 899052.
Updated•11 years ago
|
Assignee: nobody → rFobic
Updated•11 years ago
|
Assignee: rFobic → jsantell
Updated•11 years ago
|
Alias: addon-dbg
Updated•10 years ago
|
Assignee: jsantell → nobody
Reporter | ||
Updated•10 years ago
|
Summary: [Meta] Implement an Add-on Debugger → [meta] Add-on debugging
Reporter | ||
Updated•10 years ago
|
Alias: addon-dbg → dbg-addon
Depends on: 1060093
Depends on: 1075172
Depends on: 1076796
Depends on: 1075185
Depends on: 1106689
Depends on: 1116113
Depends on: 1116113
Depends on: 1122144
Depends on: 1191802
Depends on: 1208590
No longer depends on: 1116160
Updated•6 years ago
|
Product: Firefox → DevTools
Comment 2•6 years ago
|
||
Honza, Harald, could you triage this meta?
Flags: needinfo?(odvarko)
Flags: needinfo?(hkirschner)
Updated•6 years ago
|
Type: defect → task
Updated•5 years ago
|
Flags: needinfo?(hkirschner)
Updated•2 years ago
|
Attachment #9281940 -
Flags: sec-approval?
Attachment #9281940 -
Flags: data-review+
Attachment #9281940 -
Flags: checkin+
Attachment #9281940 -
Flags: approval-mozilla-beta?
Attachment #9281940 -
Flags: a11y-review+
Updated•2 years ago
|
Performance Impact: ? → ---
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•