Change net::CookieStore::SetCookiesCallback to return CookieInclusionStatus
Part of an effort to send better cookie information, and more specifically information as to *when* any *why* a cookie has been blocked from sending or storing.
Matt: //net
Maks: //service
Elly: //ios
Jochen: //chrome and //content
Torne: //android_webview
Bug: 856777
Change-Id: Ide937b3d00e0d1aca2d5155c21310fa072b022df
Reviewed-on: https://chromium-review.googlesource.com/c/1470931
Reviewed-by: Matt Menke <[email protected]>
Reviewed-by: Jochen Eisinger <[email protected]>
Reviewed-by: Richard Coles <[email protected]>
Reviewed-by: Maks Orlovich <[email protected]>
Reviewed-by: Elly Fong-Jones <[email protected]>
Commit-Queue: Aaron Tagliaboschi <[email protected]>
Cr-Commit-Position: refs/heads/master@{#634238}diff --git a/net/cookies/cookie_monster.h b/net/cookies/cookie_monster.h
index 151511f5..680df39 100644
--- a/net/cookies/cookie_monster.h
+++ b/net/cookies/cookie_monster.h
@@ -463,12 +463,13 @@
// |creation_date_to_inherit| will be set to that cookie's creation date.
//
// NOTE: There should never be more than a single matching equivalent cookie.
- bool DeleteAnyEquivalentCookie(const std::string& key,
- const CanonicalCookie& ecc,
- bool source_secure,
- bool skip_httponly,
- bool already_expired,
- base::Time* creation_date_to_inherit);
+ CanonicalCookie::CookieInclusionStatus DeleteAnyEquivalentCookie(
+ const std::string& key,
+ const CanonicalCookie& ecc,
+ bool source_secure,
+ bool skip_httponly,
+ bool already_expired,
+ base::Time* creation_date_to_inherit);
// Inserts |cc| into cookies_. Returns an iterator that points to the inserted
// cookie in cookies_. Guarantee: all iterators to cookies_ remain valid.
