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

net/base/network_delegate_impl.h

 // Copyright 2014 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_BASE_NETWORK_DELEGATE_IMPL_H_
 #define NET_BASE_NETWORK_DELEGATE_IMPL_H_
 
 #include <stdint.h>
 
 #include "base/strings/string16.h"
@@ skipping 86 matching lines @@
       const HttpResponseHeaders* original_response_headers,
       scoped_refptr<HttpResponseHeaders>* override_response_headers,
       GURL* allowed_unsafe_redirect_url) override;
 
   // Called right after a redirect response code was received.
   // |new_location| is only valid until OnURLRequestDestroyed is called for this
   // request.
   void OnBeforeRedirect(URLRequest* request, const GURL& new_location) override;
 
   // This corresponds to URLRequestDelegate::OnResponseStarted.
+  void OnResponseStarted(URLRequest* request, int net_error) override;
+  // Deprecated.
+  // TODO(maksims): Remove this;
   void OnResponseStarted(URLRequest* request) override;
 
   // Called when bytes are received from the network, such as after receiving
   // headers or reading raw response bytes. This includes localhost requests.
   // |bytes_received| is the number of bytes measured at the application layer
   // that have been received over the network for this request since the last
   // time OnNetworkBytesReceived was called. |bytes_received| will always be
   // greater than 0.
   // Currently, this is only implemented for HTTP transactions, and
   // |bytes_received| does not include TLS overhead or TCP retransmits.
   void OnNetworkBytesReceived(URLRequest* request,
                               int64_t bytes_received) override;
 
   // Called when bytes are sent over the network, such as when sending request
   // headers or uploading request body bytes. This includes localhost requests.
   // |bytes_sent| is the number of bytes measured at the application layer that
   // have been sent over the network for this request since the last time
   // OnNetworkBytesSent was called. |bytes_sent| will always be greater than 0.
   // Currently, this is only implemented for HTTP transactions, and |bytes_sent|
   // does not include TLS overhead or TCP retransmits.
   void OnNetworkBytesSent(URLRequest* request, int64_t bytes_sent) override;
 
   // Indicates that the URL request has been completed or failed.
   // |started| indicates whether the request has been started. If false,
   // some information like the socket address is not available.
+  void OnCompleted(URLRequest* request, bool started, int net_error) override;
+  // Deprecated.
+  // TODO(maksims): Remove this;
   void OnCompleted(URLRequest* request, bool started) override;
 
   // Called when an URLRequest is being destroyed. Note that the request is
   // being deleted, so it's not safe to call any methods that may result in
   // a virtual method call.
   void OnURLRequestDestroyed(URLRequest* request) override;
 
   // Corresponds to ProxyResolverJSBindings::OnError.
   void OnPACScriptError(int line_number, const base::string16& error) override;
 
@@ skipping 59 matching lines @@
   // header is stripped from the request.
   bool OnCancelURLRequestWithPolicyViolatingReferrerHeader(
       const URLRequest& request,
       const GURL& target_url,
       const GURL& referrer_url) const override;
 };
 
 }  // namespace net
 
 #endif  // NET_BASE_NETWORK_DELEGATE_IMPL_H_