Cleanup Registry API: part 3.

chrome_frame/test/chrome_frame_test_utils.cc

 // Copyright (c) 2010 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_frame/test/chrome_frame_test_utils.h"
 
 #include <atlbase.h>
 #include <atlwin.h>
 #include <iepmapi.h>
 #include <oleacc.h>
@@ skipping 100 matching lines @@
   }
 
   return window_close_attempts;
 }
 
 std::wstring GetExecutableAppPath(const std::wstring& file) {
   std::wstring kAppPathsKey =
       L"SOFTWARE\\Microsoft\\Windows\\CurrentVersion\\App Paths\\";
 
   std::wstring app_path;
-  RegKey key(HKEY_LOCAL_MACHINE, (kAppPathsKey + file).c_str());
+  RegKey key(HKEY_LOCAL_MACHINE, (kAppPathsKey + file).c_str(), KEY_READ);
   if (key.Handle()) {
     key.ReadValue(NULL, &app_path);
   }
 
   return app_path;
 }
 
 std::wstring FormatCommandForApp(const std::wstring& exe_name,
                                  const std::wstring& argument) {
   std::wstring reg_path(StringPrintf(L"Applications\\%ls\\shell\\open\\command",
                                      exe_name.c_str()));
-  RegKey key(HKEY_CLASSES_ROOT, reg_path.c_str());
+  RegKey key(HKEY_CLASSES_ROOT, reg_path.c_str(), KEY_READ);
 
   std::wstring command;
   if (key.Handle()) {
     key.ReadValue(NULL, &command);
     int found = command.find(L"%1");
     if (found >= 0) {
       command.replace(found, 2, argument);
     }
   }
   return command;
@@ skipping 1113 matching lines @@
   }
 
   DLOG(INFO) << "Started crash_service.exe so you know if a test crashes!";
   // This sleep is to ensure that the crash service is done initializing, i.e
   // the pipe creation, etc.
   Sleep(500);
   return crash_service;
 }
 
 }  // namespace chrome_frame_test