Regkey functions return error code instead of bool

chrome/installer/util/google_chrome_distribution.cc

 // Copyright (c) 2011 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.
 //
 // This file defines specific implementation of BrowserDistribution class for
 // Google Chrome.
 
 #include "chrome/installer/util/google_chrome_distribution.h"
 
 #include <windows.h>
@@ skipping 430 matching lines @@
 
 std::wstring GoogleChromeDistribution::GetDistributionData(HKEY root_key) {
   std::wstring sub_key(google_update::kRegPathClientState);
   sub_key.append(L"\\");
   sub_key.append(product_guid());
 
   base::win::RegKey client_state_key(root_key, sub_key.c_str(), KEY_READ);
   std::wstring result;
   std::wstring brand_value;
   if (client_state_key.ReadValue(google_update::kRegRLZBrandField,
-                                 &brand_value)) {
+                                 &brand_value) == ERROR_SUCCESS) {
     result = google_update::kRegRLZBrandField;
     result.append(L"=");
     result.append(brand_value);
     result.append(L"&");
   }
 
   std::wstring client_value;
   if (client_state_key.ReadValue(google_update::kRegClientField,
-                                 &client_value)) {
+                                 &client_value) == ERROR_SUCCESS) {
     result.append(google_update::kRegClientField);
     result.append(L"=");
     result.append(client_value);
     result.append(L"&");
   }
 
   std::wstring ap_value;
   // If we fail to read the ap key, send up "&ap=" anyway to indicate
   // that this was probably a stable channel release.
   client_state_key.ReadValue(google_update::kRegApField, &ap_value);
@@ skipping 217 matching lines @@
   base::LaunchApp(cmd, false, false, NULL);
 }
 #endif
 
 bool GoogleChromeDistribution::SetChannelFlags(
     bool set,
     installer::ChannelInfo* channel_info) {
   DCHECK(channel_info);
   return channel_info->SetChrome(set);
 }