Regkey functions return error code instead of bool

chrome_frame/test/ready_mode_unittest.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 <atlbase.h>
 #include <atlapp.h>
 #include <atlmisc.h>
 #include <atlwin.h>
 
 #include "testing/gtest/include/gtest/gtest.h"
@@ skipping 312 matching lines @@
 
   ReadyModeRegistryTest()
       : is_product_registered_(true),
         is_product_installed_(false),
         observer_(NULL),
         installation_state_(NULL) {
   }
 
   virtual void SetUp() {
     base::win::RegKey key;
-    ASSERT_TRUE(key.Create(HKEY_CURRENT_USER, kRootKey, KEY_ALL_ACCESS));
+    ASSERT_EQ(ERROR_SUCCESS, key.Create(HKEY_CURRENT_USER, kRootKey,
+                                        KEY_ALL_ACCESS));
     observer_ = new MockRegistryReadyModeStateObserver();
     installation_state_ = new MockInstallationState();
 
     EXPECT_CALL(*installation_state_, IsProductRegistered())
         .Times(testing::AnyNumber())
         .WillRepeatedly(ReturnPointee(&is_product_registered_));
     EXPECT_CALL(*installation_state_, IsProductInstalled())
         .Times(testing::AnyNumber())
         .WillRepeatedly(ReturnPointee(&is_product_installed_));
 
     ready_mode_state_.reset(new TimeControlledRegistryReadyModeState(
       kRootKey,
       base::TimeDelta::FromSeconds(kTemporaryDeclineDurationInSeconds),
       installation_state_,
       observer_));
   }
 
   virtual void TearDown() {
     base::win::RegKey key;
-    EXPECT_TRUE(key.Open(HKEY_CURRENT_USER, L"", KEY_ALL_ACCESS));
-    EXPECT_TRUE(key.DeleteKey(kRootKey));
+    EXPECT_EQ(ERROR_SUCCESS, key.Open(HKEY_CURRENT_USER, L"", KEY_ALL_ACCESS));
+    EXPECT_EQ(ERROR_SUCCESS, key.DeleteKey(kRootKey));
   }
 
  protected:
   void AdjustClockBySeconds(int seconds) {
     ready_mode_state_->now_ += base::TimeDelta::FromSeconds(seconds);
   }
 
   void ExpectUnregisterProductAndReturn(bool success) {
     EXPECT_CALL(*installation_state_, UnregisterProduct())
         .WillOnce(testing::DoAll(
@@ skipping 99 matching lines @@
 // Which component is responsible for messaging the user? The installer? The
 // InstallationState implementation? The ReadyModeState implementation?
 TEST_F(ReadyModeRegistryTest, AcceptChromeFrameInstallFails) {
   ASSERT_EQ(READY_MODE_ACTIVE, ready_mode_state_->GetStatus());
 
   ExpectInstallProductAndReturn(false);
   ready_mode_state_->AcceptChromeFrame();
 
   ASSERT_EQ(READY_MODE_ACTIVE, ready_mode_state_->GetStatus());
 }