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

Unified Diff: chrome/browser/extensions/extension_content_settings_apitest.cc

Issue 7713034: HostContentSettingsMap refactoring. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Created 9 years, 4 months 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/extensions/extension_content_settings_apitest.cc
diff --git a/chrome/browser/extensions/extension_content_settings_apitest.cc b/chrome/browser/extensions/extension_content_settings_apitest.cc
index ee3b2ccb3f5b24f64478be42dc9514ec57bffe5b..12e3fe00b8f26db1b811c4c36b6333422d5a6bda 100644
--- a/chrome/browser/extensions/extension_content_settings_apitest.cc
+++ b/chrome/browser/extensions/extension_content_settings_apitest.cc
@@ -31,12 +31,13 @@ IN_PROC_BROWSER_TEST_F(ExtensionApiTest, ContentSettings) {
EXPECT_TRUE(pref_service->GetBoolean(prefs::kEnableReferrers));
HostContentSettingsMap* map =
browser()->profile()->GetHostContentSettingsMap();
+ CookieContentSettings* cookie_content_settings =
+ browser()->profile()->GetCookieContentSettings();
// Check default content settings by using an unknown URL.
GURL example_url("http://www.example.com");
- EXPECT_EQ(CONTENT_SETTING_SESSION_ONLY,
- map->GetCookieContentSetting(
- example_url, example_url, false));
+ EXPECT_TRUE(cookie_content_settings->Allow(example_url, example_url, false));
+ EXPECT_TRUE(cookie_content_settings->EnforceSessionOnly(example_url));
EXPECT_EQ(CONTENT_SETTING_ALLOW,
map->GetContentSetting(example_url,
example_url,
@@ -74,8 +75,7 @@ IN_PROC_BROWSER_TEST_F(ExtensionApiTest, ContentSettings) {
// Check content settings for www.google.com
GURL url("http://www.google.com");
- EXPECT_EQ(CONTENT_SETTING_BLOCK,
- map->GetCookieContentSetting(url, url, false));
+ EXPECT_FALSE(cookie_content_settings->Allow(url, url, false));
EXPECT_EQ(CONTENT_SETTING_ALLOW,
map->GetContentSetting(
url, url, CONTENT_SETTINGS_TYPE_IMAGES, ""));

Powered by Google App Engine
This is Rietveld 408576698