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

Unified Diff: net/base/static_cookie_policy.cc

Issue 15140003: Add support for split Public Suffix List distinctions. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebased again Created 7 years, 7 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
« no previous file with comments | « net/base/sdch_manager.cc ('k') | net/cookies/cookie_monster.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/base/static_cookie_policy.cc
diff --git a/net/base/static_cookie_policy.cc b/net/base/static_cookie_policy.cc
index 55f45306d00d62e89edf57afe5d6d6fe105953bc..f7ac065ea7c1918690551be94829c8859ae26fd9 100644
--- a/net/base/static_cookie_policy.cc
+++ b/net/base/static_cookie_policy.cc
@@ -21,8 +21,11 @@ int StaticCookiePolicy::CanGetCookies(
case StaticCookiePolicy::BLOCK_ALL_THIRD_PARTY_COOKIES:
if (first_party_for_cookies.is_empty())
return OK; // Empty first-party URL indicates a first-party request.
- return RegistryControlledDomainService::SameDomainOrHost(
- url, first_party_for_cookies) ? OK : ERR_ACCESS_DENIED;
+ return registry_controlled_domains::SameDomainOrHost(
+ url,
+ first_party_for_cookies,
+ registry_controlled_domains::EXCLUDE_PRIVATE_REGISTRIES) ?
+ OK : ERR_ACCESS_DENIED;
case StaticCookiePolicy::BLOCK_ALL_COOKIES:
return ERR_ACCESS_DENIED;
default:
@@ -41,8 +44,11 @@ int StaticCookiePolicy::CanSetCookie(
case StaticCookiePolicy::BLOCK_ALL_THIRD_PARTY_COOKIES:
if (first_party_for_cookies.is_empty())
return OK; // Empty first-party URL indicates a first-party request.
- return RegistryControlledDomainService::SameDomainOrHost(
- url, first_party_for_cookies) ? OK : ERR_ACCESS_DENIED;
+ return registry_controlled_domains::SameDomainOrHost(
+ url,
+ first_party_for_cookies,
+ registry_controlled_domains::EXCLUDE_PRIVATE_REGISTRIES) ?
+ OK : ERR_ACCESS_DENIED;
case StaticCookiePolicy::BLOCK_ALL_COOKIES:
return ERR_ACCESS_DENIED;
default:
« no previous file with comments | « net/base/sdch_manager.cc ('k') | net/cookies/cookie_monster.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698