domprofiler icon indicating copy to clipboard operation
domprofiler copied to clipboard

Investigate how to make this thing work with any debuggee

Open captainbrosset opened this issue 11 years ago • 7 comments

For now this will only work with a local tab because the recorder lives in a frame script that is inserted by the panel (see https://developer.mozilla.org/en-US/docs/The_message_manager). If we want to have this work with any device/process/whatever, the recorder needs to be a protocol.js actor. I don't think there is yet a way to register new actors with the firefox devtools though.

captainbrosset avatar Oct 16 '14 22:10 captainbrosset

It would be extra hawt to be able to use this on other XUL iframes/browser elements, like those use by Firefox Hello. Perhaps somebody in #devtools has suggestions on new actor registration?

dmose avatar Nov 07 '14 17:11 dmose

Yeah that would be great. @janodvarko I'm not exactly aware about the plan to let addon define and load debugger-server actors. Is this something we can do already? Any ongoing work being done?

captainbrosset avatar Nov 08 '14 13:11 captainbrosset

@captainbrosset Yes, I am working on a new platform API that will allow dynamic registration (and unregistration) of actors coming from addons.

See https://bugzilla.mozilla.org/show_bug.cgi?id=977443 (the patch is waiting for a review)

Honza

janodvarko avatar Nov 09 '14 07:11 janodvarko

Awesome, thanks for the link. I'll follow this issue closely. This is a must have API for addons.

captainbrosset avatar Nov 09 '14 08:11 captainbrosset

https://bugzilla.mozilla.org/show_bug.cgi?id=977443 just got fixed in nightly!! Time for some custom actor registration goodness!

captainbrosset avatar Nov 20 '14 21:11 captainbrosset

Woot!

2014-11-20 15:14 GMT-06:00 Patrick Brosset [email protected]:

https://bugzilla.mozilla.org/show_bug.cgi?id=977443 just got fixed in nightly!! Time for some custom actor registration goodness!

— Reply to this email directly or view it on GitHub https://github.com/captainbrosset/domprofiler/issues/10#issuecomment-63880778 .

dmose avatar Nov 21 '14 15:11 dmose

Yep, this is one of the basic stones for custom remote features. Just to note that e10s support for the dynamic actor registration isn't yet ready. Honza

janodvarko avatar Nov 21 '14 16:11 janodvarko