Regkey functions return error code instead of bool

ceee/ie/common/ie_util_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.
 //
 // Unit tests for the CEEE IE utilities.
 
 #include "ceee/ie/common/ie_util.h"
 
 #include "base/scoped_ptr.h"
 #include "ceee/testing/utils/mock_static.h"
@@ skipping 52 matching lines @@
 TEST_F(IeUtilTest, Ie8) {
   MockIeUtil ie_util_mock;
   EXPECT_CALL(ie_util_mock, GetIeVersion())
       .WillRepeatedly(Return(ie_util::IEVERSION_IE8));
 
   EXPECT_EQ(-1, GetAverageAddonLoadTimeMs(kAddonGuid));
 
   base::win::RegKey stats_key(HKEY_CURRENT_USER, kAddonStatsRegPath, KEY_WRITE);
   ASSERT_TRUE(stats_key.Valid());
 
-  ASSERT_TRUE(stats_key.WriteValue(L"LoadTime", L"time"));
+  ASSERT_EQ(ERROR_SUCCESS, stats_key.WriteValue(L"LoadTime", L"time"));
   EXPECT_EQ(-1, GetAverageAddonLoadTimeMs(kAddonGuid));
 
   DWORD time = 11;
-  ASSERT_TRUE(stats_key.WriteValue(L"LoadTime", time));
+  ASSERT_EQ(ERROR_SUCCESS, stats_key.WriteValue(L"LoadTime", time));
 
   EXPECT_EQ(11, GetAverageAddonLoadTimeMs(kAddonGuid));
 }
 
 TEST_F(IeUtilTest, Ie9) {
   MockIeUtil ie_util_mock;
   EXPECT_CALL(ie_util_mock, GetIeVersion())
       .WillRepeatedly(Return(ie_util::IEVERSION_IE9));
 
   EXPECT_EQ(-1, GetAverageAddonLoadTimeMs(kAddonGuid));
 
   base::win::RegKey stats_key(HKEY_CURRENT_USER, kAddonStatsRegPath, KEY_WRITE);
   ASSERT_TRUE(stats_key.Valid());
 
-  ASSERT_TRUE(stats_key.WriteValue(L"LoadTimeArray", L"time"));
+  ASSERT_EQ(ERROR_SUCCESS, stats_key.WriteValue(L"LoadTimeArray", L"time"));
   EXPECT_EQ(-1, GetAverageAddonLoadTimeMs(kAddonGuid));
 
   DWORD time[] = {1, 2, 3, 4, -1, 10};
 
-  ASSERT_TRUE(stats_key.WriteValue(L"LoadTimeArray", &time, sizeof(time[0]),
-                                   REG_BINARY));
+  ASSERT_EQ(ERROR_SUCCESS, stats_key.WriteValue(L"LoadTimeArray", &time,
+                                                sizeof(time[0]), REG_BINARY));
   EXPECT_EQ(-1, GetAverageAddonLoadTimeMs(kAddonGuid));
 
-  ASSERT_TRUE(stats_key.WriteValue(L"LoadTimeArray", &time, 10, REG_BINARY));
+  ASSERT_EQ(ERROR_SUCCESS,
+      stats_key.WriteValue(L"LoadTimeArray", &time, 10, REG_BINARY));
   EXPECT_EQ(-1, GetAverageAddonLoadTimeMs(kAddonGuid));
 
-  ASSERT_TRUE(stats_key.WriteValue(L"LoadTimeArray", &time, 4 * sizeof(time[0]),
-                                   REG_BINARY));
+  ASSERT_EQ(ERROR_SUCCESS, stats_key.WriteValue(L"LoadTimeArray",
+      &time, 4 * sizeof(time[0]), REG_BINARY));
   EXPECT_EQ(2, GetAverageAddonLoadTimeMs(kAddonGuid));
 
-  ASSERT_TRUE(stats_key.WriteValue(L"LoadTimeArray", time, sizeof(time),
-                                   REG_BINARY));
+  ASSERT_EQ(ERROR_SUCCESS, stats_key.WriteValue(L"LoadTimeArray", time,
+                                                sizeof(time), REG_BINARY));
   EXPECT_EQ(4, GetAverageAddonLoadTimeMs(kAddonGuid));
 }
 
 }  // namespace