Also register read cookies in the content settings delegate.

net/url_request/url_request_http_job.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.
 
 #include "net/url_request/url_request_http_job.h"
 
 #include "base/base_switches.h"
 #include "base/command_line.h"
 #include "base/compiler_specific.h"
 #include "base/file_util.h"
@@ skipping 426 matching lines @@
 }
 
 void URLRequestHttpJob::StopCaching() {
   if (transaction_.get())
     transaction_->StopCaching();
 }
 
 void URLRequestHttpJob::OnCanGetCookiesCompleted(int policy) {
   // If the request was destroyed, then there is no more work to do.
   if (request_ && request_->delegate()) {
-    if (policy == net::ERR_ACCESS_DENIED) {
+    if (policy == net::ERR_ACCESS_DENIED &&

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

style-nit: it seems like an outer if (request_->context()->cookie_store()) { ...
} wrapper would be better since both branches have the same condition.

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

Done.

+        request_->context()->cookie_store()) {
       request_->delegate()->OnGetCookies(request_, true);
     } else if (policy == net::OK && request_->context()->cookie_store()) {
       request_->delegate()->OnGetCookies(request_, false);
       net::CookieOptions options;
       options.set_include_httponly();
       std::string cookies =
           request_->context()->cookie_store()->GetCookiesWithOptions(
               request_->url(), options);
       if (!cookies.empty()) {
         request_info_.extra_headers.SetHeader(
             net::HttpRequestHeaders::kCookie, cookies);
       }
     }
     // We may have been canceled within OnGetCookies.
     if (GetStatus().is_success()) {
       StartTransaction();
     } else {
       NotifyCanceled();
     }
   }
   Release();  // Balance AddRef taken in AddCookieHeaderAndStart
 }
 
 void URLRequestHttpJob::OnCanSetCookieCompleted(int policy) {
   // If the request was destroyed, then there is no more work to do.
   if (request_ && request_->delegate()) {
     if (policy == net::ERR_ACCESS_DENIED) {

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

why isn't OnSetCookie similarly disallowed when the cookie_store is null?

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

Done.

       request_->delegate()->OnSetCookie(
           request_,
           response_cookies_[response_cookies_save_index_],
           net::CookieOptions(),
           true);
     } else if ((policy == net::OK || policy == net::OK_FOR_SESSION_ONLY) &&
                request_->context()->cookie_store()) {
       // OK to save the current response cookie now.
       net::CookieOptions options;
       options.set_include_httponly();
@@ skipping 449 matching lines @@
     HTTPSProberDelegate* delegate =
         new HTTPSProberDelegate(request_info_.url.host(), max_age,
                                 include_subdomains,
                                 ctx->transport_security_state());
     if (!prober->ProbeHost(request_info_.url.host(), request()->context(),
                            delegate)) {
       delete delegate;
     }
   }
 }