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

net/socket/ssl_client_socket_pool.h

Index: net/socket/ssl_client_socket_pool.h
diff --git a/net/socket/ssl_client_socket_pool.h b/net/socket/ssl_client_socket_pool.h
index f895d7c42788df8d05ca7e4bbf7d5a053a402956..6ed58ed47aedc22781b691166dcc1150332e22fb 100644
--- a/net/socket/ssl_client_socket_pool.h
+++ b/net/socket/ssl_client_socket_pool.h
@@ -261,6 +261,9 @@ class SSLConnectJob : public ConnectJob {
 
   GetMessengerCallback get_messenger_callback_;
 
+  ClientSocketHandle::ConnectionAttempts connection_attempts_;
+  IPEndPoint direct_address_;

[Ryan Hamilton, 2015/03/17 03:22:28]

Looks like this is the peer address. In that case, I think I'd call it
peer_address_

[Deprecated (see juliatuttle), 2015/03/23 17:45:35]

It's specifically called direct_address_ because it's *only* populated if we
have a direct connection to the host.

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

Hm. I appreciate the distinction you're going for. But I don't think that the
name "direct" help. That sounds like "the host I'm actually talking to via TCP"
which may well be a proxy :/ server_address?

+
   base::WeakPtrFactory<SSLConnectJob> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(SSLConnectJob);