Index: chrome/browser/policy/configuration_policy_pref_store_unittest.cc |
diff --git a/chrome/browser/policy/configuration_policy_pref_store_unittest.cc b/chrome/browser/policy/configuration_policy_pref_store_unittest.cc |
index 100c77487bd3aecdc6813efde4e329da08a53318..1b95e82447961449f66dacb47908f11850e9e1a8 100644 |
--- a/chrome/browser/policy/configuration_policy_pref_store_unittest.cc |
+++ b/chrome/browser/policy/configuration_policy_pref_store_unittest.cc |
@@ -71,11 +71,11 @@ INSTANTIATE_TEST_CASE_P( |
ConfigurationPolicyPrefStoreListTestInstance, |
ConfigurationPolicyPrefStoreListTest, |
testing::Values( |
- TypeAndName(kPolicyURLsToRestoreOnStartup, |
+ TypeAndName(kPolicyRestoreOnStartupURLs, |
prefs::kURLsToRestoreOnStartup), |
- TypeAndName(kPolicyExtensionInstallAllowList, |
+ TypeAndName(kPolicyExtensionInstallWhitelist, |
prefs::kExtensionInstallAllowList), |
- TypeAndName(kPolicyExtensionInstallDenyList, |
+ TypeAndName(kPolicyExtensionInstallBlacklist, |
prefs::kExtensionInstallDenyList), |
TypeAndName(kPolicyDisabledPlugins, |
prefs::kPluginsPluginsBlacklist))); |
@@ -105,11 +105,11 @@ INSTANTIATE_TEST_CASE_P( |
ConfigurationPolicyPrefStoreStringTestInstance, |
ConfigurationPolicyPrefStoreStringTest, |
testing::Values( |
- TypeAndName(kPolicyHomePage, |
+ TypeAndName(kPolicyHomepageLocation, |
prefs::kHomePage), |
- TypeAndName(kPolicyApplicationLocale, |
+ TypeAndName(kPolicyApplicationLocaleValue, |
prefs::kApplicationLocale), |
- TypeAndName(kPolicyApplicationLocale, |
+ TypeAndName(kPolicyApplicationLocaleValue, |
prefs::kApplicationLocale), |
TypeAndName(kPolicyAuthSchemes, |
prefs::kAuthSchemes), |
@@ -270,7 +270,7 @@ TEST_F(ConfigurationPolicyPrefStoreProxyTest, ManualOptions) { |
Value::CreateStringValue("http://chromium.org/override")); |
provider.AddPolicy(kPolicyProxyServer, |
Value::CreateStringValue("chromium.org")); |
- provider.AddPolicy(kPolicyProxyMode, |
+ provider.AddPolicy(kPolicyProxyServerMode, |
Value::CreateIntegerValue( |
kPolicyManuallyConfiguredProxyMode)); |
@@ -282,7 +282,7 @@ TEST_F(ConfigurationPolicyPrefStoreProxyTest, ManualOptions) { |
TEST_F(ConfigurationPolicyPrefStoreProxyTest, ManualOptionsReversedApplyOrder) { |
MockConfigurationPolicyProvider provider; |
- provider.AddPolicy(kPolicyProxyMode, |
+ provider.AddPolicy(kPolicyProxyServerMode, |
Value::CreateIntegerValue( |
kPolicyManuallyConfiguredProxyMode)); |
provider.AddPolicy(kPolicyProxyBypassList, |
@@ -298,7 +298,7 @@ TEST_F(ConfigurationPolicyPrefStoreProxyTest, ManualOptionsReversedApplyOrder) { |
TEST_F(ConfigurationPolicyPrefStoreProxyTest, NoProxy) { |
MockConfigurationPolicyProvider provider; |
- provider.AddPolicy(kPolicyProxyMode, |
+ provider.AddPolicy(kPolicyProxyServerMode, |
Value::CreateIntegerValue(kPolicyNoProxyServerMode)); |
ConfigurationPolicyPrefStore store(&provider); |
@@ -307,7 +307,7 @@ TEST_F(ConfigurationPolicyPrefStoreProxyTest, NoProxy) { |
TEST_F(ConfigurationPolicyPrefStoreProxyTest, AutoDetect) { |
MockConfigurationPolicyProvider provider; |
- provider.AddPolicy(kPolicyProxyMode, |
+ provider.AddPolicy(kPolicyProxyServerMode, |
Value::CreateIntegerValue(kPolicyAutoDetectProxyMode)); |
ConfigurationPolicyPrefStore store(&provider); |
@@ -319,7 +319,7 @@ TEST_F(ConfigurationPolicyPrefStoreProxyTest, AutoDetectPac) { |
provider.AddPolicy(kPolicyProxyPacUrl, |
Value::CreateStringValue("http://short.org/proxy.pac")); |
provider.AddPolicy( |
- kPolicyProxyMode, |
+ kPolicyProxyServerMode, |
Value::CreateIntegerValue(kPolicyManuallyConfiguredProxyMode)); |
ConfigurationPolicyPrefStore store(&provider); |
@@ -329,7 +329,7 @@ TEST_F(ConfigurationPolicyPrefStoreProxyTest, AutoDetectPac) { |
TEST_F(ConfigurationPolicyPrefStoreProxyTest, UseSystem) { |
MockConfigurationPolicyProvider provider; |
- provider.AddPolicy(kPolicyProxyMode, |
+ provider.AddPolicy(kPolicyProxyServerMode, |
Value::CreateIntegerValue(kPolicyUseSystemProxyMode)); |
ConfigurationPolicyPrefStore store(&provider); |
@@ -339,7 +339,7 @@ TEST_F(ConfigurationPolicyPrefStoreProxyTest, UseSystem) { |
TEST_F(ConfigurationPolicyPrefStoreProxyTest, ProxyInvalid) { |
for (int i = 0; i < MODE_COUNT; ++i) { |
MockConfigurationPolicyProvider provider; |
- provider.AddPolicy(kPolicyProxyMode, Value::CreateIntegerValue(i)); |
+ provider.AddPolicy(kPolicyProxyServerMode, Value::CreateIntegerValue(i)); |
// No mode expects all three parameters being set. |
provider.AddPolicy(kPolicyProxyPacUrl, |
Value::CreateStringValue("http://short.org/proxy.pac")); |
@@ -605,7 +605,7 @@ TEST_F(ConfigurationPolicyPrefStoreRefreshTest, Refresh) { |
store_.GetValue(prefs::kHomePage, NULL)); |
EXPECT_CALL(observer_, OnPrefValueChanged(prefs::kHomePage)).Times(1); |
- provider_.AddPolicy(kPolicyHomePage, |
+ provider_.AddPolicy(kPolicyHomepageLocation, |
Value::CreateStringValue("http://www.chromium.org")); |
store_.OnUpdatePolicy(); |
Mock::VerifyAndClearExpectations(&observer_); |
@@ -618,7 +618,7 @@ TEST_F(ConfigurationPolicyPrefStoreRefreshTest, Refresh) { |
Mock::VerifyAndClearExpectations(&observer_); |
EXPECT_CALL(observer_, OnPrefValueChanged(prefs::kHomePage)).Times(1); |
- provider_.RemovePolicy(kPolicyHomePage); |
+ provider_.RemovePolicy(kPolicyHomepageLocation); |
store_.OnUpdatePolicy(); |
Mock::VerifyAndClearExpectations(&observer_); |
EXPECT_EQ(PrefStore::READ_NO_VALUE, |