[Mojo] mojo_native_viewport_service should use "--" args

[Mojo] mojo_native_viewport_service should use "--" args

I expected the mojo_native_viewport_service to use -- as a prefix for its
arguments:

mojo_shell --args-for="mojo://mojo_native_viewport_service/ --use-headless-config"

but prior to this CL, it was expect its arguments without the leading "--".
After this CL, it expects the leading "--".

R=davemoore@chromium.org,hansmuller@chromium.org

Committed: https://crrev.com/5acd9fe37eae1ff654eb195960d30be0fa55f03f
Cr-Commit-Position: refs/heads/master@{#299810}

[hansmuller, 2014-10-15 20:07:03]

On 2014/10/15 16:56:46, abarth wrote:

This change LGTM. Perhaps native_viewport/main.cc should create a
base::CommandLine from application's args. Then using the constants as-is makes
sense.

[DaveMoore, 2014-10-15 20:51:52]

On 2014/10/15 20:07:03, hansmuller wrote:
> On 2014/10/15 16:56:46, abarth wrote:
> 
> This change LGTM. Perhaps native_viewport/main.cc should create a
> base::CommandLine from application's args. Then using the constants as-is
makes
> sense.

I considered using CommandLine as you describe (In fact it was what I did
initially). But it complicates using the constants in the ApplicationManager
api, which takes a vector of strings so I didn't. Having the -- as part of the
arg wasn't something I thought of, because it's different from switches but as
these are "args" I think it's fine.

LGTM.

[abarth-chromium, 2014-10-16 00:39:46]

The CQ bit was checked by abarth@chromium.org

[commit-bot: I haz the power, 2014-10-16 00:42:48]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/661513002/1

[commit-bot: I haz the power, 2014-10-16 01:39:17]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2014-10-16 01:39:46]

Patchset 1 (id:??) landed as
https://crrev.com/5acd9fe37eae1ff654eb195960d30be0fa55f03f
Cr-Commit-Position: refs/heads/master@{#299810}