Regkey functions return error code instead of bool

chrome/installer/util/installation_state.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/installer/util/installation_state.h"
 
 #include "base/logging.h"
 #include "base/string_util.h"
 #include "base/version.h"
 #include "base/win/registry.h"
 #include "chrome/installer/util/google_update_constants.h"
 #include "chrome/installer/util/package_properties.h"
 
 namespace installer {
 
 ProductState::ProductState() {
 }
 
 void ProductState::Initialize(bool system_install,
                               const std::wstring& version_key,
                               const std::wstring& state_key) {
   const HKEY root_key = system_install ? HKEY_LOCAL_MACHINE : HKEY_CURRENT_USER;
   base::win::RegKey key(root_key, version_key.c_str(), KEY_QUERY_VALUE);
   std::wstring version_str;
-  if (key.ReadValue(google_update::kRegVersionField, &version_str)) {
+  LONG result = key.ReadValue(google_update::kRegVersionField, &version_str);
+  if (result == ERROR_SUCCESS) {
     version_.reset(Version::GetVersionFromString(WideToASCII(version_str)));
     if (version_.get() != NULL) {
       // The product is installed.  Check for the channel value (absent if not
       // installed/managed by Google Update).
-      if (!key.Open(root_key, state_key.c_str(), KEY_QUERY_VALUE) ||
-          !channel_.Initialize(key)) {
+      result = key.Open(root_key, state_key.c_str(), KEY_QUERY_VALUE);
+      if (result != ERROR_SUCCESS || !channel_.Initialize(key)) {
         channel_.set_value(std::wstring());
       }
     }
   } else {
     version_.reset();
   }
 }
 
 const Version& ProductState::version() const {
   DCHECK(version_.get() != NULL);
@@ skipping 84 matching lines @@
 
 // Included for testing.
 void InstallationState::SetProductState(bool system_install,
     BrowserDistribution::Type type, const ProductState& product_state) {
   ProductState& target = (system_install ? system_products_ :
       user_products_)[IndexFromDistType(type)];
   target.CopyFrom(product_state);
 }
 
 }  // namespace installer