Move OpenProcessHandle to Process::Open.

components/browser_watcher/watcher_metrics_provider_win.cc

 // Copyright (c) 2014 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 "components/browser_watcher/watcher_metrics_provider_win.h"
 
 #include <limits>
 #include <vector>
 
 #include "base/metrics/sparse_histogram.h"
-#include "base/process/process_handle.h"
+#include "base/process/process.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_piece.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/win/registry.h"
 
 namespace browser_watcher {
 
 namespace {
 
 void CompileAsserts() {
@@ skipping 22 matching lines @@
     return true;
 
   // This is a very inexpensive check for the common case of our own PID.
   if (static_cast<base::ProcessId>(pid) == base::GetCurrentProcId())
     return false;
 
   // The process is not our own - see whether a process with this PID exists.
   // This is more expensive than the above check, but should also be very rare,
   // as this only happens more than once for a given PID if a user is running
   // multiple Chrome instances concurrently.
-  base::ProcessHandle process = base::kNullProcessHandle;
-  if (base::OpenProcessHandle(static_cast<base::ProcessId>(pid), &process)) {
-    base::CloseProcessHandle(process);
-
+  base::Process process =
+      base::Process::Open(static_cast<base::ProcessId>(pid));
+  if (process.IsValid()) {
     // The fact that it was possible to open the process says it's live.
     return false;
   }
 
   return true;
 }
 
 void RecordExitCodes(const base::string16& registry_path) {
   base::win::RegKey regkey(HKEY_CURRENT_USER,
                            registry_path.c_str(),
@@ skipping 165 matching lines @@
   // user account, there's a small race that will double-report the exit codes
   // from both/multiple instances. This ought to be vanishingly rare and will
   // only manifest as low-level "random" noise. To work around this it would be
   // necessary to implement some form of global locking, which is not worth it
   // here.
   RecordExitCodes(registry_path_);
   MaybeRecordExitFunnels(registry_path_, report_exit_funnels_);
 }
 
 }  // namespace browser_watcher