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

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

Issue 8383004: Adding CookieSettings for storing cookie content settings. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixing the rebase. Created 9 years, 2 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_special_storage_policy.cc
diff --git a/chrome/browser/extensions/extension_special_storage_policy.cc b/chrome/browser/extensions/extension_special_storage_policy.cc
index eddaedf74f198b4ba3d0ace1cd4d65386b5bd356..79ef082989d30fd998400e83689bfd826fe1bcc3 100644
--- a/chrome/browser/extensions/extension_special_storage_policy.cc
+++ b/chrome/browser/extensions/extension_special_storage_policy.cc
@@ -6,7 +6,7 @@
#include "base/bind.h"
#include "base/logging.h"
-#include "chrome/browser/content_settings/host_content_settings_map.h"
+#include "chrome/browser/content_settings/cookie_settings.h"
#include "chrome/common/content_settings.h"
#include "chrome/common/content_settings_types.h"
#include "chrome/common/extensions/extension.h"
@@ -14,8 +14,8 @@
#include "content/browser/browser_thread.h"
ExtensionSpecialStoragePolicy::ExtensionSpecialStoragePolicy(
- HostContentSettingsMap* host_content_settings_map)
- : host_content_settings_map_(host_content_settings_map) {}
+ CookieSettings* cookie_settings)
+ : cookie_settings_(cookie_settings) {}
ExtensionSpecialStoragePolicy::~ExtensionSpecialStoragePolicy() {}
@@ -32,22 +32,19 @@ bool ExtensionSpecialStoragePolicy::IsStorageUnlimited(const GURL& origin) {
}
bool ExtensionSpecialStoragePolicy::IsStorageSessionOnly(const GURL& origin) {
- if (host_content_settings_map_ == NULL)
+ if (cookie_settings_ == NULL)
return false;
- ContentSetting content_setting = host_content_settings_map_->
- GetCookieContentSetting(origin, origin, true);
- return (content_setting == CONTENT_SETTING_SESSION_ONLY);
+ return cookie_settings_->IsCookieSessionOnly(origin);
}
bool ExtensionSpecialStoragePolicy::HasSessionOnlyOrigins() {
- if (host_content_settings_map_ == NULL)
+ if (cookie_settings_ == NULL)
return false;
- if (host_content_settings_map_->GetDefaultContentSetting(
- CONTENT_SETTINGS_TYPE_COOKIES, NULL) == CONTENT_SETTING_SESSION_ONLY)
+ if (cookie_settings_->GetDefaultCookieSetting(NULL) ==
+ CONTENT_SETTING_SESSION_ONLY)
return true;
ContentSettingsForOneType entries;
- host_content_settings_map_->GetSettingsForOneType(
- CONTENT_SETTINGS_TYPE_COOKIES, "", &entries);
+ cookie_settings_->GetCookieSettings(&entries);
for (size_t i = 0; i < entries.size(); ++i) {
if (entries[i].setting == CONTENT_SETTING_SESSION_ONLY)
return true;

Powered by Google App Engine
This is Rietveld 408576698