Change PrefStore::ReadResult to a boolean.

chrome/browser/policy/managed_mode_policy_provider_unittest.cc

 // Copyright (c) 2012 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 "base/prefs/testing_pref_store.h"
 #include "chrome/browser/policy/configuration_policy_provider_test.h"
 #include "chrome/browser/policy/managed_mode_policy_provider.h"
 #include "chrome/browser/policy/policy_bundle.h"
 #include "chrome/browser/policy/policy_map.h"
 #include "testing/gtest/include/gtest/gtest.h"
@@ skipping 81 matching lines @@
 void TestHarness::InstallDictionaryPolicy(
     const std::string& policy_name,
     const base::DictionaryValue* policy_value) {
   InstallPolicy(policy_name, policy_value->DeepCopy());
 }
 
 void TestHarness::InstallPolicy(const std::string& policy_name,
                                 base::Value* policy_value) {
   base::DictionaryValue* cached_policy = NULL;
   base::Value* value = NULL;
-  PrefStore::ReadResult result =
-      pref_store_->GetMutableValue(ManagedModePolicyProvider::kPolicies,
-                                   &value);
-  switch (result) {
-    case PrefStore::READ_NO_VALUE:
-      cached_policy = new base::DictionaryValue;
-      pref_store_->SetValue(ManagedModePolicyProvider::kPolicies,
-                            cached_policy);
-      break;
-    case PrefStore::READ_OK:
-      ASSERT_TRUE(value->GetAsDictionary(&cached_policy));
-      break;
-    default:
-      FAIL() << "Invalid result reading policy: " << result;
-      return;
+  if (pref_store_->GetMutableValue(ManagedModePolicyProvider::kPolicies,
+                                   &value)) {
+    ASSERT_TRUE(value->GetAsDictionary(&cached_policy));
+  } else {
+    cached_policy = new base::DictionaryValue;
+    pref_store_->SetValue(ManagedModePolicyProvider::kPolicies,
+                          cached_policy);
   }
+
   cached_policy->SetWithoutPathExpansion(policy_name, policy_value);
 }
 
 }  // namespace
 
 // Instantiate abstract test case for basic policy reading tests.
 INSTANTIATE_TEST_CASE_P(
     ManagedModePolicyProviderTest,
     ConfigurationPolicyProviderTest,
     testing::Values(TestHarness::Create));
@@ skipping 45 matching lines @@
   // A newly-created provider should have the same policies.
   ManagedModePolicyProvider new_provider(pref_store_);
   new_provider.Init();
   EXPECT_TRUE(new_provider.policies().Equals(expected_bundle));
 
   // Cleanup.
   new_provider.Shutdown();
 }
 
 }  // namespace policy