Check for CloseHandle failures even when not debugging

base/process/kill_win.cc

 // Copyright (c) 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 "base/process/kill.h"
 
 #include <io.h>
 #include <windows.h>
 
 #include "base/bind.h"
@@ skipping 141 matching lines @@
                             const ProcessFilter* filter) {
   bool result = true;
   DWORD start_time = GetTickCount();
 
   NamedProcessIterator iter(executable_name, filter);
   for (const ProcessEntry* entry = iter.NextProcessEntry(); entry;
        entry = iter.NextProcessEntry()) {
     DWORD remaining_wait = static_cast<DWORD>(std::max(
         static_cast<int64>(0),
         wait.InMilliseconds() - (GetTickCount() - start_time)));
-    HANDLE process = OpenProcess(SYNCHRONIZE,
-                                 FALSE,
-                                 entry->th32ProcessID);
-    DWORD wait_result = WaitForSingleObject(process, remaining_wait);
-    CloseHandle(process);
+    Process process(Process::OpenWithAccess(entry->th32ProcessID, SYNCHRONIZE));
+    DWORD wait_result = WaitForSingleObject(process.Handle(), remaining_wait);
     result &= (wait_result == WAIT_OBJECT_0);
   }
 
   return result;
 }
 
 bool CleanupProcesses(const FilePath::StringType& executable_name,
                       TimeDelta wait,
                       int exit_code,
                       const ProcessFilter* filter) {
@@ skipping 12 matching lines @@
   }
 
   MessageLoop::current()->PostDelayedTask(
       FROM_HERE,
       Bind(&TimerExpiredTask::TimedOut,
            Owned(new TimerExpiredTask(process.Pass()))),
       TimeDelta::FromMilliseconds(kWaitInterval));
 }
 
 }  // namespace base