Added ConfigurationPolicyProvider::RefreshPolicies.

chrome/browser/policy/cloud_policy_provider_unittest.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.
 
 #include "chrome/browser/policy/cloud_policy_provider.h"
 
 #include "base/basictypes.h"
 #include "base/values.h"
+#include "chrome/browser/browser_process.h"
+#include "chrome/browser/policy/browser_policy_connector.h"
 #include "chrome/browser/policy/cloud_policy_cache_base.h"
 #include "chrome/browser/policy/cloud_policy_provider_impl.h"
 #include "chrome/browser/policy/configuration_policy_pref_store.h"
 #include "policy/policy_constants.h"
 #include "testing/gmock/include/gmock/gmock.h"
 
 using testing::AnyNumber;
 using testing::_;
 
 namespace policy {
@@ skipping 21 matching lines @@
   }
 
   PolicyMap* raw_recommended_policy() {
     return &recommended_policy_;
   }
 
   void SetUnmanaged() {
     is_unmanaged_ = true;
   }
 
-  void SetFetchingDone() {
-    // Implement pure virtual method.
-  }
-
   void set_initialized(bool initialized) {
     initialization_complete_ = initialized;
   }
 
  private:
   DISALLOW_COPY_AND_ASSIGN(MockCloudPolicyCache);
 };
 
 class CloudPolicyProviderTest : public testing::Test {
  protected:
+  virtual void SetUp() OVERRIDE {
+    browser_policy_connector_.reset(new BrowserPolicyConnector());

[Mattias Nissler (ping if slow), 2011/11/18 16:14:11]

No need to wrap in a scoped_ptr.

[Joao da Silva, 2011/11/21 14:55:04]

Done.

+  }
+
   void CreateCloudPolicyProvider(CloudPolicyCacheBase::PolicyLevel level) {
-    cloud_policy_provider_.reset(new CloudPolicyProviderImpl(
-        GetChromePolicyDefinitionList(), level));
+    cloud_policy_provider_.reset(
+        new CloudPolicyProviderImpl(
+            browser_policy_connector_.get(),
+            GetChromePolicyDefinitionList(),
+            level));
   }
 
   // Appends the caches to a provider and then provides the policies to
   // |policy_map_|.
   void RunCachesThroughProvider(MockCloudPolicyCache caches[], int n,
                                 CloudPolicyCacheBase::PolicyLevel level) {
     CloudPolicyProviderImpl provider(
+        g_browser_process->browser_policy_connector(),
         GetChromePolicyDefinitionList(),
         level);
     for (int i = 0; i < n; i++) {
       provider.AppendCache(&caches[i]);
     }
     policy_map_.reset(new PolicyMap());
     provider.Provide(policy_map_.get());
   }
 
   // Checks a string policy in |policy_map_|.
@@ skipping 20 matching lines @@
   }
 
   void CombineTwoPolicyMaps(const PolicyMap& base,
                             const PolicyMap& overlay,
                             PolicyMap* out_map) {
     DCHECK(cloud_policy_provider_.get());
     cloud_policy_provider_->CombineTwoPolicyMaps(base, overlay, out_map);
   }
 
  private:
-  // Some tests need a list of policies that doesn't contain any proxy
-  // policies. Note: these policies will be handled as if they had the
-  // type of Value::TYPE_INTEGER.
-  static const ConfigurationPolicyType simple_policies[];
-
+  scoped_ptr<BrowserPolicyConnector> browser_policy_connector_;
   scoped_ptr<CloudPolicyProviderImpl> cloud_policy_provider_;
   scoped_ptr<PolicyMap> policy_map_;
 };
 
 // Proxy setting distributed over multiple caches.
 TEST_F(CloudPolicyProviderTest,
        ProxySettingDistributedOverMultipleCaches) {
   // There are proxy_policy_count()+1 = 6 caches and they are mixed together by
   // one instance of CloudPolicyProvider. The first cache has some policies but
   // no proxy-related ones. The following caches have each one proxy-policy set.
@@ skipping 105 matching lines @@
   B.Set(kPolicyProxyPacUrl, Value::CreateIntegerValue(b_value));
   B.Set(kPolicyProxyBypassList, Value::CreateIntegerValue(b_value));
 
   CombineTwoPolicyMaps(A, B, &C);
 
   EXPECT_TRUE(A.Equals(C));
   EXPECT_FALSE(B.Equals(C));
 }
 
 }  // namespace policy