304469 - Add -new-tab option to auto-remote

Status: RESOLVED FIXED

(Toolkit :: Startup and Profile System, defect)

Description

[Akkana Peck]

There's no longer a reliable way to open a url in a new tab (see bug 298960).
Bsmedberg said implementing a -new-tab commandline option, similar to the
existing -new-window option, wouldn't be hard at all, so I'm filing this to make
sure the request doesn't get lost.

Comment 1

[:Gavin Sharp [email: [e-mail]]]

Attachment: patch

Comment 2

[Akkana Peck]

Thanks! Works nicely here. I can even pass multiple -new-window and -new-tab
clauses in the same command. (All new-tabs are handled by the old window, maybe
because of the time it takes for the new window to appear. As far as I'm
concerned that behavior is Just Fine and it's probably what most people would
want anyway.)

Might it be worth putting both clauses in the same try { } clause? I don't think
it makes any difference functionally, really just a question of which style
benjamin prefers.

If you need another reviewer (or benjamin is too busy) I'd be happy to stamp it.

Comment 3

[:Gavin Sharp [email: [e-mail]]]

Checked in on trunk.

Checking in nsBrowserContentHandler.js;
/cvsroot/mozilla/browser/components/nsBrowserContentHandler.js,v  <-- 
nsBrowserContentHandler.js
new revision: 1.13; previous revision: 1.12
done

Comment 4

[:Gavin Sharp [email: [e-mail]]]

1.8 Branch:

Checking in nsBrowserContentHandler.js;
/cvsroot/mozilla/browser/components/nsBrowserContentHandler.js,v  <-- 
nsBrowserContentHandler.js
new revision: 1.12.2.1; previous revision: 1.12
done