Pass command line arguments through with --load-and-launch-app.

Pass command line arguments through with --load-and-launch-app.

This lets uninstalled unpacked apps be started via this flag and passed
files to process.

BUG=246510
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=205503

[koz (OOO until 15th September), 2013-06-05 08:34:11]

lgtm

https://codereview.chromium.org/16357005/diff/5001/apps/app_load_service.h
File apps/app_load_service.h (right):

https://codereview.chromium.org/16357005/diff/5001/apps/app_load_service.h#ne...
apps/app_load_service.h:27: RELOAD_LAUNCH,
nit: I think it's fine to remove the RELOAD_ prefix here, but up to you. Or if
you're into the whole verbosity thing the prefix could be POST_RELOAD_LAUNCH.
Just that the current names are sort of weird - RELOAD_RESTART has two words
that are alternatives to each other.

https://codereview.chromium.org/16357005/diff/5001/apps/app_load_service.h#ne...
apps/app_load_service.h:65: std::map<std::string, PostReloadAction>
reload_actions_;
s/reload_actions_/post_reload_actions_/ ?

[benwells, 2013-06-06 00:22:42]

+sky for startup_browser_creator

https://codereview.chromium.org/16357005/diff/5001/apps/app_load_service.h
File apps/app_load_service.h (right):

https://codereview.chromium.org/16357005/diff/5001/apps/app_load_service.h#ne...
apps/app_load_service.h:27: RELOAD_LAUNCH,
On 2013/06/05 08:34:11, koz wrote:
> nit: I think it's fine to remove the RELOAD_ prefix here, but up to you. Or if
> you're into the whole verbosity thing the prefix could be POST_RELOAD_LAUNCH.
> Just that the current names are sort of weird - RELOAD_RESTART has two words
> that are alternatives to each other.

Done.

https://codereview.chromium.org/16357005/diff/5001/apps/app_load_service.h#ne...
apps/app_load_service.h:65: std::map<std::string, PostReloadAction>
reload_actions_;
On 2013/06/05 08:34:11, koz wrote:
> s/reload_actions_/post_reload_actions_/ ?

Done.

[sky, 2013-06-06 16:06:42]

LGTM

[commit-bot: I haz the power, 2013-06-07 00:48:25]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/benwells@chromium.org/16357005/14001

[commit-bot: I haz the power, 2013-06-07 02:17:49]

Retried try job too often on linux_chromeos for step(s) interactive_ui_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_chro...

[commit-bot: I haz the power, 2013-06-07 04:36:56]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/benwells@chromium.org/16357005/14001

[commit-bot: I haz the power, 2013-06-07 06:20:17]

Sorry for I got bad news for ya.
Compile failed with a clobber build on win_x64_rel.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_x64_re...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2013-06-11 04:50:17]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/benwells@chromium.org/16357005/14001

[commit-bot: I haz the power, 2013-06-11 12:20:24]

Change committed as 205503