Don't use start command when starting native messaging hosts.

chrome/browser/extensions/api/messaging/native_process_launcher_win.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/browser/extensions/api/messaging/native_process_launcher.h"
 
 #include <windows.h>
 
 #include "base/command_line.h"
 #include "base/logging.h"
@@ skipping 110 matching lines @@
   DWORD comspec_length = ::GetEnvironmentVariable(L"COMSPEC", NULL, 0);
   if (comspec_length == 0) {
     LOG(ERROR) << "COMSPEC is not set";
     return false;
   }
   scoped_ptr<wchar_t[]> comspec(new wchar_t[comspec_length]);
   ::GetEnvironmentVariable(L"COMSPEC", comspec.get(), comspec_length);
 
   string16 command_line_string = command_line.GetCommandLineString();
 
-  // 'start' command has a moronic syntax: if first argument is quoted then it
-  // interprets it as a command title. Host path may need to be in quotes, so
-  // we always need to specify the title as the first argument.
   string16 command = base::StringPrintf(
-      L"%ls /c start \"Chrome Native Messaging Host\" /b "
-      L"%ls < %ls > %ls",
+      L"%ls /c %ls < %ls > %ls",
       comspec.get(), command_line_string.c_str(),
       in_pipe_name.c_str(), out_pipe_name.c_str());
 
   base::LaunchOptions options;
   options.start_hidden = true;
   base::ProcessHandle cmd_handle;
   if (!base::LaunchProcess(command.c_str(), options, &cmd_handle)) {
     LOG(ERROR) << "Error launching process "
                << command_line.GetProgram().MaybeAsASCII();
     return false;
   }
 
   bool stdout_connected = ConnectNamedPipe(stdout_pipe.Get(), NULL) ?
       TRUE : GetLastError() == ERROR_PIPE_CONNECTED;
   bool stdin_connected = ConnectNamedPipe(stdin_pipe.Get(), NULL) ?
       TRUE : GetLastError() == ERROR_PIPE_CONNECTED;
   if (!stdout_connected || !stdin_connected) {
     base::KillProcess(cmd_handle, 0, false);
     base::CloseProcessHandle(cmd_handle);
     LOG(ERROR) << "Failed to connect IO pipes when starting "
                << command_line.GetProgram().MaybeAsASCII();
     return false;
   }
 
-  // Check that cmd.exe has completed with 0 exit code to make sure it was
-  // able to connect IO pipes.
-  int error_code;
-  if (!base::WaitForExitCodeWithTimeout(
-          cmd_handle, &error_code,
-          base::TimeDelta::FromMilliseconds(kTimeoutMs)) ||
-      error_code != 0) {
-    LOG(ERROR) << "cmd.exe did not exit cleanly";
-    base::KillProcess(cmd_handle, 0, false);
-    base::CloseProcessHandle(cmd_handle);
-    return false;
-  }
-
   base::CloseProcessHandle(cmd_handle);
 
   *read_file = stdout_pipe.Take();
   *write_file = stdin_pipe.Take();
 
   return true;
 }
 
 }  // namespace extensions