Wait on a pipe for the test server to start up

net/test/test_server_win.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 "net/test/test_server.h"
 
 #include <windows.h>
 #include <wincrypt.h>
 
 #include "base/base_paths.h"
@@ skipping 16 matching lines @@
   startup_info.cb = sizeof(startup_info);
   startup_info.dwFlags = STARTF_USESHOWWINDOW;
   startup_info.wShowWindow = start_hidden ? SW_HIDE : SW_SHOW;
   PROCESS_INFORMATION process_info;
 
   // If this code is run under a debugger, the test server process is
   // automatically associated with a job object created by the debugger.
   // The CREATE_BREAKAWAY_FROM_JOB flag is used to prevent this.
   if (!CreateProcess(NULL,
                      const_cast<wchar_t*>(cmdline.c_str()), NULL, NULL,
-                     FALSE, CREATE_BREAKAWAY_FROM_JOB, NULL, NULL,
+                     TRUE, CREATE_BREAKAWAY_FROM_JOB, NULL, NULL,
                      &startup_info, &process_info)) {
     LOG(ERROR) << "Could not create process.";
     return false;
   }
   CloseHandle(process_info.hThread);
 
   // If the caller wants the process handle, we won't close it.
   if (process_handle) {
     *process_handle = process_info.hProcess;
   } else {
@@ skipping 52 matching lines @@
       return false;
     }
     command_line.append(L" --https=\"");
     command_line.append(certificate_path.value());
     command_line.append(L"\"");
   }
 
   if (type_ == TYPE_HTTPS_CLIENT_AUTH)
     command_line.append(L" --ssl-client-auth");
 
+  HANDLE child_read = NULL;
+  HANDLE child_write = NULL;
+  if (!CreatePipe(&child_read, &child_write, NULL, 0)) {
+    PLOG(ERROR) << "Failed to create pipe";
+    return false;
+  }
+  child_fd_.Set(child_read);
+  ScopedHandle scoped_child_write(child_write);
+
+  // Have the child inherit the write half.
+  if (!SetHandleInformation(child_write, HANDLE_FLAG_INHERIT,
+                            HANDLE_FLAG_INHERIT)) {
+    PLOG(ERROR) << "Failed to enable pipe inheritance";
+    return false;
+  }
+
+  // Pass the handle on the command-line. Although HANDLE is a
+  // pointer, truncating it on 64-bit machines is okay. See
+  // http://msdn.microsoft.com/en-us/library/aa384203.aspx
+  //
+  // "64-bit versions of Windows use 32-bit handles for
+  // interoperability. When sharing a handle between 32-bit and 64-bit
+  // applications, only the lower 32 bits are significant, so it is
+  // safe to truncate the handle (when passing it from 64-bit to
+  // 32-bit) or sign-extend the handle (when passing it from 32-bit to
+  // 64-bit)."
+  command_line.append(
+      L" --startup-pipe=" +
+      ASCIIToWide(base::IntToString(reinterpret_cast<uintptr_t>(child_write))));
+
   if (!LaunchTestServerAsJob(command_line,
                              true,
                              &process_handle_,
                              &job_handle_)) {
     LOG(ERROR) << "Failed to launch " << command_line;
     return false;
   }
 
   return true;
 }
 
+bool TestServer::WaitToStart() {
+  char buf[8];
+  DWORD bytes_read;
+  BOOL result = ReadFile(child_fd_, buf, sizeof(buf), &bytes_read, NULL);
+  child_fd_.Close();
+  return result && bytes_read > 0;
+}
+
 bool TestServer::CheckCATrusted() {
   HCERTSTORE cert_store = CertOpenSystemStore(NULL, L"ROOT");
   if (!cert_store) {
     LOG(ERROR) << " could not open trusted root CA store";
     return false;
   }
   PCCERT_CONTEXT cert =
       CertFindCertificateInStore(cert_store,
                                  X509_ASN_ENCODING | PKCS_7_ASN_ENCODING,
                                  0,
                                  CERT_FIND_ISSUER_STR,
                                  L"Test CA",
                                  NULL);
   if (cert)
     CertFreeCertificateContext(cert);
   CertCloseStore(cert_store, 0);
 
   if (!cert) {
     LOG(ERROR) << " TEST CONFIGURATION ERROR: you need to import the test ca "
                   "certificate to your trusted roots for this test to work. "
                   "For more info visit:\n"
                   "http://dev.chromium.org/developers/testing\n";
     return false;
   }
 
   return true;
 }
 
 }  // namespace net