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

net/socket/ssl_client_socket_pool.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_SOCKET_SSL_CLIENT_SOCKET_POOL_H_
 #define NET_SOCKET_SSL_CLIENT_SOCKET_POOL_H_
 
 #include <map>
 #include <string>
 #include <vector>
@@ skipping 243 matching lines @@
   State next_state_;
   CompletionCallback io_callback_;
   scoped_ptr<ClientSocketHandle> transport_socket_handle_;
   scoped_ptr<SSLClientSocket> ssl_socket_;
 
   SSLConnectJobMessenger* messenger_;
   HttpResponseInfo error_response_info_;
 
   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);
 };
 
 class NET_EXPORT_PRIVATE SSLClientSocketPool
     : public ClientSocketPool,
       public HigherLayeredPool,
       public SSLConfigService::Observer {
  public:
@@ skipping 128 matching lines @@
   const scoped_refptr<SSLConfigService> ssl_config_service_;
   MessengerMap messenger_map_;
   bool enable_ssl_connect_job_waiting_;
 
   DISALLOW_COPY_AND_ASSIGN(SSLClientSocketPool);
 };
 
 }  // namespace net
 
 #endif  // NET_SOCKET_SSL_CLIENT_SOCKET_POOL_H_