Don't launch service process if it's already running

chrome/browser/service/service_process_control_browsertest.cc

 // Copyright (c) 2010 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.
 
 #include "chrome/test/in_process_browser_test.h"
 #include "chrome/test/ui_test_utils.h"
 #include "chrome/browser/browser.h"
 #include "chrome/browser/service/service_process_control.h"
 #include "chrome/browser/service/service_process_control_manager.h"
 #include "chrome/common/service_process_type.h"
@@ skipping 47 matching lines @@
 IN_PROC_BROWSER_TEST_F(ServiceProcessControlBrowserTest, LaunchAndIPC) {
   LaunchServiceProcessControl();
 
   // Make sure we are connected to the service process.
   EXPECT_TRUE(process()->is_connected());
   SayHelloAndWait();
 
   // And then shutdown the service process.
   EXPECT_TRUE(process()->Shutdown());
 }
+
+// This tests the case when a service process is launched when browser
+// starts but we try to launch it again in the remoting setup dialog.
+// TODO(hclam): We actually need to implement singleton in the service
+// process so that even if we launch the service process twice one
+// of the them will shutdown itself and we are still able to connect
+// to the first one that gets executed.
+IN_PROC_BROWSER_TEST_F(ServiceProcessControlBrowserTest, LaunchTwice) {
+  // Launch the service process the first time.
+  LaunchServiceProcessControl();
+
+  // Make sure we are connected to the service process.
+  EXPECT_TRUE(process()->is_connected());
+  SayHelloAndWait();
+
+  // Launch the service process again.
+  LaunchServiceProcessControl();
+  EXPECT_TRUE(process()->is_connected());
+  SayHelloAndWait();
+
+  // And then shutdown the service process.
+  EXPECT_TRUE(process()->Shutdown());
+}
 #endif
 
 DISABLE_RUNNABLE_METHOD_REFCOUNT(ServiceProcessControlBrowserTest);