Make PPAPI test servers start in parallel

chrome/test/ppapi/ppapi_test.cc

 // Copyright (c) 2012 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/ppapi/ppapi_test.h"
 
 #include "base/command_line.h"
 #include "base/file_util.h"
 #include "base/logging.h"
 #include "base/path_service.h"
@@ skipping 16 matching lines @@
 #include "content/public/browser/dom_operation_notification_details.h"
 #include "content/public/test/test_renderer_host.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/notification_types.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/common/content_paths.h"
 #include "content/public/common/content_switches.h"
 #include "content/public/test/browser_test_utils.h"
 #include "net/base/net_util.h"
 #include "net/base/test_data_directory.h"
-#include "net/test/test_server.h"
 #include "ppapi/shared_impl/ppapi_switches.h"
 #include "ui/gl/gl_switches.h"
 #include "webkit/plugins/plugin_switches.h"
 
 using content::DomOperationNotificationDetails;
 using content::RenderViewHost;
 
 namespace {
 
 // Platform-specific filename relative to the chrome executable.
@@ skipping 118 matching lines @@
 void PPAPITestBase::RunTestAndReload(const std::string& test_case) {
   GURL url = GetTestFileUrl(test_case);
   RunTestURL(url);
   // If that passed, we simply run the test again, which navigates again.
   RunTestURL(url);
 }
 
 void PPAPITestBase::RunTestViaHTTP(const std::string& test_case) {
   FilePath document_root;
   ASSERT_TRUE(ui_test_utils::GetRelativeBuildDirectory(&document_root));
-  RunHTTPTestServer(document_root, test_case, "");
+  RunHTTPTestServer(document_root, test_case, "", NULL);
 }
 
 void PPAPITestBase::RunTestWithSSLServer(const std::string& test_case) {
   FilePath document_root;
   ASSERT_TRUE(ui_test_utils::GetRelativeBuildDirectory(&document_root));
   net::TestServer test_server(net::TestServer::TYPE_HTTPS,
                               net::BaseTestServer::SSLOptions(),
                               document_root);
-  ASSERT_TRUE(test_server.Start());
+  ASSERT_TRUE(test_server.StartInBackground());
   uint16_t port = test_server.host_port_pair().port();
-  RunHTTPTestServer(document_root, test_case,
-                    StringPrintf("ssl_server_port=%d", port));
+  RunHTTPTestServer(document_root,
+                    test_case,
+                    StringPrintf("ssl_server_port=%d", port),
+                    &test_server);
 }
 
 void PPAPITestBase::RunTestWithWebSocketServer(const std::string& test_case) {
   net::TestServer server(net::TestServer::TYPE_WS,
                          net::TestServer::kLocalhost,
                          net::GetWebSocketTestDataDirectory());
-  ASSERT_TRUE(server.Start());
+  ASSERT_TRUE(server.StartInBackground());
   uint16_t port = server.host_port_pair().port();
   FilePath http_document_root;
   ASSERT_TRUE(ui_test_utils::GetRelativeBuildDirectory(&http_document_root));
-  RunHTTPTestServer(http_document_root, test_case,
-                    StringPrintf("websocket_port=%d", port));
+  RunHTTPTestServer(http_document_root,
+                    test_case,
+                    StringPrintf("websocket_port=%d", port),
+                    &server);
 }
 
 void PPAPITestBase::RunTestIfAudioOutputAvailable(
     const std::string& test_case) {
   RunTest(test_case);
 }
 
 void PPAPITestBase::RunTestViaHTTPIfAudioOutputAvailable(
     const std::string& test_case) {
   RunTestViaHTTP(test_case);
@@ skipping 21 matching lines @@
 
   ui_test_utils::NavigateToURL(browser(), test_url);
 
   ASSERT_TRUE(observer.Run()) << handler.error_message();
   EXPECT_STREQ("PASS", handler.message().c_str());
 }
 
 void PPAPITestBase::RunHTTPTestServer(
     const FilePath& document_root,
     const std::string& test_case,
-    const std::string& extra_params) {
+    const std::string& extra_params,
+    net::TestServer* extra_server) {
   net::TestServer test_server(net::TestServer::TYPE_HTTP,
                               net::TestServer::kLocalhost,
                               document_root);
-  ASSERT_TRUE(test_server.Start());
+  ASSERT_TRUE(test_server.StartInBackground());
+  if (extra_server)
+    ASSERT_TRUE(extra_server->BlockUntilStarted());
+  ASSERT_TRUE(test_server.BlockUntilStarted());
   std::string query = BuildQuery("files/test_case.html?", test_case);
   if (!extra_params.empty())
     query = StringPrintf("%s&%s", query.c_str(), extra_params.c_str());
 
   GURL url = test_server.GetURL(query);
   RunTestURL(url);
 }
 
 PPAPITest::PPAPITest() {
 }
@@ skipping 74 matching lines @@
     const std::string& base,
     const std::string& test_case) {
   return StringPrintf("%smode=nacl_newlib&testcase=%s", base.c_str(),
                       test_case.c_str());
 }
 
 void PPAPIBrokerInfoBarTest::SetUpOnMainThread() {
   // The default content setting for the PPAPI broker is ASK. We purposefully
   // don't call PPAPITestBase::SetUpOnMainThread() to keep it that way.
 }