Implement the windows policy provider based on the AsyncPolicyLoader.

chrome/browser/policy/policy_loader_win_unittest.cc

Index: chrome/browser/policy/policy_loader_win_unittest.cc
diff --git a/chrome/browser/policy/configuration_policy_provider_win_unittest.cc b/chrome/browser/policy/policy_loader_win_unittest.cc
similarity index 92%
rename from chrome/browser/policy/configuration_policy_provider_win_unittest.cc
rename to chrome/browser/policy/policy_loader_win_unittest.cc
index d8d9142353c6210eba7169326691a3bf1f41eec0..a25b5688aad90aacb31dcb4518f554919e845396 100644
--- a/chrome/browser/policy/configuration_policy_provider_win_unittest.cc
+++ b/chrome/browser/policy/policy_loader_win_unittest.cc
@@ -2,7 +2,8 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include <gtest/gtest.h>
+#include "chrome/browser/policy/policy_loader_win.h"
+
 #include <windows.h>
 
 #include "base/json/json_writer.h"
@@ -10,9 +11,7 @@
 #include "base/string_number_conversions.h"
 #include "base/utf_string_conversions.h"
 #include "base/win/registry.h"
-#include "chrome/browser/policy/asynchronous_policy_test_base.h"
 #include "chrome/browser/policy/configuration_policy_provider_test.h"
-#include "chrome/browser/policy/configuration_policy_provider_win.h"
 #include "chrome/browser/policy/policy_bundle.h"
 #include "chrome/browser/policy/policy_map.h"
 #include "policy/policy_constants.h"
@@ -141,7 +140,7 @@ void TestHarness::SetUp() {}
 
 ConfigurationPolicyProvider* TestHarness::CreateProvider(
     const PolicyDefinitionList* policy_definition_list) {
-  return new ConfigurationPolicyProviderWin(policy_definition_list);
+  return WinPolicyLoader::CreateProvider(policy_definition_list);
 }
 
 void TestHarness::InstallEmptyPolicy() {}
@@ -210,22 +209,20 @@ PolicyProviderTestHarness* TestHarness::CreateHKLM() {
 
 // Instantiate abstract test case for basic policy reading tests.
 INSTANTIATE_TEST_CASE_P(
-    ConfigurationPolicyProviderWinTest,
+    WinPolicyLoaderTest,
     ConfigurationPolicyProviderTest,
     testing::Values(TestHarness::CreateHKCU, TestHarness::CreateHKLM));
 
 // Test cases for windows policy provider specific functionality.
-class ConfigurationPolicyProviderWinTest : public AsynchronousPolicyTestBase {
+class WinPolicyLoaderTest : public PolicyTestBase {

[Mattias Nissler (ping if slow), 2012/06/04 09:29:41]

same comment as for mac regarding clashing test names

[Joao da Silva, 2012/06/06 13:05:24]

Done.

  protected:
-  ConfigurationPolicyProviderWinTest()
-      : provider_(&test_policy_definitions::kList) {}
-  virtual ~ConfigurationPolicyProviderWinTest() {}
+  WinPolicyLoaderTest() {}
+  virtual ~WinPolicyLoaderTest() {}
 
   ScopedGroupPolicyRegistrySandbox registry_sandbox_;
-  ConfigurationPolicyProviderWin provider_;
 };
 
-TEST_F(ConfigurationPolicyProviderWinTest, HKLMOverHKCU) {
+TEST_F(WinPolicyLoaderTest, HKLMOverHKCU) {
   RegKey hklm_key(HKEY_LOCAL_MACHINE,
                   policy::kRegistryMandatorySubKey,
                   KEY_ALL_ACCESS);
@@ -237,7 +234,9 @@ TEST_F(ConfigurationPolicyProviderWinTest, HKLMOverHKCU) {
   hkcu_key.WriteValue(UTF8ToUTF16(test_policy_definitions::kKeyString).c_str(),
                       UTF8ToUTF16("hkcu").c_str());
 
-  provider_.RefreshPolicies();
+  scoped_ptr<AsyncPolicyProvider> provider_(
+      WinPolicyLoader::CreateProvider(&test_policy_definitions::kList));
+  provider_->RefreshPolicies();
   loop_.RunAllPending();
 
   PolicyBundle expected_bundle;
@@ -246,7 +245,7 @@ TEST_F(ConfigurationPolicyProviderWinTest, HKLMOverHKCU) {
            POLICY_LEVEL_MANDATORY,
            POLICY_SCOPE_MACHINE,
            base::Value::CreateStringValue("hklm"));
-  EXPECT_TRUE(provider_.policies().Equals(expected_bundle));
+  EXPECT_TRUE(provider_->policies().Equals(expected_bundle));
 }
 
 }  // namespace policy