Move OpenProcessHandle to Process::Open.

chrome/browser/chromeos/policy/device_status_collector.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/chromeos/policy/device_status_collector.h"
 
 #include <stdint.h>
 #include <limits>
 #include <sys/statvfs.h>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/location.h"
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/posix/eintr_wrapper.h"
 #include "base/prefs/pref_registry_simple.h"
 #include "base/prefs/pref_service.h"
 #include "base/prefs/scoped_user_pref_update.h"
-#include "base/process/process_handle.h"
+#include "base/process/process.h"
 #include "base/process/process_iterator.h"
 #include "base/process/process_metrics.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/sys_info.h"
 #include "base/task_runner_util.h"
 #include "base/values.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/chromeos/policy/browser_policy_connector_chromeos.h"
 #include "chrome/browser/chromeos/settings/cros_settings.h"
 #include "chrome/common/chrome_version_info.h"
@@ skipping 399 matching lines @@
     resource_usage_.pop_front();
 }
 
 std::vector<double> DeviceStatusCollector::GetPerProcessCPUUsage() {
   std::vector<double> cpu_usage;
   base::ProcessIterator process_iter(nullptr);
 
   const int num_processors = base::SysInfo::NumberOfProcessors();
   while (const base::ProcessEntry* process_entry =
          process_iter.NextProcessEntry()) {
-    base::ProcessHandle process;
-    if (!base::OpenProcessHandle(process_entry->pid(), &process)) {
+    base::Process process = base::Process::Open(process_entry->pid());
+    if (!process.IsValid()) {
       LOG(ERROR) << "Could not create process handle for process "
                   << process_entry->pid();
       continue;
     }
     scoped_ptr<base::ProcessMetrics> metrics(
-        base::ProcessMetrics::CreateProcessMetrics(process));
+        base::ProcessMetrics::CreateProcessMetrics(process.Handle()));
     const double usage = metrics->GetPlatformIndependentCPUUsage();
-    base::CloseProcessHandle(process);
     DCHECK_LE(0, usage);
     if (usage > 0) {
       // Convert CPU usage from "percentage of a single core" to "percentage of
       // all CPU available".
       cpu_usage.push_back(usage / num_processors);
     }
   }
   return cpu_usage;
 }
 
@@ skipping 326 matching lines @@
   ScheduleGeolocationUpdateRequest();
 }
 
 void DeviceStatusCollector::ReceiveVolumeInfo(
     const std::vector<em::VolumeInfo>& info) {
   if (report_hardware_status_)
     volume_info_ = info;
 }
 
 }  // namespace policy