Stop using third_party\python_26 for many tests.

net/test/spawned_test_server/local_test_server.cc

 // Copyright 2013 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/spawned_test_server/local_test_server.h"
 
 #include "base/command_line.h"
 #include "base/json/json_reader.h"
 #include "base/logging.h"
 #include "base/path_service.h"
@@ skipping 108 matching lines @@
 }
 
 bool LocalTestServer::Stop() {
   CleanUpWhenStoppingServer();
 
   if (!process_handle_)
     return true;
 
   // First check if the process has already terminated.
   bool ret = base::WaitForSingleProcess(process_handle_, base::TimeDelta());
-  if (!ret)
+  if (!ret) {
+#if defined(OS_WIN)

[Paweł Hajdan Jr., 2013/12/02 19:37:56]

Let's close the job handle always when returning from this method, not just when
WaitForSingleProcess fails.

[M-A Ruel, 2013/12/02 20:13:16]

and removed stale comment at lines 136-137.

+    job_handle_.Close();
+#endif
     ret = base::KillProcess(process_handle_, 1, true);
+  }
 
+  // TODO(maruel): This doesn't work on Windows because it kills cmd.exe
+  // instead of python.exe. http://crbug.com/321703
   if (ret) {
     base::CloseProcessHandle(process_handle_);
     process_handle_ = base::kNullProcessHandle;
   } else {
     VLOG(1) << "Kill failed?";
   }
 
   return ret;
 }
 
@@ skipping 101 matching lines @@
       break;
     default:
       NOTREACHED();
       return false;
   }
 
   return true;
 }
 
 }  // namespace net