| Index: chrome/browser/content_settings/content_settings_default_provider_unittest.cc
|
| diff --git a/chrome/browser/content_settings/content_settings_default_provider_unittest.cc b/chrome/browser/content_settings/content_settings_default_provider_unittest.cc
|
| index 21f0ff7d75a566b893092ade42fb9ecda1cc9ca5..3f07131a113a52e6de9878ef450e1f9b8bc1b443 100644
|
| --- a/chrome/browser/content_settings/content_settings_default_provider_unittest.cc
|
| +++ b/chrome/browser/content_settings/content_settings_default_provider_unittest.cc
|
| @@ -43,11 +43,12 @@ TEST_F(DefaultProviderTest, DefaultValues) {
|
| CONTENT_SETTINGS_TYPE_COOKIES,
|
| std::string(),
|
| false));
|
| - provider_.SetContentSetting(ContentSettingsPattern::Wildcard(),
|
| - ContentSettingsPattern::Wildcard(),
|
| - CONTENT_SETTINGS_TYPE_COOKIES,
|
| - std::string(),
|
| - CONTENT_SETTING_BLOCK);
|
| + provider_.SetWebsiteSetting(
|
| + ContentSettingsPattern::Wildcard(),
|
| + ContentSettingsPattern::Wildcard(),
|
| + CONTENT_SETTINGS_TYPE_COOKIES,
|
| + std::string(),
|
| + Value::CreateIntegerValue(CONTENT_SETTING_BLOCK));
|
| EXPECT_EQ(CONTENT_SETTING_BLOCK,
|
| GetContentSetting(&provider_,
|
| GURL(),
|
| @@ -63,11 +64,12 @@ TEST_F(DefaultProviderTest, DefaultValues) {
|
| CONTENT_SETTINGS_TYPE_GEOLOCATION,
|
| std::string(),
|
| false));
|
| - provider_.SetContentSetting(ContentSettingsPattern::Wildcard(),
|
| - ContentSettingsPattern::Wildcard(),
|
| - CONTENT_SETTINGS_TYPE_GEOLOCATION,
|
| - std::string(),
|
| - CONTENT_SETTING_BLOCK);
|
| + provider_.SetWebsiteSetting(
|
| + ContentSettingsPattern::Wildcard(),
|
| + ContentSettingsPattern::Wildcard(),
|
| + CONTENT_SETTINGS_TYPE_GEOLOCATION,
|
| + std::string(),
|
| + Value::CreateIntegerValue(CONTENT_SETTING_BLOCK));
|
| EXPECT_EQ(CONTENT_SETTING_BLOCK,
|
| GetContentSetting(&provider_,
|
| GURL(),
|
| @@ -88,11 +90,12 @@ TEST_F(DefaultProviderTest, IgnoreNonDefaultSettings) {
|
| CONTENT_SETTINGS_TYPE_COOKIES,
|
| std::string(),
|
| false));
|
| - provider_.SetContentSetting(ContentSettingsPattern::FromURL(primary_url),
|
| - ContentSettingsPattern::FromURL(secondary_url),
|
| - CONTENT_SETTINGS_TYPE_COOKIES,
|
| - std::string(),
|
| - CONTENT_SETTING_BLOCK);
|
| + provider_.SetWebsiteSetting(
|
| + ContentSettingsPattern::FromURL(primary_url),
|
| + ContentSettingsPattern::FromURL(secondary_url),
|
| + CONTENT_SETTINGS_TYPE_COOKIES,
|
| + std::string(),
|
| + Value::CreateIntegerValue(CONTENT_SETTING_BLOCK));
|
| EXPECT_EQ(CONTENT_SETTING_ALLOW,
|
| GetContentSetting(&provider_,
|
| primary_url,
|
| @@ -108,20 +111,22 @@ TEST_F(DefaultProviderTest, Observer) {
|
| OnContentSettingChanged(
|
| _, _, CONTENT_SETTINGS_TYPE_IMAGES, ""));
|
| provider_.AddObserver(&mock_observer);
|
| - provider_.SetContentSetting(ContentSettingsPattern::Wildcard(),
|
| - ContentSettingsPattern::Wildcard(),
|
| - CONTENT_SETTINGS_TYPE_IMAGES,
|
| - std::string(),
|
| - CONTENT_SETTING_BLOCK);
|
| + provider_.SetWebsiteSetting(
|
| + ContentSettingsPattern::Wildcard(),
|
| + ContentSettingsPattern::Wildcard(),
|
| + CONTENT_SETTINGS_TYPE_IMAGES,
|
| + std::string(),
|
| + Value::CreateIntegerValue(CONTENT_SETTING_BLOCK));
|
|
|
| EXPECT_CALL(mock_observer,
|
| OnContentSettingChanged(
|
| _, _, CONTENT_SETTINGS_TYPE_GEOLOCATION, ""));
|
| - provider_.SetContentSetting(ContentSettingsPattern::Wildcard(),
|
| - ContentSettingsPattern::Wildcard(),
|
| - CONTENT_SETTINGS_TYPE_GEOLOCATION,
|
| - std::string(),
|
| - CONTENT_SETTING_BLOCK);
|
| + provider_.SetWebsiteSetting(
|
| + ContentSettingsPattern::Wildcard(),
|
| + ContentSettingsPattern::Wildcard(),
|
| + CONTENT_SETTINGS_TYPE_GEOLOCATION,
|
| + std::string(),
|
| + Value::CreateIntegerValue(CONTENT_SETTING_BLOCK));
|
| }
|
|
|
|
|
| @@ -132,11 +137,12 @@ TEST_F(DefaultProviderTest, ObserveDefaultPref) {
|
| scoped_ptr<Value> default_value(prefs->FindPreference(
|
| prefs::kDefaultContentSettings)->GetValue()->DeepCopy());
|
|
|
| - provider_.SetContentSetting(ContentSettingsPattern::Wildcard(),
|
| - ContentSettingsPattern::Wildcard(),
|
| - CONTENT_SETTINGS_TYPE_COOKIES,
|
| - std::string(),
|
| - CONTENT_SETTING_BLOCK);
|
| + provider_.SetWebsiteSetting(
|
| + ContentSettingsPattern::Wildcard(),
|
| + ContentSettingsPattern::Wildcard(),
|
| + CONTENT_SETTINGS_TYPE_COOKIES,
|
| + std::string(),
|
| + Value::CreateIntegerValue(CONTENT_SETTING_BLOCK));
|
| EXPECT_EQ(CONTENT_SETTING_BLOCK,
|
| GetContentSetting(&provider_,
|
| GURL(),
|
| @@ -189,11 +195,12 @@ TEST_F(DefaultProviderTest, OffTheRecord) {
|
|
|
| // Changing content settings on the main provider should also affect the
|
| // incognito map.
|
| - provider_.SetContentSetting(ContentSettingsPattern::Wildcard(),
|
| - ContentSettingsPattern::Wildcard(),
|
| - CONTENT_SETTINGS_TYPE_COOKIES,
|
| - std::string(),
|
| - CONTENT_SETTING_BLOCK);
|
| + provider_.SetWebsiteSetting(
|
| + ContentSettingsPattern::Wildcard(),
|
| + ContentSettingsPattern::Wildcard(),
|
| + CONTENT_SETTINGS_TYPE_COOKIES,
|
| + std::string(),
|
| + Value::CreateIntegerValue(CONTENT_SETTING_BLOCK));
|
| EXPECT_EQ(CONTENT_SETTING_BLOCK,
|
| GetContentSetting(&provider_,
|
| GURL(),
|
| @@ -211,11 +218,12 @@ TEST_F(DefaultProviderTest, OffTheRecord) {
|
| true));
|
|
|
| // Changing content settings on the incognito provider should be ignored.
|
| - otr_provider.SetContentSetting(ContentSettingsPattern::Wildcard(),
|
| - ContentSettingsPattern::Wildcard(),
|
| - CONTENT_SETTINGS_TYPE_COOKIES,
|
| - std::string(),
|
| - CONTENT_SETTING_ALLOW);
|
| + otr_provider.SetWebsiteSetting(
|
| + ContentSettingsPattern::Wildcard(),
|
| + ContentSettingsPattern::Wildcard(),
|
| + CONTENT_SETTINGS_TYPE_COOKIES,
|
| + std::string(),
|
| + Value::CreateIntegerValue(CONTENT_SETTING_ALLOW));
|
| EXPECT_EQ(CONTENT_SETTING_BLOCK,
|
| GetContentSetting(&provider_,
|
| GURL(),
|
|
|