Update TestURLRequest constructor interface in unit tests.

net/url_request/url_request.h

Index: net/url_request/url_request.h
diff --git a/net/url_request/url_request.h b/net/url_request/url_request.h
index 5580b63eef8925633345c155f356b403b120a26d..8da9dcc7626aa25853d61105c2631e7a5bebe3e7 100644
--- a/net/url_request/url_request.h
+++ b/net/url_request/url_request.h
@@ -648,6 +648,9 @@ class NET_EXPORT URLRequest : NON_EXPORTED_BASE(public base::NonThreadSafe),
   // Allow the URLRequestJob class to set our status too
   void set_status(const URLRequestStatus& value) { status_ = value; }
 
+  // Provide derived classes with read-only access to the network delegate.
+  NetworkDelegate* network_delegate() { return network_delegate_; }

[mmenke, 2013/03/10 20:40:56]

I don't think we need this.  The one class that uses it can just cache the value
from its constructor.

As the main entry point to the network stack, I prefer not to expand the
interface except when needed.

These functions are mostly for URLRequestJobs, which have their own copy of the
delegate pointer.  I don't want to introduce ambiguities as to how they should
access the delegate.

+
   // Allow the URLRequestJob to redirect this request.  Returns OK if
   // successful, otherwise an error code is returned.
   int Redirect(const GURL& location, int http_status_code);