Make URLRequest::Read to return net errors or bytes read instead of a bool

net/url_request/url_request_test_util.h

Index: net/url_request/url_request_test_util.h
diff --git a/net/url_request/url_request_test_util.h b/net/url_request/url_request_test_util.h
index e7c38b4a149765797157761fc98716b37413ca6d..9e03f999e47eb17bb9be071556de055212fe190e 100644
--- a/net/url_request/url_request_test_util.h
+++ b/net/url_request/url_request_test_util.h
@@ -180,6 +180,7 @@ class TestDelegate : public URLRequest::Delegate {
     return full_request_headers_;
   }
   void ClearFullRequestHeaders();
+  int request_status() const { return request_status_; }
 
   // URLRequest::Delegate:
   void OnReceivedRedirect(URLRequest* request,
@@ -193,6 +194,9 @@ class TestDelegate : public URLRequest::Delegate {
   void OnSSLCertificateError(URLRequest* request,
                              const SSLInfo& ssl_info,
                              bool fatal) override;
+  void OnResponseStarted(URLRequest* request, int net_error) override;
+  // Deprecated.
+  // TODO(maksims): Remove this.
   void OnResponseStarted(URLRequest* request) override;
   void OnReadCompleted(URLRequest* request, int bytes_read) override;
 
@@ -225,6 +229,9 @@ class TestDelegate : public URLRequest::Delegate {
   bool have_full_request_headers_;
   HttpRequestHeaders full_request_headers_;
 
+  // tracks status of request
+  int request_status_;
+
   // our read buffer
   scoped_refptr<IOBuffer> buf_;
 };
@@ -327,10 +334,15 @@ class TestNetworkDelegate : public NetworkDelegateImpl {
       scoped_refptr<HttpResponseHeaders>* override_response_headers,
       GURL* allowed_unsafe_redirect_url) override;
   void OnBeforeRedirect(URLRequest* request, const GURL& new_location) override;
+  void OnResponseStarted(URLRequest* request, int net_error) override;
+  // Deprecated.
   void OnResponseStarted(URLRequest* request) override;
   void OnNetworkBytesReceived(URLRequest* request,
                               int64_t bytes_received) override;
   void OnNetworkBytesSent(URLRequest* request, int64_t bytes_sent) override;
+  void OnCompleted(URLRequest* request, bool started, int net_error) override;
+  // Deprecated.
+  // TODO(maksims): Remove this.
   void OnCompleted(URLRequest* request, bool started) override;
   void OnURLRequestDestroyed(URLRequest* request) override;
   void OnPACScriptError(int line_number, const base::string16& error) override;