832163 - Add a command for toggling the debugger server on desktop

Status: RESOLVED FIXED

(DevTools :: Debugger, defect, P3)

Description

[Victor Porof [:vporof][:vp]]

See https://bugzilla.mozilla.org/show_bug.cgi?id=751677#c5:

"For desktop vs. desktop debugging sessions, some UI for starting a server is also needed. Ninjas currently rely on running bits of code found in various mythical gists, which is less than optimal."

Comment 1

[Rob Campbell [:rc] (:robcee)]

maybe a gcli command? This is a pretty niche feature for, as you say, ninjas.

Comment 2

[Rob Campbell [:rc] (:robcee)]

moving back to debugger based on Panos' feelings that Joe would like commands to live in their respective components. Sheesh!

Comment 3

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

Joe showed us something along these lines earlier today, so I guess it's his bug now!

Comment 4

[Paul Rouget [:paul]]

This is needed for our styleeditor work as well

Comment 5

[Girish Sharma [:Optimizer]]

Can the UI sit in the new options panel that is coming up in bug 851546 ?
If this lands first, I can import it in the Connect tab of the options panel, sitting just next to the connect screen. Once the user starts the debugger, we can also tell him the port and host on which the server is running.

It would be even simpler (to port to options panel) to do this if the UI for starting debugger server sits in the connect screen itself. We can have something like:


 _____________________________________________________________________________
|                                                                             |
|                                   |                                         |
|  Connect to remote device         |    Start the debugger on this machine   |
|            -------------          |    so that remote clients can connect   |
|    Host : |             |         |                                         |
|            -------------          |                                         |
|            -------------         or        --------------                   |
|    Port : |             |         |       | Start Server |                  |
|            -------------          |        --------------                   |
|            ---------              |                                         |
|           | Connect |             |                                         |
|            ---------              |                                         |
|                                   |                                         |
|_____________________________________________________________________________|

Comment 6

[Paul Rouget [:paul]]

I like Optimizer's idea. But a simple command could be a good first step.

> listen 6000

Comment 7

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

(In reply to Paul Rouget [:paul] from comment #6)
> I like Optimizer's idea.

Me too.

Comment 8

[Joe Walker [:jwalker] (needinfo me or ping on irc)]

(In reply to Panos Astithas [:past] from comment #7)
> (In reply to Paul Rouget [:paul] from comment #6)
> > I like Optimizer's idea.
> 
> Me too.

Lets discuss the options dialog in it's bug, and leave this bug for the command.
OK?

Comment 9

[Dave Camp (:dcamp)]

(In reply to Joe Walker [:jwalker] from comment #8)

> Lets discuss the options dialog in it's bug, and leave this bug for the
> command.
> OK?

Specializing the subject accordingly.

Comment 10

[Joe Walker [:jwalker] (needinfo me or ping on irc)]

Attachment: wip1

Comment 11

[Joe Walker [:jwalker] (needinfo me or ping on irc)]

Bug 839862 contains a 'listen' command.