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

Unified Diff: net/cookies/canonical_cookie.cc

Issue 11308272: Add IncludeForRequestURL method to CanonicalCookie (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 1 month 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/cookies/canonical_cookie.cc
diff --git a/net/cookies/canonical_cookie.cc b/net/cookies/canonical_cookie.cc
index 83e0e6023f753015bea039c253ddee8d88d5495e..b5f9e2e89cee20d2cbc311a23f969259b3d44728 100644
--- a/net/cookies/canonical_cookie.cc
+++ b/net/cookies/canonical_cookie.cc
@@ -371,6 +371,21 @@ bool CanonicalCookie::IsDomainMatch(const std::string& host) const {
domain_.length(), domain_) == 0);
}
+bool CanonicalCookie::IncludeForRequest(const GURL& url) const {
+ // Secure cookies should not be included in requests for URLs with an
+ // unsecure scheme.
+ if (IsSecure() && !url.SchemeIsSecure())
+ return false;
+ // Don't include cookies for requests that don't apply to the cookie domain.
+ if (!IsDomainMatch(url.host()))
+ return false;
+ // Don't include cookies for requests that
+ if (!IsOnPath(url.path()))
+ return false;
+
+ return true;
+}
+
std::string CanonicalCookie::DebugString() const {
return base::StringPrintf(
"name: %s value: %s domain: %s path: %s creation: %"

Powered by Google App Engine
This is Rietveld 408576698