Closed Bug 789114 Opened 8 years ago Closed 8 years ago

Make B2G and Fennec browser actors inherit the extensibility API changes

Categories

(DevTools :: Debugger, defect, P2)

defect

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 18

People

(Reporter: past, Assigned: past)

References

Details

Attachments

(1 file, 1 obsolete file)

The patch in bug 753401 allows add-ons to provide their own actors for the remote protocol. However this was tested and verified to work for desktop add-ons only, and B2G and Fennec add-ons need some more changes to work.
Attached patch Patch v1 (obsolete) — Splinter Review
Trying to minimize changes to B2G and Fennec actors. glandium can you verify that this patch works for you in both cases?
Attachment #658924 - Flags: feedback?(mh+mozilla)
Comment on attachment 658924 [details] [diff] [review]
Patch v1

As said on irc, only tested on b2g, assuming the fennec part corresponds to the previous iteration (which, looking at the code, looks like is the case)
Attachment #658924 - Flags: feedback?(mh+mozilla) → feedback+
Attached patch Patch v2Splinter Review
This is slightly better. If you can confirm it still works, I'll ask for reviews.
Attachment #658924 - Attachment is obsolete: true
Attachment #659727 - Flags: feedback?(mh+mozilla)
Comment on attachment 659727 [details] [diff] [review]
Patch v2

It works for me on Fennec (haven't checked B2G)
Attachment #659727 - Flags: feedback?(mh+mozilla) → feedback+
Comment on attachment 659727 [details] [diff] [review]
Patch v2

This patch allows dynamically-added actors to be usable via the remote debugging protocol for Fennec and B2G. For B2G I'm not sure if add-ons will be allowed (perhaps provided by the vendor, like what we do with PDF.js in desktop?), but the API is still useful regardless. The changes are pretty straight-forward.
Attachment #659727 - Flags: review?(mihai.sucan)
Attachment #659727 - Flags: review?(mark.finkle)
Attachment #659727 - Flags: review?(21)
Comment on attachment 659727 [details] [diff] [review]
Patch v2

The Fennec changes look fine
Attachment #659727 - Flags: review?(mark.finkle) → review+
Comment on attachment 659727 [details] [diff] [review]
Patch v2

Looks good.
Attachment #659727 - Flags: review?(mihai.sucan) → review+
Blocks: 751034
Depends on: 790952
Blocks: 790952
No longer depends on: 790952
https://hg.mozilla.org/mozilla-central/rev/642f820edff8
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 18
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.