Plumb ConnectionAttempts from HttpNetworkTransaction to URLRequest.

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 39 matching lines @@
 
   // Overridden from URLRequestJob:
   void SetPriority(RequestPriority priority) override;
   void Start() override;
   void Kill() override;
 
   RequestPriority priority() const {
     return priority_;
   }
 
+  void GetConnectionAttempts(ConnectionAttempts* out) const override;

[Randy Smith (Not in Mondays), 2015/05/01 17:03:44]

IWYU: I think you should forward declare ConnectionAttempts in this file even
though it's clearly not causing compile problems.

[Randy Smith (Not in Mondays), 2015/05/01 17:03:44]

It looks to me as if priority() isn't an override of URLRequestJob, which means
this should be above it with the other overrides from URLRequestJob.

[Deprecated (see juliatuttle), 2015/05/04 19:53:37]

Done.

[Deprecated (see juliatuttle), 2015/05/04 19:53:37]

Done.

+
  private:
   enum CompletionCause {
     ABORTED,
     FINISHED
   };
 
   typedef base::RefCountedData<bool> SharedBoolean;
 
   class HttpFilterContext;
 
@@ skipping 194 matching lines @@
   const HttpUserAgentSettings* http_user_agent_settings_;
 
   base::WeakPtrFactory<URLRequestHttpJob> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(URLRequestHttpJob);
 };
 
 }  // namespace net
 
 #endif  // NET_URL_REQUEST_URL_REQUEST_HTTP_JOB_H_