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

Unified Diff: net/base/static_cookie_policy.cc

Issue 2655443003: Unify the "get" and "set" cookie access settings. (Closed)
Patch Set: fix android Created 3 years, 11 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: net/base/static_cookie_policy.cc
diff --git a/net/base/static_cookie_policy.cc b/net/base/static_cookie_policy.cc
index 021cacef4cae27310773b60e0e5ca1351d4e51ae..7d327c9c0d75cfbf14f5a46172df9baa2b08beb3 100644
--- a/net/base/static_cookie_policy.cc
+++ b/net/base/static_cookie_policy.cc
@@ -11,29 +11,7 @@
namespace net {
-int StaticCookiePolicy::CanGetCookies(
- const GURL& url,
- const GURL& first_party_for_cookies) const {
- switch (type_) {
- case StaticCookiePolicy::ALLOW_ALL_COOKIES:
- return OK;
- 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 registry_controlled_domains::SameDomainOrHost(
- url,
- first_party_for_cookies,
- registry_controlled_domains::INCLUDE_PRIVATE_REGISTRIES) ?
- OK : ERR_ACCESS_DENIED;
- case StaticCookiePolicy::BLOCK_ALL_COOKIES:
- return ERR_ACCESS_DENIED;
- default:
- NOTREACHED();
- return ERR_ACCESS_DENIED;
- }
-}
-
-int StaticCookiePolicy::CanSetCookie(
+int StaticCookiePolicy::CanAccessCookies(
const GURL& url,
const GURL& first_party_for_cookies) const {
switch (type_) {

Powered by Google App Engine
This is Rietveld 408576698