875103 - Add remote debugger support to Marionette

Status: RESOLVED DUPLICATE of bug 1100124

(Remote Protocol :: Marionette, defect, P5)

Description

[Jonathan Griffin (:jgriffin)]

After bug 797529 lands, Marionette and the remote debugger will no longer be mutually exclusive.  At that point, it would be great to enable remote debugging of Marionette scripts, so that a developer debugging one could step through or pause such a script during execution.

It looks like the way to approach that would be to create a debugger actor for Marionette to use to execute scripts (maybe similar to what is being done for Scratchpad in bug 825039).  Panos, would that approach work for Marionette?

I think we'd need to keep the current evalInSandbox approach as well, for cases where the debugger isn't active (e.g., B2G).

Comment 1

[Panos Astithas (he/him) [:past] (please ni?)]

I haven't looked into bug 797529 yet, but wouldn't just using the chrome debugging actor (and the Browser Debugger tool) work for your case? Bug 809561 has a similar goal for the xpcshell test harness and a bit more detail, and there may be bugs filed for mochitests as well.

Comment 2

[Jonathan Griffin (:jgriffin)]

Cool, thanks, I haven't looked into it in any detail yet.  Marionette executes scripts in content as well as chrome; would the chrome debugging actor be usable for that?

Comment 3

[Panos Astithas (he/him) [:past] (please ni?)]

Yes, the only scripts that are off-limits are the ones that run in the debugger server's compartment (server machinery and actors).

Comment 4

[David Burns :automatedtester]

jgriffin is this still needed?

Comment 5

[Jonathan Griffin (:jgriffin)]

It would be nice, but would be pretty disruptive, probably, so I'd rank it as pretty low priority.