Revert 153131 (Histograms showed it doesn't help much)

net/url_request/url_request_http_job.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_HTTP_JOB_H_
 #define NET_URL_REQUEST_URL_REQUEST_HTTP_JOB_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 47 matching lines @@
   void ProcessPublicKeyPinsHeader();
 
   // |result| should be net::OK, or the request is canceled.
   void OnHeadersReceivedCallback(int result);
   void OnStartCompleted(int result);
   void OnReadCompleted(int result);
   void NotifyBeforeSendHeadersCallback(int result);
 
   void RestartTransactionWithAuth(const AuthCredentials& credentials);
 
-  void RetryRequestOnError(int result);
-
-  // Returns true if this request should be retried after receiving the given
-  // error as a result of a call to Start.
-  bool ShouldRetryRequest(int result) const;
-
   // Overridden from URLRequestJob:
   virtual void SetUpload(UploadData* upload) OVERRIDE;
   virtual void SetExtraRequestHeaders(
       const HttpRequestHeaders& headers) OVERRIDE;
   virtual void Start() OVERRIDE;
   virtual void Kill() OVERRIDE;
   virtual LoadState GetLoadState() const OVERRIDE;
   virtual UploadProgress GetUploadProgress() const OVERRIDE;
   virtual bool GetMimeType(std::string* mime_type) const OVERRIDE;
   virtual bool GetCharset(std::string* charset) OVERRIDE;
@@ skipping 65 matching lines @@
   typedef base::RefCountedData<bool> SharedBoolean;
 
   class HttpFilterContext;
   class HttpTransactionDelegateImpl;
 
   virtual ~URLRequestHttpJob();
 
   void RecordTimer();
   void ResetTimer();
 
-  void RecordRetryResult(int result) const;
-
   virtual void UpdatePacketReadTimes() OVERRIDE;
   void RecordPacketStats(FilterContext::StatisticSelector statistic) const;
 
   void RecordCompressionHistograms();
   bool IsCompressibleContent() const;
 
   // Starts the transaction if extensions using the webrequest API do not
   // object.
   void StartTransaction();
   void StartTransactionInternal();
@@ skipping 71 matching lines @@
 
   // Flag used to verify that |this| is not deleted while we are awaiting
   // a callback from the NetworkDelegate. Used as a fail-fast mechanism.
   // True if we are waiting a callback and
   // NetworkDelegate::NotifyURLRequestDestroyed has not been called, yet,
   // to inform the NetworkDelegate that it may not call back.
   bool awaiting_callback_;
 
   scoped_ptr<HttpTransactionDelegateImpl> http_transaction_delegate_;
 
-  // True if the request job has automatically retried the request as a result
-  // of an error.
-  bool has_retried_;
-
-  // The network error code error that |this| was automatically retried as a
-  // result of, if any.  If this request has not been retried, must be 0.
-  // Reset once it's been used to log histograms.
-  int error_before_retry_;
-
   DISALLOW_COPY_AND_ASSIGN(URLRequestHttpJob);
 };
 
 }  // namespace net
 
 #endif  // NET_URL_REQUEST_URL_REQUEST_HTTP_JOB_H_