Shell: Make a separate binary for child processes.

shell/desktop/main.cc

Index: shell/desktop/main.cc
diff --git a/shell/desktop/main.cc b/shell/desktop/main.cc
index cf264373e0248c6b7f5c79f1db5eaad2ff71741d..6c2ed7b97fbd62c35e3609f288415b6db2bec96a 100644
--- a/shell/desktop/main.cc
+++ b/shell/desktop/main.cc
@@ -17,7 +17,6 @@
 #include "base/message_loop/message_loop.h"
 #include "base/synchronization/waitable_event.h"
 #include "base/trace_event/trace_event.h"
-#include "shell/child_main.h"
 #include "shell/command_line_util.h"
 #include "shell/context.h"
 #include "shell/init.h"
@@ -113,87 +112,78 @@ int main(int argc, char** argv) {
 
   const base::CommandLine& command_line =
       *base::CommandLine::ForCurrentProcess();
-
-  // TODO(vtl): Unify parent and child process cases to the extent possible.
-  int exit_code = 0;
-  if (command_line.HasSwitch(switches::kChildProcess)) {
-    exit_code = mojo::shell::ChildMain();
-  } else {
-    // Only check the command line for the main process.
-    const std::set<std::string> all_switches = switches::GetAllSwitches();
-    const base::CommandLine::SwitchMap switches = command_line.GetSwitches();
-    bool found_unknown_switch = false;
-    for (const auto& s : switches) {
-      if (all_switches.find(s.first) == all_switches.end()) {
-        std::cerr << "unknown switch: " << s.first << std::endl;
-        found_unknown_switch = true;
-      }
+  CHECK(!command_line.HasSwitch(switches::kChildProcess));

[DaveMoore, 2015/04/07 22:27:30]

Seems like we can drop this too. If you're running the wrong shell / child it's
your own damn fault.

+
+  // Only check the command line for the main process.
+  const std::set<std::string> all_switches = switches::GetAllSwitches();
+  const base::CommandLine::SwitchMap switches = command_line.GetSwitches();
+  bool found_unknown_switch = false;
+  for (const auto& s : switches) {
+    if (all_switches.find(s.first) == all_switches.end()) {
+      std::cerr << "unknown switch: " << s.first << std::endl;
+      found_unknown_switch = true;
     }
+  }
 
-    if (found_unknown_switch ||
-        (!command_line.HasSwitch(switches::kEnableExternalApplications) &&
-         (command_line.HasSwitch(switches::kHelp) ||
-          command_line.GetArgs().empty()))) {
-      Usage();
-      return 0;
-    }
+  if (found_unknown_switch ||
+      (!command_line.HasSwitch(switches::kEnableExternalApplications) &&
+       (command_line.HasSwitch(switches::kHelp) ||
+        command_line.GetArgs().empty()))) {
+    Usage();
+    return 0;
+  }
 
-    if (command_line.HasSwitch(switches::kTraceStartup)) {
-      g_tracing = true;
-      base::trace_event::CategoryFilter category_filter(
-          command_line.GetSwitchValueASCII(switches::kTraceStartup));
-      base::trace_event::TraceLog::GetInstance()->SetEnabled(
-          category_filter, base::trace_event::TraceLog::RECORDING_MODE,
-          base::trace_event::TraceOptions(
-              base::trace_event::RECORD_UNTIL_FULL));
-    }
+  if (command_line.HasSwitch(switches::kTraceStartup)) {
+    g_tracing = true;
+    base::trace_event::CategoryFilter category_filter(
+        command_line.GetSwitchValueASCII(switches::kTraceStartup));
+    base::trace_event::TraceLog::GetInstance()->SetEnabled(
+        category_filter, base::trace_event::TraceLog::RECORDING_MODE,
+        base::trace_event::TraceOptions(base::trace_event::RECORD_UNTIL_FULL));
+  }
 
-    if (command_line.HasSwitch(switches::kCPUProfile)) {
+  if (command_line.HasSwitch(switches::kCPUProfile)) {
 #if !defined(NDEBUG) || !defined(ENABLE_PROFILING)
-      LOG(ERROR) << "Profiling requires is_debug=false and "
-                 << "enable_profiling=true. Continuing without profiling.";
+    LOG(ERROR) << "Profiling requires is_debug=false and "
+               << "enable_profiling=true. Continuing without profiling.";
 // StartProfiling() and StopProfiling() are a no-op.
 #endif
-      base::debug::StartProfiling("mojo_shell.pprof");
-    }
+    base::debug::StartProfiling("mojo_shell.pprof");
+  }
 
-    // We want the shell::Context to outlive the MessageLoop so that pipes are
-    // all gracefully closed / error-out before we try to shut the Context down.
-    mojo::shell::Context shell_context;
-    {
-      base::MessageLoop message_loop;
-      if (!shell_context.Init()) {
-        Usage();
-        return 0;
-      }
-      if (g_tracing) {
-        message_loop.PostDelayedTask(FROM_HERE,
-                                     base::Bind(StopTracingAndFlushToDisk),
-                                     base::TimeDelta::FromSeconds(5));
-      }
-
-      // The mojo_shell --args-for command-line switch is handled specially
-      // because it can appear more than once. The base::CommandLine class
-      // collapses multiple occurrences of the same switch.
-      for (int i = 1; i < argc; i++) {
-        ApplyApplicationArgs(&shell_context, argv[i]);
-      }
-
-      message_loop.PostTask(
-          FROM_HERE,
-          base::Bind(&mojo::shell::RunCommandLineApps, &shell_context));
-      message_loop.Run();
-
-      // Must be called before |message_loop| is destroyed.
-      shell_context.Shutdown();
+  // We want the shell::Context to outlive the MessageLoop so that pipes are all
+  // gracefully closed / error-out before we try to shut the Context down.
+  mojo::shell::Context shell_context;
+  {
+    base::MessageLoop message_loop;
+    if (!shell_context.Init()) {
+      Usage();
+      return 0;
     }
-
-    if (command_line.HasSwitch(switches::kCPUProfile)) {
-      base::debug::StopProfiling();
+    if (g_tracing) {
+      message_loop.PostDelayedTask(FROM_HERE,
+                                   base::Bind(StopTracingAndFlushToDisk),
+                                   base::TimeDelta::FromSeconds(5));
     }
+
+    // The mojo_shell --args-for command-line switch is handled specially
+    // because it can appear more than once. The base::CommandLine class
+    // collapses multiple occurrences of the same switch.
+    for (int i = 1; i < argc; i++)
+      ApplyApplicationArgs(&shell_context, argv[i]);
+
+    message_loop.PostTask(
+        FROM_HERE,
+        base::Bind(&mojo::shell::RunCommandLineApps, &shell_context));
+    message_loop.Run();
+
+    // Must be called before |message_loop| is destroyed.
+    shell_context.Shutdown();
   }
 
+  if (command_line.HasSwitch(switches::kCPUProfile))
+    base::debug::StopProfiling();
   if (g_tracing)
     StopTracingAndFlushToDisk();
-  return exit_code;
+  return 0;
 }