Forward --no-sandbox flag, if present, to LoadAndLaunch platform app tests.

apps/load_and_launch_browsertest.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Tests for the --load-and-launch-app switch.
 // The two cases are when chrome is running and another process uses the switch
 // and when chrome is started from scratch.
 
 #include "apps/switches.h"
 #include "base/process/launch.h"
 #include "base/test/test_timeouts.h"
 #include "chrome/browser/apps/app_browsertest_util.h"
 #include "chrome/browser/extensions/extension_browsertest.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/common/chrome_switches.h"
+#include "content/public/common/content_switches.h"
 #include "content/public/test/test_launcher.h"
 #include "extensions/test/extension_test_message_listener.h"
 
 using extensions::PlatformAppBrowserTest;
 
 namespace apps {
 
+namespace {
+
+const char* kSwitchesToCopy[] = {
+    switches::kUserDataDir,
+    switches::kNoSandbox,
+};
+
+}  // namespace
+
 // TODO(jackhou): Enable this test once it works on OSX. It currently does not
 // work for the same reason --app-id doesn't. See http://crbug.com/148465
 #if defined(OS_MACOSX)
 #define MAYBE_LoadAndLaunchAppChromeRunning \
         DISABLED_LoadAndLaunchAppChromeRunning
 #else
 #define MAYBE_LoadAndLaunchAppChromeRunning LoadAndLaunchAppChromeRunning
 #endif
 
 // Case where Chrome is already running.
 IN_PROC_BROWSER_TEST_F(PlatformAppBrowserTest,
                        MAYBE_LoadAndLaunchAppChromeRunning) {
   ExtensionTestMessageListener launched_listener("Launched", false);
 
   const base::CommandLine& cmdline = *base::CommandLine::ForCurrentProcess();
   base::CommandLine new_cmdline(cmdline.GetProgram());
-
-  const char* kSwitchNames[] = {
-    switches::kUserDataDir,
-  };
-  new_cmdline.CopySwitchesFrom(cmdline, kSwitchNames, arraysize(kSwitchNames));
+  new_cmdline.CopySwitchesFrom(cmdline, kSwitchesToCopy,
+                               arraysize(kSwitchesToCopy));
 
   base::FilePath app_path = test_data_dir_
       .AppendASCII("platform_apps")
       .AppendASCII("minimal");
 
   new_cmdline.AppendSwitchNative(apps::kLoadAndLaunchApp,
                                  app_path.value());
 
   new_cmdline.AppendSwitch(content::kLaunchAsBrowser);
   base::Process process =
@@ skipping 12 matching lines @@
 #else
 #define MAYBE_LoadAndLaunchAppWithFile LoadAndLaunchAppWithFile
 #endif
 
 IN_PROC_BROWSER_TEST_F(PlatformAppBrowserTest,
                        MAYBE_LoadAndLaunchAppWithFile) {
   ExtensionTestMessageListener launched_listener("Launched", false);
 
   const base::CommandLine& cmdline = *base::CommandLine::ForCurrentProcess();
   base::CommandLine new_cmdline(cmdline.GetProgram());
-
-  const char* kSwitchNames[] = {
-    switches::kUserDataDir,
-  };
-  new_cmdline.CopySwitchesFrom(cmdline, kSwitchNames, arraysize(kSwitchNames));
+  new_cmdline.CopySwitchesFrom(cmdline, kSwitchesToCopy,
+                               arraysize(kSwitchesToCopy));
 
   base::FilePath app_path = test_data_dir_
       .AppendASCII("platform_apps")
       .AppendASCII("load_and_launch_file");
 
   base::FilePath test_file_path = test_data_dir_
       .AppendASCII("platform_apps")
       .AppendASCII("launch_files")
       .AppendASCII("test.txt");
 
@@ skipping 55 matching lines @@
         LoadAndLaunchAppChromeNotRunning
 #endif
 
 // Case where Chrome is not running.
 IN_PROC_BROWSER_TEST_F(PlatformAppLoadAndLaunchBrowserTest,
                        MAYBE_LoadAndLaunchAppChromeNotRunning) {
   LoadAndLaunchApp();
 }
 
 }  // namespace apps