[content shell] add support for getting tests from the command line.

[content shell] add support for getting tests from the command line.

Related to this, parse a single dash on the command line as argument, instead of a switch with no value and name

BUG=111316
TEST=out/Debug/content_shell --dump-render-tree path/to/test.html works


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=162947

[jochen (gone - plz use gerrit), 2012-10-18 20:45:22]

plz review

Brett: base/command_line*
Marja: content/shell/*

[marja, 2012-10-18 20:53:57]

http://codereview.chromium.org/11184050/diff/1/base/command_line.cc
File base/command_line.cc (right):

http://codereview.chromium.org/11184050/diff/1/base/command_line.cc#newcode54
base/command_line.cc:54: if (prefix_length  == 0 || prefix_length ==
string.length())
nit: extra space here

http://codereview.chromium.org/11184050/diff/1/content/shell/shell_browser_ma...
File content/shell/shell_browser_main.cc (right):

http://codereview.chromium.org/11184050/diff/1/content/shell/shell_browser_ma...
content/shell/shell_browser_main.cc:67: static size_t pos = 0;
Uhh, can you make pos an in-out parameter instead.

[jochen (gone - plz use gerrit), 2012-10-18 21:06:42]

http://codereview.chromium.org/11184050/diff/1/base/command_line.cc
File base/command_line.cc (right):

http://codereview.chromium.org/11184050/diff/1/base/command_line.cc#newcode54
base/command_line.cc:54: if (prefix_length  == 0 || prefix_length ==
string.length())
On 2012/10/18 20:53:57, marja wrote:
> nit: extra space here

Done.

http://codereview.chromium.org/11184050/diff/1/content/shell/shell_browser_ma...
File content/shell/shell_browser_main.cc (right):

http://codereview.chromium.org/11184050/diff/1/content/shell/shell_browser_ma...
content/shell/shell_browser_main.cc:67: static size_t pos = 0;
On 2012/10/18 20:53:57, marja wrote:
> Uhh, can you make pos an in-out parameter instead.

Done.

[marja, 2012-10-18 21:12:49]

thx, lgtm.

[jochen (gone - plz use gerrit), 2012-10-18 23:12:08]

-brettw
+jar

Jim, can you plz review base/command_line* ?

[jar (doing other things), 2012-10-19 02:30:03]

lgtm

[commit-bot: I haz the power, 2012-10-19 05:17:53]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jochen@chromium.org/11184050/6

[commit-bot: I haz the power, 2012-10-19 06:20:01]

Change committed as 162947