Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1029)

Unified Diff: chrome/browser/policy/configuration_policy_provider_mac_unittest.cc

Issue 8586030: Added ConfigurationPolicyProvider::RefreshPolicies. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixed LoginUtilsTest Created 9 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/policy/configuration_policy_provider_mac_unittest.cc
diff --git a/chrome/browser/policy/configuration_policy_provider_mac_unittest.cc b/chrome/browser/policy/configuration_policy_provider_mac_unittest.cc
index a0b8a0b7e7a363dde35e9ac1ca7626f57be993bc..b8173cfdcd618eeef1fc052f34256e7bb479bf62 100644
--- a/chrome/browser/policy/configuration_policy_provider_mac_unittest.cc
+++ b/chrome/browser/policy/configuration_policy_provider_mac_unittest.cc
@@ -137,7 +137,8 @@ TEST_F(ConfigurationPolicyProviderMacTest, Invalid) {
prefs_->AddTestItem(name, invalid_data.get(), true);
// Create the provider and have it read |prefs_|.
- provider_.ForceReload();
+ provider_.RefreshPolicies();
+ loop_.RunAllPending();
PolicyMap policy_map;
EXPECT_TRUE(provider_.Provide(&policy_map));
EXPECT_TRUE(policy_map.empty());
@@ -152,7 +153,8 @@ TEST_F(ConfigurationPolicyProviderMacTest, TestNonForcedValue) {
prefs_->AddTestItem(name, test_value.get(), false);
// Create the provider and have it read |prefs_|.
- provider_.ForceReload();
+ provider_.RefreshPolicies();
+ loop_.RunAllPending();
PolicyMap policy_map;
EXPECT_TRUE(provider_.Provide(&policy_map));
EXPECT_TRUE(policy_map.empty());

Powered by Google App Engine
This is Rietveld 408576698