199566 - Add -inspector command-line param

Status: RESOLVED FIXED

(Other Applications :: DOM Inspector, enhancement)

Description

[Brian 'netdragon' Bober]

"mozilla -venkman" exists. "mozilla -inspector" would be nice

"mozilla chrome://inspector/content/inspector.xul" is close as a workaround.

Comment 1

[Brian 'netdragon' Bober]

dbaron mentioned that "mozilla -chrome 
chrome://inspector/content/inspector.xul" gives it its own window. An even 
better workaround.

Comment 2

[timeless]

.

Comment 3

[timeless]

.

Comment 4

[Christopher Aillon (sabbatical, not receiving bugmail)]

I thought we already had this (based on the fact that the documentation hewitt
wrote says we do).

I temporarily commented that out, but I have this working in my tree.  I will
probably upload a patch tomorrow sometime once I make it play nicely with xpis.

Comment 5

[Christopher Aillon (sabbatical, not receiving bugmail)]

Attachment: Command Handler

Should do the trick.

Comment 6

[Christopher Aillon (sabbatical, not receiving bugmail)]

Comment on attachment 123595 [details] [diff] [review]
Command Handler

Seeking r+sr, but feel free to defer on one of them if you are so inclined.

Comment 7

[Boris Zbarsky [:bzbarsky]]

I don't really know much about any of the code involved, so I'm not sure I'm a
good choice to r or sr this.  In any case, I will not be able to get to it for a
while (possibly till mid-June).

Comment 8

[Christopher Aillon (sabbatical, not receiving bugmail)]

Comment on attachment 123595 [details] [diff] [review]
Command Handler

Okay, I might have misunderstood our irc conversation.	I thought you knew
about this code and wanted to review it.  No matter.  I have a new patch
forthcoming which will also allow users to specify a URL to inspect.

Comment 9

[Christopher Aillon (sabbatical, not receiving bugmail)]

Attachment: Also allow uris to be passed in

./mozilla -inspector <url>
works with this patch

Comment 10

[timeless]

Comment on attachment 123710 [details] [diff] [review]
Also allow uris to be passed in

mac script should reference Viewer not bin
(as discussed on irc)

Comment 11

[Johnny Stenback (:jst)]

Comment on attachment 123710 [details] [diff] [review]
Also allow uris to be passed in

- In inspector-cmdline.js:

+  createInstance : function  (outer, iid)

I see "function(...)", "function (...)", and "function	(...)" in this file.
Pick one, and stick with it (I'd suggest no spaces).

sr=jst with that (and other comments in this bug) addressed.

Comment 12

[Christopher Aillon (sabbatical, not receiving bugmail)]

Comment on attachment 123710 [details] [diff] [review]
Also allow uris to be passed in

Low risk Inspector only patch: give it its own cmdline handler.

Comment 13

[Asa Dotzler [:asa]]

Comment on attachment 123710 [details] [diff] [review]
Also allow uris to be passed in

a=asa (on behalf of drivers) for checkin to 1.4

Comment 14

[Christopher Aillon (sabbatical, not receiving bugmail)]

Checked in