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

Unified Diff: chrome/browser/renderer_host/resource_dispatcher_host.cc

Issue 5318002: Also register read cookies in the content settings delegate. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: updates Created 10 years 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: chrome/browser/renderer_host/resource_dispatcher_host.cc
diff --git a/chrome/browser/renderer_host/resource_dispatcher_host.cc b/chrome/browser/renderer_host/resource_dispatcher_host.cc
index 4c6ec1e8ac7012d0b8c954d35ae07151c7cf4bf0..ec284c94afe010aa0c957fce18761ee1cb958301 100644
--- a/chrome/browser/renderer_host/resource_dispatcher_host.cc
+++ b/chrome/browser/renderer_host/resource_dispatcher_host.cc
@@ -1079,6 +1079,30 @@ void ResourceDispatcherHost::OnSSLCertificateError(
SSLManager::OnSSLCertificateError(this, request, cert_error, cert);
}
+void ResourceDispatcherHost::OnGetCookies(
+ net::URLRequest* request,
+ bool blocked_by_policy) {
+ VLOG(1) << "OnGetCookies: " << request->url().spec();
+
+ int render_process_id, render_view_id;
+ if (!RenderViewForRequest(request, &render_process_id, &render_view_id))
+ return;
+
+ ChromeURLRequestContext* context =
+ static_cast<ChromeURLRequestContext*>(request->context());
+ if (context->IsExternal())
+ return;
+
+ net::CookieMonster* cookie_monster =
+ context->cookie_store()->GetCookieMonster();
+ net::CookieList cookie_list =
+ cookie_monster->GetAllCookiesForURL(request->url());
+ CallRenderViewHostContentSettingsDelegate(
+ render_process_id, render_view_id,
+ &RenderViewHostDelegate::ContentSettings::OnCookiesRead,
+ request->url(), cookie_list, blocked_by_policy);
+}
+
void ResourceDispatcherHost::OnSetCookie(net::URLRequest* request,
const std::string& cookie_line,
const net::CookieOptions& options,
@@ -1091,7 +1115,7 @@ void ResourceDispatcherHost::OnSetCookie(net::URLRequest* request,
CallRenderViewHostContentSettingsDelegate(
render_process_id, render_view_id,
- &RenderViewHostDelegate::ContentSettings::OnCookieAccessed,
+ &RenderViewHostDelegate::ContentSettings::OnCookieChanged,
request->url(), cookie_line, options, blocked_by_policy);
}

Powered by Google App Engine
This is Rietveld 408576698