Index: chrome/browser/content_settings/content_settings_pref_provider_unittest.cc |
diff --git a/chrome/browser/content_settings/content_settings_pref_provider_unittest.cc b/chrome/browser/content_settings/content_settings_pref_provider_unittest.cc |
index 2fc3c057b762368eed5777b2f947995ccf28851a..2451a563d1d834548683451a0a3f500efb35710b 100644 |
--- a/chrome/browser/content_settings/content_settings_pref_provider_unittest.cc |
+++ b/chrome/browser/content_settings/content_settings_pref_provider_unittest.cc |
@@ -29,6 +29,8 @@ |
#include "components/content_settings/core/browser/content_settings_pref.h" |
#include "components/content_settings/core/browser/content_settings_rule.h" |
#include "components/content_settings/core/browser/content_settings_utils.h" |
+#include "components/content_settings/core/browser/website_settings_info.h" |
+#include "components/content_settings/core/browser/website_settings_registry.h" |
#include "components/content_settings/core/test/content_settings_test_utils.h" |
#include "components/pref_registry/pref_registry_syncable.h" |
#include "content/public/test/test_browser_thread_bundle.h" |
@@ -104,7 +106,7 @@ TEST_F(PrefProviderTest, Observer) { |
ContentSettingsPattern pattern = |
ContentSettingsPattern::FromString("[*.]example.com"); |
- content_settings::MockObserver mock_observer; |
+ MockObserver mock_observer; |
EXPECT_CALL(mock_observer, |
OnContentSettingChanged(pattern, |
ContentSettingsPattern::Wildcard(), |
@@ -184,9 +186,10 @@ TEST_F(PrefProviderTest, Incognito) { |
CONTENT_SETTINGS_TYPE_IMAGES, |
std::string(), |
false)); |
+ const WebsiteSettingsInfo* info = |
+ WebsiteSettingsRegistry::GetInstance()->Get(CONTENT_SETTINGS_TYPE_IMAGES); |
// But the value should not be overridden in the OTR user prefs accidentally. |
- EXPECT_FALSE(otr_user_prefs->IsSetInOverlay( |
- prefs::kContentSettingsImagesPatternPairs)); |
+ EXPECT_FALSE(otr_user_prefs->IsSetInOverlay(info->pref_name())); |
pref_content_settings_provider.ShutdownOnUIThread(); |
pref_content_settings_provider_incognito.ShutdownOnUIThread(); |
@@ -422,11 +425,12 @@ TEST_F(PrefProviderTest, Deadlock) { |
// is sent, and this used to happen when |PrefProvider| was still holding its |
// lock. |
+ const WebsiteSettingsInfo* info = |
+ WebsiteSettingsRegistry::GetInstance()->Get(CONTENT_SETTINGS_TYPE_IMAGES); |
PrefProvider provider(&prefs, false); |
DeadlockCheckerObserver observer(&prefs, &provider); |
{ |
- DictionaryPrefUpdate update(&prefs, |
- prefs::kContentSettingsImagesPatternPairs); |
+ DictionaryPrefUpdate update(&prefs, info->pref_name()); |
base::DictionaryValue* mutable_settings = update.Get(); |
mutable_settings->SetWithoutPathExpansion("www.example.com,*", |
new base::DictionaryValue()); |
@@ -582,11 +586,12 @@ TEST_F(PrefProviderTest, ClearAllContentSettingsRules) { |
provider.ClearAllContentSettingsRules(CONTENT_SETTINGS_TYPE_GEOLOCATION); |
provider.ClearAllContentSettingsRules(CONTENT_SETTINGS_TYPE_PLUGINS); |
+ WebsiteSettingsRegistry* registry = WebsiteSettingsRegistry::GetInstance(); |
// Test that the preferences for images, geolocation and plugins are empty. |
const char* empty_prefs[] = { |
- prefs::kContentSettingsImagesPatternPairs, |
- prefs::kContentSettingsGeolocationPatternPairs, |
- prefs::kContentSettingsPluginsPatternPairs |
+ registry->Get(CONTENT_SETTINGS_TYPE_IMAGES)->pref_name().c_str(), |
+ registry->Get(CONTENT_SETTINGS_TYPE_GEOLOCATION)->pref_name().c_str(), |
+ registry->Get(CONTENT_SETTINGS_TYPE_PLUGINS)->pref_name().c_str(), |
}; |
for (const char* pref : empty_prefs) { |
@@ -597,8 +602,8 @@ TEST_F(PrefProviderTest, ClearAllContentSettingsRules) { |
// Test that the preferences for cookies and notifications are not empty. |
const char* nonempty_prefs[] = { |
- prefs::kContentSettingsCookiesPatternPairs, |
- prefs::kContentSettingsNotificationsPatternPairs |
+ registry->Get(CONTENT_SETTINGS_TYPE_COOKIES)->pref_name().c_str(), |
+ registry->Get(CONTENT_SETTINGS_TYPE_NOTIFICATIONS)->pref_name().c_str(), |
}; |
for (const char* pref : nonempty_prefs) { |