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

Unified Diff: net/url_request/url_request.h

Issue 6995013: More progress towards removing content settings code from the content layer. We can't use Cookie... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: fix net_unittests Created 9 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
Index: net/url_request/url_request.h
===================================================================
--- net/url_request/url_request.h (revision 84586)
+++ net/url_request/url_request.h (working copy)
@@ -188,18 +188,17 @@
int cert_error,
X509Certificate* cert);
- // Called when reading cookies. |blocked_by_policy| is true if access to
- // cookies was denied due to content settings. This method will never be
- // invoked when LOAD_DO_NOT_SEND_COOKIES is specified.
- virtual void OnGetCookies(URLRequest* request, bool blocked_by_policy);
+ // Called when reading cookies to allow the delegate to block access to the
+ // cookie. This method will never be invoked when LOAD_DO_NOT_SEND_COOKIES
+ // is specified.
+ virtual bool CanGetCookies(URLRequest* request);
- // Called when a cookie is set. |blocked_by_policy| is true if the cookie
- // was rejected due to content settings. This method will never be invoked
- // when LOAD_DO_NOT_SAVE_COOKIES is specified.
- virtual void OnSetCookie(URLRequest* request,
- const std::string& cookie_line,
- const CookieOptions& options,
- bool blocked_by_policy);
+ // Called when a cookie is set to allow the delegate to block access to the
+ // cookie. This method will never be invoked when LOAD_DO_NOT_SAVE_COOKIES
+ // is specified.
+ virtual bool CanSetCookie(URLRequest* request,
+ const std::string& cookie_line,
+ CookieOptions* options);
// After calling Start(), the delegate will receive an OnResponseStarted
// callback when the request has completed. If an error occurred, the

Powered by Google App Engine
This is Rietveld 408576698