Do cookie checks in NetworkDelegate instead of the URLRequest::Delegate.

net/proxy/network_delegate_error_observer_unittest.cc

 // Copyright (c) 2012 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/proxy/network_delegate_error_observer.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/message_loop_proxy.h"
 #include "base/threading/thread.h"
@@ skipping 45 matching lines @@
                                 const string16& error) OVERRIDE {
     got_pac_error_ = true;
   }
   virtual AuthRequiredResponse OnAuthRequired(
       URLRequest* request,
       const AuthChallengeInfo& auth_info,
       const AuthCallback& callback,
       AuthCredentials* credentials) OVERRIDE {
     return AUTH_REQUIRED_RESPONSE_NO_ACTION;
   }
+  virtual bool OnReadingCookies(
+      const URLRequest* request,
+      const CookieList& cookie_list) {

[willchan no longer on Chromium, 2012/03/01 20:30:10]

OVERRIDE

+    return true;
+  }
+  virtual bool OnSettingCookie(
+      const URLRequest* request,
+      const std::string& cookie_line,
+      CookieOptions* options) {

[willchan no longer on Chromium, 2012/03/01 20:30:10]

OVERRIDE

+    return true;
+  }
 
   bool got_pac_error_;
 };
 
 }  // namespace
 
 // Check that the OnPACScriptError method can be called from an arbitrary
 // thread.
 TEST(NetworkDelegateErrorObserverTest, CallOnThread) {
   base::Thread thread("test_thread");
@@ skipping 20 matching lines @@
   thread.message_loop()->PostTask(
       FROM_HERE,
       base::Bind(&NetworkDelegateErrorObserver::OnPACScriptError,
                  base::Unretained(&observer), 42, string16()));
   thread.Stop();
   MessageLoop::current()->RunAllPending();
   // Shouldn't have crashed until here...
 }
 
 }  // namespace net