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

Unified Diff: net/base/network_delegate.cc

Issue 9572001: Do cookie checks in NetworkDelegate instead of the URLRequest::Delegate. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: TestShellNetworkDelegate Created 8 years, 10 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/network_delegate.cc
diff --git a/net/base/network_delegate.cc b/net/base/network_delegate.cc
index 87ecf39eb3076ebbc55134dab882ef3cb5e6f7b6..6d0b106bfcd6638129cc13d2641e54cfe8ccb97e 100644
--- a/net/base/network_delegate.cc
+++ b/net/base/network_delegate.cc
@@ -90,4 +90,19 @@ NetworkDelegate::AuthRequiredResponse NetworkDelegate::NotifyAuthRequired(
return OnAuthRequired(request, auth_info, callback, credentials);
}
+bool NetworkDelegate::NotifyReadingCookies(
+ const URLRequest* request,
+ const CookieList& cookie_list) {
+ DCHECK(CalledOnValidThread());
willchan no longer on Chromium 2012/03/06 02:59:01 Add DCHECKs like in URLRequest to make sure the ap
jochen (gone - plz use gerrit) 2012/03/08 13:00:02 Done.
+ return CanGetCookies(request, cookie_list);
+}
+
+bool NetworkDelegate::NotifySettingCookie(
+ const URLRequest* request,
+ const std::string& cookie_line,
+ CookieOptions* options) {
+ DCHECK(CalledOnValidThread());
+ return CanSetCookie(request, cookie_line, options);
+}
+
} // namespace net

Powered by Google App Engine
This is Rietveld 408576698