Allow URLRequests from one context to have different NetworkDelegates.

net/url_request/url_request_test_util.h

 // 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.
 
 #ifndef NET_URL_REQUEST_URL_REQUEST_TEST_UTIL_H_
 #define NET_URL_REQUEST_URL_REQUEST_TEST_UTIL_H_
 
 #include <stdlib.h>
 
 #include <map>
@@ skipping 99 matching lines @@
  protected:
   virtual ~TestURLRequestContextGetter();
 
  private:
   const scoped_refptr<base::SingleThreadTaskRunner> network_task_runner_;
   scoped_ptr<TestURLRequestContext> context_;
 };
 
 //-----------------------------------------------------------------------------
 
+// TODO(mmenke):  This class doesn't really seem to serve a purpose.  Get rid
+// of it, and make tests use context->CreateRequest instead.
 class TestURLRequest : public URLRequest {
  public:
   TestURLRequest(const GURL& url,
                  RequestPriority priority,
                  Delegate* delegate,
                  TestURLRequestContext* context);
   virtual ~TestURLRequest();
 };
 
 //-----------------------------------------------------------------------------
@@ skipping 121 matching lines @@
       GURL redirect_on_headers_received_url) {
     redirect_on_headers_received_url_ = redirect_on_headers_received_url;
   }
 
   void set_allowed_unsafe_redirect_url(GURL allowed_unsafe_redirect_url) {
     allowed_unsafe_redirect_url_ = allowed_unsafe_redirect_url;
   }
 
   void set_cookie_options(int o) {cookie_options_bit_mask_ = o; }
 
+  size_t total_seen_requests() const { return event_order_.size(); }

[pauljensen, 2014/07/31 15:38:17]

I didn't see any caller of this function nor a use of the next two changes.  Am
I missing something?

[mmenke, 2014/07/31 16:03:14]

No, you're not.  These accidentally slipped in from the that creates the new
wrapper job class.  I've removed these changes from this CL.

   int last_error() const { return last_error_; }
   int error_count() const { return error_count_; }
   int created_requests() const { return created_requests_; }
+  int sent_headers_for_requests() const { return sent_headers_for_requests_; }
+  int redirected_requests() const { return redirected_requests_; }
   int destroyed_requests() const { return destroyed_requests_; }
   int completed_requests() const { return completed_requests_; }
   int canceled_requests() const { return canceled_requests_; }
   int blocked_get_cookies_count() const { return blocked_get_cookies_count_; }
   int blocked_set_cookie_count() const { return blocked_set_cookie_count_; }
   int set_cookie_count() const { return set_cookie_count_; }
 
   void set_can_access_files(bool val) { can_access_files_ = val; }
   bool can_access_files() const { return can_access_files_; }
 
@@ skipping 58 matching lines @@
 
   void InitRequestStatesIfNew(int request_id);
 
   GURL redirect_on_headers_received_url_;
   // URL marked as safe for redirection at the onHeadersReceived stage.
   GURL allowed_unsafe_redirect_url_;
 
   int last_error_;
   int error_count_;
   int created_requests_;
+  int sent_headers_for_requests_;
+  // Number of requests for which OnBeforeRedirect is called.  Some redirects
+  // may not have been followed.
+  int redirected_requests_;
   int destroyed_requests_;
   int completed_requests_;
   int canceled_requests_;
   int cookie_options_bit_mask_;
   int blocked_get_cookies_count_;
   int blocked_set_cookie_count_;
   int set_cookie_count_;
   int observed_before_proxy_headers_sent_callbacks_;
   // Last observed proxy in before proxy header sent callback.
   HostPortPair last_observed_proxy_;
@@ skipping 55 matching lines @@
       NetworkDelegate* network_delegate) const OVERRIDE;
   void set_main_intercept_job(URLRequestJob* job);
 
  private:
   mutable URLRequestJob* main_intercept_job_;
 };
 
 }  // namespace net
 
 #endif  // NET_URL_REQUEST_URL_REQUEST_TEST_UTIL_H_