Removed ConfigurationPolicyProvider::Provide().

chrome/browser/policy/policy_bundle_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 "chrome/browser/policy/policy_bundle.h"
 
 #include "base/memory/scoped_ptr.h"
 #include "base/values.h"
 #include "chrome/browser/policy/policy_map.h"
 #include "policy/policy_constants.h"
@@ skipping 170 matching lines @@
   EXPECT_TRUE(
       merged.Get(POLICY_DOMAIN_EXTENSIONS, kExtension0).Equals(policy0));
   // extension1 comes only from bundle1.
   EXPECT_TRUE(
       merged.Get(POLICY_DOMAIN_EXTENSIONS, kExtension1).Equals(policy1));
   // extension2 comes only from bundle2.
   EXPECT_TRUE(
       merged.Get(POLICY_DOMAIN_EXTENSIONS, kExtension2).Equals(policy2));
 }
 
+TEST(PolicyBundleTest, Equals) {
+  PolicyBundle bundle;
+  AddTestPolicies(&bundle.Get(POLICY_DOMAIN_CHROME, std::string()));
+  AddTestPolicies(&bundle.Get(POLICY_DOMAIN_EXTENSIONS, kExtension0));
+
+  PolicyBundle other;
+  EXPECT_FALSE(bundle.Equals(other));
+  other.CopyFrom(bundle);
+  EXPECT_TRUE(bundle.Equals(other));
+
+  AddTestPolicies(&bundle.Get(POLICY_DOMAIN_EXTENSIONS, kExtension1));
+  EXPECT_FALSE(bundle.Equals(other));
+  other.CopyFrom(bundle);
+  EXPECT_TRUE(bundle.Equals(other));
+  AddTestPolicies(&other.Get(POLICY_DOMAIN_EXTENSIONS, kExtension2));
+  EXPECT_FALSE(bundle.Equals(other));
+
+  other.CopyFrom(bundle);
+  bundle.Get(POLICY_DOMAIN_CHROME, std::string())
+        .Set(kPolicy0, POLICY_LEVEL_MANDATORY, POLICY_SCOPE_USER,

[Mattias Nissler (ping if slow), 2012/05/14 16:55:59]

4 spaces indentation

[Joao da Silva, 2012/05/15 13:07:05]

Done.

+             base::Value::CreateIntegerValue(123));
+  EXPECT_FALSE(bundle.Equals(other));
+  other.CopyFrom(bundle);
+  EXPECT_TRUE(bundle.Equals(other));
+  bundle.Get(POLICY_DOMAIN_CHROME, std::string())
+        .Set(kPolicy0, POLICY_LEVEL_MANDATORY, POLICY_SCOPE_MACHINE,

[Mattias Nissler (ping if slow), 2012/05/14 16:55:59]

same here

[Joao da Silva, 2012/05/15 13:07:05]

Done.

+             base::Value::CreateIntegerValue(123));
+  EXPECT_FALSE(bundle.Equals(other));
+
+  // Test non-const Get().
+  bundle.Clear();
+  other.Clear();
+  PolicyMap& policy_map = bundle.Get(POLICY_DOMAIN_CHROME, "");
+  EXPECT_TRUE(bundle.Equals(other));
+  policy_map.Set(kPolicy0, POLICY_LEVEL_MANDATORY, POLICY_SCOPE_USER,
+                 base::Value::CreateIntegerValue(123));
+  EXPECT_FALSE(bundle.Equals(other));
+}
+
 }  // namespace policy