Plumb connection attempts from (non-proxy) ConnectJobs to HttpNetworkTransaction.

net/http/http_network_transaction.h

Index: net/http/http_network_transaction.h
diff --git a/net/http/http_network_transaction.h b/net/http/http_network_transaction.h
index c098d75888d94b7af1d855eecdd2141fd2117a76..69246d6b597231daa0c56aacec097f02895acf9a 100644
--- a/net/http/http_network_transaction.h
+++ b/net/http/http_network_transaction.h
@@ -99,6 +99,8 @@ class NET_EXPORT_PRIVATE HttpNetworkTransaction
                                   const ProxyInfo& used_proxy_info,
                                   HttpStream* stream) override;
 

[Ryan Hamilton, 2015/03/24 19:20:19]

Probably should have a comment here. The other methods do not because they're
all overrides.

+  const ClientSocketHandle::ConnectionAttempts& connection_attempts() const;

[Ryan Hamilton, 2015/03/24 19:20:19]

This makes me wonder if ConnectionAttempts should be defined in some other .h
file since it's getting exposed up the stack. It doesn't seem like consumers
should care about ClientSocketHandles. maybe just
net/socket/connection_attempts.h ?

[Deprecated (see juliatuttle), 2015/04/08 19:17:19]

Done.

+
  private:
   friend class HttpNetworkTransactionSSLTest;
 
@@ -257,6 +259,8 @@ class NET_EXPORT_PRIVATE HttpNetworkTransaction
 
   void SetStream(HttpStream* stream);
 
+  void CopyConnectionAttempts();
+
   scoped_refptr<HttpAuthController>
       auth_controllers_[HttpAuth::AUTH_NUM_TARGETS];
 
@@ -327,6 +331,8 @@ class NET_EXPORT_PRIVATE HttpNetworkTransaction
   BeforeNetworkStartCallback before_network_start_callback_;
   BeforeProxyHeadersSentCallback before_proxy_headers_sent_callback_;
 
+  ClientSocketHandle::ConnectionAttempts connection_attempts_;
+
   DISALLOW_COPY_AND_ASSIGN(HttpNetworkTransaction);
 };