Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Investigate how to make this thing work with any debuggee #10

Open
captainbrosset opened this issue Oct 16, 2014 · 7 comments
Open

Investigate how to make this thing work with any debuggee #10

captainbrosset opened this issue Oct 16, 2014 · 7 comments

Comments

@captainbrosset
Copy link
Owner

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.

@dmose
Copy link

dmose commented Nov 7, 2014

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?

@captainbrosset
Copy link
Owner Author

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?

@janodvarko
Copy link

@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

@captainbrosset
Copy link
Owner Author

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

@captainbrosset
Copy link
Owner Author

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

@dmose
Copy link

dmose commented Nov 21, 2014

Woot!

2014-11-20 15:14 GMT-06:00 Patrick Brosset notifications@github.com:

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
#10 (comment)
.

@janodvarko
Copy link

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants