| Index: chrome/browser/content_settings/host_content_settings_map_unittest.cc
|
| diff --git a/chrome/browser/content_settings/host_content_settings_map_unittest.cc b/chrome/browser/content_settings/host_content_settings_map_unittest.cc
|
| index f2c60c4f9ed415b462c7221950da06f418a2272e..f7d78aed7730705b7f03ae8875b7e58bbf301665 100644
|
| --- a/chrome/browser/content_settings/host_content_settings_map_unittest.cc
|
| +++ b/chrome/browser/content_settings/host_content_settings_map_unittest.cc
|
| @@ -13,6 +13,7 @@
|
| #include "chrome/browser/content_settings/cookie_settings.h"
|
| #include "chrome/browser/content_settings/host_content_settings_map.h"
|
| #include "chrome/browser/content_settings/mock_settings_observer.h"
|
| +#include "chrome/common/chrome_content_settings_client.h"
|
| #include "chrome/common/chrome_switches.h"
|
| #include "chrome/common/pref_names.h"
|
| #include "chrome/common/url_constants.h"
|
| @@ -32,9 +33,14 @@ class HostContentSettingsMapTest : public testing::Test {
|
| HostContentSettingsMapTest() : ui_thread_(BrowserThread::UI, &message_loop_) {
|
| }
|
|
|
| + content_settings::ContentSettingsClient* client() {
|
| + return &client_;
|
| + }
|
| +
|
| protected:
|
| base::MessageLoop message_loop_;
|
| content::TestBrowserThread ui_thread_;
|
| + content_settings::ChromeContentSettingsClient client_;
|
| };
|
|
|
| TEST_F(HostContentSettingsMapTest, DefaultValues) {
|
| @@ -78,7 +84,7 @@ TEST_F(HostContentSettingsMapTest, IndividualSettings) {
|
| // Check returning individual settings.
|
| GURL host("http://example.com/");
|
| ContentSettingsPattern pattern =
|
| - ContentSettingsPattern::FromString("[*.]example.com");
|
| + ContentSettingsPattern::FromString(client(), "[*.]example.com");
|
| EXPECT_EQ(CONTENT_SETTING_ALLOW,
|
| host_content_settings_map->GetContentSetting(
|
| host, host, CONTENT_SETTINGS_TYPE_IMAGES, std::string()));
|
| @@ -151,7 +157,7 @@ TEST_F(HostContentSettingsMapTest, IndividualSettings) {
|
|
|
| // Check returning all hosts for a setting.
|
| ContentSettingsPattern pattern2 =
|
| - ContentSettingsPattern::FromString("[*.]example.org");
|
| + ContentSettingsPattern::FromString(client(), "[*.]example.org");
|
| host_content_settings_map->SetContentSetting(
|
| pattern2,
|
| ContentSettingsPattern::Wildcard(),
|
| @@ -186,9 +192,9 @@ TEST_F(HostContentSettingsMapTest, Clear) {
|
|
|
| // Check clearing one type.
|
| ContentSettingsPattern pattern =
|
| - ContentSettingsPattern::FromString("[*.]example.org");
|
| + ContentSettingsPattern::FromString(client(), "[*.]example.org");
|
| ContentSettingsPattern pattern2 =
|
| - ContentSettingsPattern::FromString("[*.]example.net");
|
| + ContentSettingsPattern::FromString(client(), "[*.]example.net");
|
| host_content_settings_map->SetContentSetting(
|
| pattern2,
|
| ContentSettingsPattern::Wildcard(),
|
| @@ -235,9 +241,9 @@ TEST_F(HostContentSettingsMapTest, Patterns) {
|
| GURL host2("http://www.example.com/");
|
| GURL host3("http://example.org/");
|
| ContentSettingsPattern pattern1 =
|
| - ContentSettingsPattern::FromString("[*.]example.com");
|
| + ContentSettingsPattern::FromString(client(), "[*.]example.com");
|
| ContentSettingsPattern pattern2 =
|
| - ContentSettingsPattern::FromString("example.org");
|
| + ContentSettingsPattern::FromString(client(), "example.org");
|
| EXPECT_EQ(CONTENT_SETTING_ALLOW,
|
| host_content_settings_map->GetContentSetting(
|
| host1, host1, CONTENT_SETTINGS_TYPE_IMAGES, std::string()));
|
| @@ -274,7 +280,7 @@ TEST_F(HostContentSettingsMapTest, Observer) {
|
| MockSettingsObserver observer;
|
|
|
| ContentSettingsPattern primary_pattern =
|
| - ContentSettingsPattern::FromString("[*.]example.com");
|
| + ContentSettingsPattern::FromString(client(), "[*.]example.com");
|
| ContentSettingsPattern secondary_pattern =
|
| ContentSettingsPattern::Wildcard();
|
| EXPECT_CALL(observer,
|
| @@ -356,7 +362,7 @@ TEST_F(HostContentSettingsMapTest, ObserveExceptionPref) {
|
| prefs::kContentSettingsPatternPairs)->GetValue()->DeepCopy());
|
|
|
| ContentSettingsPattern pattern =
|
| - ContentSettingsPattern::FromString("[*.]example.com");
|
| + ContentSettingsPattern::FromString(client(), "[*.]example.com");
|
| GURL host("http://example.com");
|
|
|
| EXPECT_EQ(CONTENT_SETTING_ALLOW,
|
| @@ -398,7 +404,7 @@ TEST_F(HostContentSettingsMapTest, HostTrimEndingDotCheck) {
|
| CookieSettings::Factory::GetForProfile(&profile).get();
|
|
|
| ContentSettingsPattern pattern =
|
| - ContentSettingsPattern::FromString("[*.]example.com");
|
| + ContentSettingsPattern::FromString(client(), "[*.]example.com");
|
| GURL host_ending_with_dot("http://example.com./");
|
|
|
| EXPECT_EQ(CONTENT_SETTING_ALLOW,
|
| @@ -552,11 +558,11 @@ TEST_F(HostContentSettingsMapTest, NestedSettings) {
|
|
|
| GURL host("http://a.b.example.com/");
|
| ContentSettingsPattern pattern1 =
|
| - ContentSettingsPattern::FromString("[*.]example.com");
|
| + ContentSettingsPattern::FromString(client(), "[*.]example.com");
|
| ContentSettingsPattern pattern2 =
|
| - ContentSettingsPattern::FromString("[*.]b.example.com");
|
| + ContentSettingsPattern::FromString(client(), "[*.]b.example.com");
|
| ContentSettingsPattern pattern3 =
|
| - ContentSettingsPattern::FromString("a.b.example.com");
|
| + ContentSettingsPattern::FromString(client(), "a.b.example.com");
|
|
|
| host_content_settings_map->SetContentSetting(
|
| pattern1,
|
| @@ -621,7 +627,7 @@ TEST_F(HostContentSettingsMapTest, OffTheRecord) {
|
|
|
| GURL host("http://example.com/");
|
| ContentSettingsPattern pattern =
|
| - ContentSettingsPattern::FromString("[*.]example.com");
|
| + ContentSettingsPattern::FromString(client(), "[*.]example.com");
|
|
|
| EXPECT_EQ(CONTENT_SETTING_ALLOW,
|
| host_content_settings_map->GetContentSetting(
|
| @@ -763,7 +769,7 @@ TEST_F(HostContentSettingsMapTest,
|
|
|
| // Set pattern for JavaScript setting.
|
| ContentSettingsPattern pattern =
|
| - ContentSettingsPattern::FromString("[*.]example.com");
|
| + ContentSettingsPattern::FromString(client(), "[*.]example.com");
|
| host_content_settings_map->SetContentSetting(
|
| pattern,
|
| ContentSettingsPattern::Wildcard(),
|
| @@ -803,7 +809,7 @@ TEST_F(HostContentSettingsMapTest,
|
|
|
| // Set an exception to allow "[*.]example.com"
|
| ContentSettingsPattern pattern =
|
| - ContentSettingsPattern::FromString("[*.]example.com");
|
| + ContentSettingsPattern::FromString(client(), "[*.]example.com");
|
|
|
| host_content_settings_map->SetContentSetting(
|
| pattern,
|
| @@ -898,7 +904,7 @@ TEST_F(HostContentSettingsMapTest, GetContentSetting) {
|
| GURL host("http://example.com/");
|
| GURL embedder("chrome://foo");
|
| ContentSettingsPattern pattern =
|
| - ContentSettingsPattern::FromString("[*.]example.com");
|
| + ContentSettingsPattern::FromString(client(), "[*.]example.com");
|
| host_content_settings_map->SetContentSetting(
|
| pattern,
|
| ContentSettingsPattern::Wildcard(),
|
|
|