[Chromoting] Reimplemented the worker process launcher to take into account the encountered issues:

remoting/host/win/launch_process_with_token.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 "remoting/host/win/launch_process_with_token.h"
 
 #include <windows.h>
 #include <winternl.h>
 
 #include "base/logging.h"
@@ skipping 14 matching lines @@
 
 // Undocumented WINSTATIONINFOCLASS value causing
 // winsta!WinStationQueryInformationW() to return the name of the pipe for
 // requesting cross-session process creation.
 const WINSTATIONINFOCLASS kCreateProcessPipeNameClass =
     static_cast<WINSTATIONINFOCLASS>(0x21);
 
 const int kPipeBusyWaitTimeoutMs = 2000;
 const int kPipeConnectMaxAttempts = 3;
 
-// The minimum and maximum delays between attempts to inject host process into
-// a session.
-const int kMaxLaunchDelaySeconds = 60;
-const int kMinLaunchDelaySeconds = 1;
-
 // Name of the default session desktop.
 const char kDefaultDesktopName[] = "winsta0\\default";
 
 // Copies the process token making it a primary impersonation token.
 // The returned handle will have |desired_access| rights.
 bool CopyProcessToken(DWORD desired_access, ScopedHandle* token_out) {
   ScopedHandle process_token;
   if (!OpenProcessToken(GetCurrentProcess(),
                         TOKEN_DUPLICATE | desired_access,
                         process_token.Receive())) {
@@ skipping 235 matching lines @@
         PROCESS_SUSPEND_RESUME;
     response.process_information.hProcess =
         OpenProcess(desired_access,
                     FALSE,
                     response.process_information.dwProcessId);
     if (!response.process_information.hProcess) {
       LOG_GETLASTERROR(ERROR) << "Failed to open process "
                               << response.process_information.dwProcessId;
       return false;
     }
+
+    // N.B. THREAD_ALL_ACCESS is different in XP and Vista+ versions of
+    // the SDK. |desired_access| below is effectively THREAD_ALL_ACCESS from
+    // the XP version of the SDK.
+    desired_access =
+        STANDARD_RIGHTS_REQUIRED |
+        SYNCHRONIZE |
+        THREAD_TERMINATE |
+        THREAD_SUSPEND_RESUME |
+        THREAD_GET_CONTEXT |
+        THREAD_SET_CONTEXT |
+        THREAD_QUERY_INFORMATION |
+        THREAD_SET_INFORMATION |
+        THREAD_SET_THREAD_TOKEN |
+        THREAD_IMPERSONATE |
+        THREAD_DIRECT_IMPERSONATION;
+    response.process_information.hThread =
+        OpenThread(desired_access,
+                   FALSE,
+                   response.process_information.dwThreadId);
+    if (!response.process_information.hThread) {
+      LOG_GETLASTERROR(ERROR) << "Failed to open thread "
+                              << response.process_information.dwThreadId;
+      CloseHandle(response.process_information.hProcess);
+      return false;
+    }
   }
 
   *process_information_out = response.process_information;
   return true;
 }
 
 } // namespace
 
 namespace remoting {
 
@@ skipping 103 matching lines @@
     return false;
   }
 
   CHECK(process_info.IsValid());
   process_out->Set(process_info.TakeProcessHandle());
   thread_out->Set(process_info.TakeThreadHandle());
   return true;
 }
 
 } // namespace remoting