Revert of Simplify mojo_shell since it's now only used for Mandoline.

Revert of Simplify mojo_shell since it's now only used for Mandoline. (patchset #6 id:100001 of https://codereview.chromium.org/1057603003/)

Reason for revert:
This CL causes problems for application testing:
1) Removing url-mappings breaks the ability to mock out service dependencies of test applications.
2) We want to connect directly to the test applications specified on the command line, not mojo:window_manager for apptests.
3) The apptest_runner.py currently uses --args-for, but that doesn't matter so much...
I've started working around these issues in WIP a CL, <https://codereview.chromium.org/1070463003/>;, but it would be better to not let this CL bake if we go intend to revert some of its behavior changes.

Original issue's description:
> Simplify mojo_shell since it's now only used for Mandoline.
>
> -no need to map kiosk_wm to window_manager, instead, make kiosk_wm's binary output be window_manager.mojom
> -no need for app-specific args, instead use base::CommandLine. This is important for Mandoline since we'll be sharing code with Chrome which does use these flags.
> -mojo_shell url is enough to start a browser now
>
> Committed: https://crrev.com/143eb32f63bd929f7d730970e2477804f318a7eb
> Cr-Commit-Position: refs/heads/master@{#324579}

TBR=ben@chromium.org,jam@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Committed: https://crrev.com/f1d1609fce75c570455aaa526f6023ea97a80c0d
Cr-Commit-Position: refs/heads/master@{#324712}

[msw, 2015-04-10 22:45:51]

Created Revert of Simplify mojo_shell since it's now only used for Mandoline.

[commit-bot: I haz the power, 2015-04-10 22:46:53]

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

[commit-bot: I haz the power, 2015-04-10 22:51:59]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-04-10 22:52:00]

Failed to apply patch for mojo/shell/BUILD.gn:
While running git apply --index -3 -p1;
  error: patch failed: mojo/shell/BUILD.gn:361
  Falling back to three-way merge...
  Applied patch to 'mojo/shell/BUILD.gn' with conflicts.
  U mojo/shell/BUILD.gn

Patch:       mojo/shell/BUILD.gn
Index: mojo/shell/BUILD.gn
diff --git a/mojo/shell/BUILD.gn b/mojo/shell/BUILD.gn
index
e83912ca10fd1bb357d549b88c6650fa919e5341..1dbe6ad77363bd85d31e7590b2fba3330862c4c5
100644
--- a/mojo/shell/BUILD.gn
+++ b/mojo/shell/BUILD.gn
@@ -114,6 +114,8 @@
     "child_process.h",
     "child_process_host.cc",
     "child_process_host.h",
+    "command_line_util.cc",
+    "command_line_util.h",
     "context.cc",
     "context.h",
     "filename_util.cc",
@@ -361,6 +363,7 @@
 test("mojo_shell_tests") {
   sources = [
     "app_child_process_host_unittest.cc",
+    "command_line_util_unittest.cc",
     "data_pipe_peek_unittest.cc",
     "in_process_native_runner_unittest.cc",
     "native_runner_unittest.cc",

[commit-bot: I haz the power, 2015-04-10 22:54:42]

The CQ bit was unchecked by commit-bot@chromium.org

[msw, 2015-04-10 22:55:08]

The CQ bit was checked by msw@chromium.org

[commit-bot: I haz the power, 2015-04-10 22:57:38]

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

[commit-bot: I haz the power, 2015-04-10 22:59:54]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-04-10 22:59:55]

Failed to apply patch for mojo/shell/BUILD.gn:
While running git apply --index -3 -p1;
  error: patch failed: mojo/shell/BUILD.gn:361
  Falling back to three-way merge...
  Applied patch to 'mojo/shell/BUILD.gn' with conflicts.
  U mojo/shell/BUILD.gn

Patch:       mojo/shell/BUILD.gn
Index: mojo/shell/BUILD.gn
diff --git a/mojo/shell/BUILD.gn b/mojo/shell/BUILD.gn
index
e83912ca10fd1bb357d549b88c6650fa919e5341..1dbe6ad77363bd85d31e7590b2fba3330862c4c5
100644
--- a/mojo/shell/BUILD.gn
+++ b/mojo/shell/BUILD.gn
@@ -114,6 +114,8 @@ source_set("lib") {
     "child_process.h",
     "child_process_host.cc",
     "child_process_host.h",
+    "command_line_util.cc",
+    "command_line_util.h",
     "context.cc",
     "context.h",
     "filename_util.cc",
@@ -361,6 +363,7 @@ mojom("app_child_process_bindings") {
 test("mojo_shell_tests") {
   sources = [
     "app_child_process_host_unittest.cc",
+    "command_line_util_unittest.cc",
     "data_pipe_peek_unittest.cc",
     "in_process_native_runner_unittest.cc",
     "native_runner_unittest.cc",

[commit-bot: I haz the power, 2015-04-10 23:02:19]

The CQ bit was unchecked by commit-bot@chromium.org

[msw, 2015-04-10 23:40:50]

The CQ bit was checked by msw@chromium.org

[commit-bot: I haz the power, 2015-04-10 23:42:22]

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

[commit-bot: I haz the power, 2015-04-10 23:44:38]

Committed patchset #3 (id:310001)

[commit-bot: I haz the power, 2015-04-10 23:45:29]

Patchset 3 (id:??) landed as
https://crrev.com/f1d1609fce75c570455aaa526f6023ea97a80c0d
Cr-Commit-Position: refs/heads/master@{#324712}

[jam, 2015-04-13 14:53:59]

Regarding 1: I was thinking that tests could use
Context::url_resolver()->AddURLMapping() method to setup whatever mapping they
need; i.e. it's not clear to me why we need this generic command-line method as
well.

2: I'm not sure what this means, can you expand?