Enable multiprocess by default on desktop Mandoline.

This also turns on the new EDK by default (we see hangs with the old one) and disables the sandbox while a recent regression is tracked down.

BUG=478251,546644

Review URL: https://codereview.chromium.org/1406403004

Cr-Commit-Position: refs/heads/master@{#357360}

4 files changed, 9 insertions, 11 deletions

diff --git a/mojo/package_manager/package_manager_impl.cc b/mojo/package_manager/package_manager_impl.cc
index c1880c5..cb77e05 100644
--- a/mojo/package_manager/package_manager_impl.cc
+++ b/mojo/package_manager/package_manager_impl.cc
@@ -167,8 +167,8 @@ bool PackageManagerImpl::ShouldHandleWithContentHandler(
   // Why doesn't that happen when running different apps? Because
   // your_thing.mojo!base::AtExitManager and
   // my_thing.mojo!base::AtExitManager are different symbols.
-  bool use_real_qualifier = base::CommandLine::ForCurrentProcess()->HasSwitch(
-      switches::kEnableMultiprocess);
+  bool use_real_qualifier = !base::CommandLine::ForCurrentProcess()->HasSwitch(
+      switches::kMojoSingleProcess);
 
   GURL content_handler_url;
   // The response begins with a #!mojo <content-handler-url>.
diff --git a/mojo/runner/context.cc b/mojo/runner/context.cc
index e4819a7..a9af309 100644
--- a/mojo/runner/context.cc
+++ b/mojo/runner/context.cc
@@ -208,10 +208,10 @@ bool Context::Init() {
   RegisterLocalAliases(package_manager_);
 
   scoped_ptr<shell::NativeRunnerFactory> runner_factory;
-  if (command_line.HasSwitch(switches::kEnableMultiprocess))
-    runner_factory.reset(new OutOfProcessNativeRunnerFactory(this));
-  else
+  if (command_line.HasSwitch(switches::kMojoSingleProcess))
     runner_factory.reset(new InProcessNativeRunnerFactory(this));
+  else
+    runner_factory.reset(new OutOfProcessNativeRunnerFactory(this));
   application_manager_.reset(new shell::ApplicationManager(
       make_scoped_ptr(package_manager_), runner_factory.Pass(),
       task_runners_->blocking_pool()));
diff --git a/mojo/shell/switches.cc b/mojo/shell/switches.cc
index fafe057..d7c5e08 100644
--- a/mojo/shell/switches.cc
+++ b/mojo/shell/switches.cc
@@ -11,15 +11,13 @@ namespace switches {
 // If set apps downloaded are not deleted.
 const char kDontDeleteOnDownload[] = "dont-delete-on-download";
 
-// Load apps in separate processes.
-// TODO(vtl): Work in progress; doesn't work. Flip this to "disable" (or maybe
-// change it to "single-process") when it works.
-const char kEnableMultiprocess[] = "enable-multiprocess";
-
 // Disables the sandbox for debugging. (Why the Mojo prefix on the constant?
 // Because otherwise we conflict with content.)
 const char kMojoNoSandbox[] = "no-sandbox";
 
+// Load apps in a single processes.
+const char kMojoSingleProcess[] = "single-process";
+
 // If set apps downloaded are saved in with a predictable filename, to help
 // remote debugging: when gdb is used through gdbserver, it needs to be able to
 // find locally any loaded library. For this, gdb use the filename of the
diff --git a/mojo/shell/switches.h b/mojo/shell/switches.h
index 7ae5da1..9ed1f74 100644
--- a/mojo/shell/switches.h
+++ b/mojo/shell/switches.h
@@ -13,8 +13,8 @@ namespace switches {
 // All switches in alphabetical order. The switches should be documented
 // alongside the definition of their values in the .cc file.
 extern const char kDontDeleteOnDownload[];
-extern const char kEnableMultiprocess[];
 extern const char kMojoNoSandbox[];
+extern const char kMojoSingleProcess[];
 extern const char kPredictableAppFilenames[];
 extern const char kUseUpdater[];