Plumb ConnectionAttempts from HttpNetworkTransaction to URLRequest.

net/url_request/url_request_job.h

Index: net/url_request/url_request_job.h
diff --git a/net/url_request/url_request_job.h b/net/url_request/url_request_job.h
index b6ae4bbeb13af7129c5d875c019962cf94b672de..2b2adb66f75761c0d6fc97e25f30b973384f9060 100644
--- a/net/url_request/url_request_job.h
+++ b/net/url_request/url_request_job.h
@@ -19,6 +19,7 @@
 #include "net/base/request_priority.h"
 #include "net/base/upload_progress.h"
 #include "net/cookies/canonical_cookie.h"
+#include "net/socket/connection_attempts.h"

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

Replace with forward decl.

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

Can't, it's a typedef.

 #include "net/url_request/redirect_info.h"
 #include "net/url_request/url_request.h"
 #include "url/gurl.h"
@@ -226,6 +227,8 @@ class NET_EXPORT URLRequestJob
   // canceled by an explicit NetworkDelegate::NotifyURLRequestDestroyed() call.
   virtual void NotifyURLRequestDestroyed();
 
+  virtual void GetConnectionAttempts(ConnectionAttempts* out) const;

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

I think commenting of method functionality is more important in base classes
than usual.  Maybe a note about what this is doing (though that's actually
pretty obvious from the signature) along with when during the job lifetime it's
appropriate to call it or something like that?  (I don't have strong feelings
about what should be in the comment, I just don't want people going "Yuck,
another random base class method I've got no clue about" when they see it.)

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

Done.

+
   // Given |policy|, |referrer|, and |redirect_destination|, returns the
   // referrer URL mandated by |request|'s referrer policy.
   static GURL ComputeReferrerForRedirect(URLRequest::ReferrerPolicy policy,