Don't grant unnecessary handle privileges in OpenProcessHandle.

chrome/test/page_cycler/page_cycler_test.cc

 // Copyright (c) 2006-2009 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/basictypes.h"
 #include "base/command_line.h"
 #include "base/file_path.h"
 #include "base/file_util.h"
 #include "base/path_service.h"
 #include "base/process_util.h"
@@ skipping 130 matching lines @@
 
   void PrintIOPerfInfo(const char* test_name) {
     FilePath data_dir;
     PathService::Get(chrome::DIR_USER_DATA, &data_dir);
     int browser_process_pid = ChromeBrowserProcessId(data_dir);
     ChromeProcessList chrome_processes(GetRunningChromeProcesses(data_dir));
 
     ChromeProcessList::const_iterator it;
     for (it = chrome_processes.begin(); it != chrome_processes.end(); ++it) {
       base::ProcessHandle process_handle;
-      if (!base::OpenProcessHandle(*it, &process_handle)) {
+      if (!base::OpenPrivilegedProcessHandle(*it, &process_handle)) {
         NOTREACHED();
       }
 
       scoped_ptr<base::ProcessMetrics> process_metrics;
       process_metrics.reset(
           base::ProcessMetrics::CreateProcessMetrics(process_handle));
       IoCounters io_counters;
       memset(&io_counters, 0, sizeof(io_counters));
 
       if (process_metrics.get()->GetIOCounters(&io_counters)) {
@@ skipping 49 matching lines @@
   void PrintMemoryUsageInfo(const char* test_name) {
     FilePath data_dir;
     PathService::Get(chrome::DIR_USER_DATA, &data_dir);
     int browser_process_pid = ChromeBrowserProcessId(data_dir);
     ChromeProcessList chrome_processes(GetRunningChromeProcesses(data_dir));
 
 #if !defined(OS_MACOSX)
     ChromeProcessList::const_iterator it;
     for (it = chrome_processes.begin(); it != chrome_processes.end(); ++it) {
       base::ProcessHandle process_handle;
-      if (!base::OpenProcessHandle(*it, &process_handle)) {
+      if (!base::OpenPrivilegedProcessHandle(*it, &process_handle)) {
         NOTREACHED();
       }
 
       scoped_ptr<base::ProcessMetrics> process_metrics;
       process_metrics.reset(
           base::ProcessMetrics::CreateProcessMetrics(process_handle));
 
       std::string chrome_name = (*it == browser_process_pid) ? "_b" : "_r";
 
       std::string trace_name(test_name);
@@ skipping 194 matching lines @@
 
 TEST_F(PageCyclerTest, BloatHttp) {
   RunTest("bloat", true);
 }
 
 TEST_F(PageCyclerReferenceTest, BloatHttp) {
   RunTest("bloat", true);
 }
 
 }  // namespace