Also register read cookies in the content settings delegate.

chrome/browser/renderer_host/resource_dispatcher_host.cc

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // See http://dev.chromium.org/developers/design-documents/multi-process-resource-loading
 
 #include "chrome/browser/renderer_host/resource_dispatcher_host.h"
 
 #include <vector>
 
@@ skipping 1060 matching lines @@
 }
 
 void ResourceDispatcherHost::OnSSLCertificateError(
     URLRequest* request,
     int cert_error,
     net::X509Certificate* cert) {
   DCHECK(request);
   SSLManager::OnSSLCertificateError(this, request, cert_error, cert);
 }
 
+void ResourceDispatcherHost::OnGetCookies(
+    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::CookieOptions options;
+  options.set_include_httponly();
+  CallRenderViewHostContentSettingsDelegate(
+      render_process_id, render_view_id,
+      &RenderViewHostDelegate::ContentSettings::OnCookiesRead,
+      request->url(), context->cookie_store()->GetCookieMonster(), options,
+      blocked_by_policy);
+}
+
 void ResourceDispatcherHost::OnSetCookie(URLRequest* request,
                                          const std::string& cookie_line,
                                          const net::CookieOptions& options,
                                          bool blocked_by_policy) {
   VLOG(1) << "OnSetCookie: " << request->url().spec();
 
   int render_process_id, render_view_id;
   if (!RenderViewForRequest(request, &render_process_id, &render_view_id))
     return;
 
   CallRenderViewHostContentSettingsDelegate(
       render_process_id, render_view_id,
       &RenderViewHostDelegate::ContentSettings::OnCookieAccessed,

[darin (slow to review), 2010/11/30 18:24:11]

it seems like OnCookieAccessed should be renamed to better reflect the fact that
it is about "mutation" of a cookie whereas OnCookiesRead is about cookies being
read.  OnCookieChanged is probably better than OnCookieAccessed.

[jochen (gone - plz use gerrit), 2010/12/03 16:02:32]

Done.

       request->url(), cookie_line, options, blocked_by_policy);
 }
 
 void ResourceDispatcherHost::OnResponseStarted(URLRequest* request) {
   VLOG(1) << "OnResponseStarted: " << request->url().spec();
   ResourceDispatcherHostRequestInfo* info = InfoForRequest(request);
   if (PauseRequestIfNeeded(info)) {
     VLOG(1) << "OnResponseStarted pausing: " << request->url().spec();
     return;
   }
@@ skipping 828 matching lines @@
   return is_prefetch_enabled_;
 }
 
 // static
 void ResourceDispatcherHost::set_is_prefetch_enabled(bool value) {
   is_prefetch_enabled_ = value;
 }
 
 // static
 bool ResourceDispatcherHost::is_prefetch_enabled_ = false;