This CL changes the lifespan of SSLConnectJobMessengers so that they are created only when needed,

net/socket/ssl_client_socket_pool.cc

Index: net/socket/ssl_client_socket_pool.cc
diff --git a/net/socket/ssl_client_socket_pool.cc b/net/socket/ssl_client_socket_pool.cc
index e54f79c929cee2b2eacb398bfa8896bcc64b212b..b8b09f2db1bb6e2730f48ca25475f2ca026d48a7 100644
--- a/net/socket/ssl_client_socket_pool.cc
+++ b/net/socket/ssl_client_socket_pool.cc
@@ -104,7 +104,12 @@ SSLConnectJobMessenger::SocketAndCallback::SocketAndCallback(
 SSLConnectJobMessenger::SocketAndCallback::~SocketAndCallback() {
 }
 
-SSLConnectJobMessenger::SSLConnectJobMessenger() : weak_factory_(this) {
+SSLConnectJobMessenger::SSLConnectJobMessenger(
+    const ConnectionCompleteCallback& connection_complete_callback,
+    std::string group_name)
+    : group_name_(group_name),
+      weak_factory_(this),
+      connection_complete_callback_(connection_complete_callback) {
 }
 
 SSLConnectJobMessenger::~SSLConnectJobMessenger() {
@@ -125,9 +130,8 @@ void SSLConnectJobMessenger::RemovePendingSocket(SSLClientSocket* ssl_socket) {
 }
 
 bool SSLConnectJobMessenger::CanProceed(SSLClientSocket* ssl_socket) {
-  // If the session is in the session cache, or there are no connecting
-  // sockets, allow the connection to proceed.
-  return ssl_socket->InSessionCache() || connecting_sockets_.empty();
+  // If there are no connecting sockets allow the connection to proceed.
+  return connecting_sockets_.empty();
 }
 
 void SSLConnectJobMessenger::MonitorConnectionResult(
@@ -146,10 +150,17 @@ void SSLConnectJobMessenger::AddPendingSocket(SSLClientSocket* ssl_socket,
 }
 
 void SSLConnectJobMessenger::OnSSLHandshakeCompleted() {
+  SSLClientSocket* ssl_socket = NULL;
+  if (!connecting_sockets_.empty())
+    ssl_socket = connecting_sockets_.front();
   connecting_sockets_.clear();
   SSLPendingSocketsAndCallbacks temp_list;
   temp_list.swap(pending_sockets_and_callbacks_);
   RunAllCallbacks(temp_list);
+  if (ssl_socket) {
+    connection_complete_callback_.Run(ssl_socket->GetSessionCacheKey(),
+                                      group_name_);
+  }
 }
 
 void SSLConnectJobMessenger::RunAllCallbacks(
@@ -175,7 +186,8 @@ SSLConnectJob::SSLConnectJob(const std::string& group_name,
                              ClientSocketFactory* client_socket_factory,
                              HostResolver* host_resolver,
                              const SSLClientSocketContext& context,
-                             SSLConnectJobMessenger* messenger,
+                             UncachedSessionCallback uncached_session_callback,
+                             bool enable_ssl_connect_job_waiting,
                              Delegate* delegate,
                              NetLog* net_log)
     : ConnectJob(group_name,
@@ -198,8 +210,10 @@ SSLConnectJob::SSLConnectJob(const std::string& group_name,
                     : context.ssl_session_cache_shard)),
       io_callback_(
           base::Bind(&SSLConnectJob::OnIOComplete, base::Unretained(this))),
-      messenger_(messenger),
-      weak_factory_(this) {
+      messenger_(NULL),
+      weak_factory_(this),
+      uncached_session_callback_(uncached_session_callback),
+      enable_ssl_connect_job_waiting_(enable_ssl_connect_job_waiting) {
 }
 
 SSLConnectJob::~SSLConnectJob() {
@@ -242,6 +256,11 @@ void SSLConnectJob::GetAdditionalErrorState(ClientSocketHandle* handle) {
     handle->set_is_ssl_error(true);
 }
 
+void SSLConnectJob::RemoveMessenger(const std::string& cache_key) {
+  if (ssl_socket_->GetSessionCacheKey() == cache_key)
+    messenger_ = NULL;
+}
+
 void SSLConnectJob::OnIOComplete(int result) {
   int rv = DoLoop(result);
   if (rv != ERR_IO_PENDING)
@@ -402,23 +421,29 @@ int SSLConnectJob::DoCreateSSLSocket() {
       params_->host_and_port(),
       params_->ssl_config(),
       context_);
+
+  if (!ssl_socket_->InSessionCache() && enable_ssl_connect_job_waiting_)
+    RunUncachedSessionCallback(ssl_socket_->GetSessionCacheKey());
+
   return OK;
 }
 
 int SSLConnectJob::DoCheckForResume() {
   next_state_ = STATE_SSL_CONNECT;
-  if (!messenger_)
+
+  if (ssl_socket_->InSessionCache() || !messenger_)
     return OK;
 
-  // TODO(mshelley): Remove duplicate InSessionCache() calls.
   if (messenger_->CanProceed(ssl_socket_.get())) {
-    if (!ssl_socket_->InSessionCache())
-      messenger_->MonitorConnectionResult(ssl_socket_.get());
+    messenger_->MonitorConnectionResult(ssl_socket_.get());
     return OK;
   }
-  messenger_->AddPendingSocket(ssl_socket_.get(),
-                               base::Bind(&SSLConnectJob::ResumeSSLConnection,
-                                          weak_factory_.GetWeakPtr()));
+
+  messenger_->AddPendingSocket(
+      ssl_socket_.get(),
+      base::Bind(&net::SSLConnectJob::ResumeSSLConnection,

[Ryan Sleevi, 2014/08/12 00:27:42]

no need for net:: here

[mshelley, 2014/08/12 21:47:01]

Done.

+                 weak_factory_.GetWeakPtr()));
+
   return ERR_IO_PENDING;
 }
 
@@ -559,6 +584,10 @@ void SSLConnectJob::ResumeSSLConnection() {
   OnIOComplete(OK);
 }
 
+void SSLConnectJob::RunUncachedSessionCallback(std::string cache_key) {
+  messenger_ = uncached_session_callback_.Run(cache_key, group_name());
+}
+
 SSLConnectJob::State SSLConnectJob::GetInitialState(
     SSLSocketParams::ConnectionType connection_type) {
   switch (connection_type) {
@@ -586,6 +615,7 @@ SSLClientSocketPool::SSLConnectJobFactory::SSLConnectJobFactory(
     HostResolver* host_resolver,
     const SSLClientSocketContext& context,
     bool enable_ssl_connect_job_waiting,
+    SSLConnectJob::UncachedSessionCallback uncached_session_callback,
     NetLog* net_log)
     : transport_pool_(transport_pool),
       socks_pool_(socks_pool),
@@ -594,8 +624,8 @@ SSLClientSocketPool::SSLConnectJobFactory::SSLConnectJobFactory(
       host_resolver_(host_resolver),
       context_(context),
       enable_ssl_connect_job_waiting_(enable_ssl_connect_job_waiting),
-      net_log_(net_log),
-      messenger_map_(new MessengerMap) {
+      uncached_session_callback_(uncached_session_callback),
+      net_log_(net_log) {
   base::TimeDelta max_transport_timeout = base::TimeDelta();
   base::TimeDelta pool_timeout;
   if (transport_pool_)
@@ -615,7 +645,6 @@ SSLClientSocketPool::SSLConnectJobFactory::SSLConnectJobFactory(
 }
 
 SSLClientSocketPool::SSLConnectJobFactory::~SSLConnectJobFactory() {
-  STLDeleteValues(messenger_map_.get());
 }
 
 SSLClientSocketPool::SSLClientSocketPool(
@@ -656,6 +685,8 @@ SSLClientSocketPool::SSLClientSocketPool(
                                        cert_transparency_verifier,
                                        ssl_session_cache_shard),
                 enable_ssl_connect_job_waiting,
+                base::Bind(&SSLClientSocketPool::AddSSLConnectJobMessenger,
+                           base::Unretained(this)),
                 net_log)),
       ssl_config_service_(ssl_config_service) {
   if (ssl_config_service_.get())
@@ -669,45 +700,38 @@ SSLClientSocketPool::SSLClientSocketPool(
 }
 
 SSLClientSocketPool::~SSLClientSocketPool() {
+  for (MessengerMap::iterator it = messenger_map_.begin();
+       it != messenger_map_.end();
+       ++it) {
+    delete it->second;
+  }
   if (ssl_config_service_.get())
     ssl_config_service_->RemoveObserver(this);
 }
 
-scoped_ptr<ConnectJob>
-SSLClientSocketPool::SSLConnectJobFactory::NewConnectJob(
+scoped_ptr<ConnectJob> SSLClientSocketPool::SSLConnectJobFactory::NewConnectJob(
     const std::string& group_name,
     const PoolBase::Request& request,
-    ConnectJob::Delegate* delegate) const {
-  SSLConnectJobMessenger* messenger = NULL;
-  if (enable_ssl_connect_job_waiting_) {
-    std::string cache_key = SSLClientSocket::CreateSessionCacheKey(
-        request.params()->host_and_port(), context_.ssl_session_cache_shard);
-    MessengerMap::const_iterator it = messenger_map_->find(cache_key);
-    if (it == messenger_map_->end()) {
-      std::pair<MessengerMap::iterator, bool> iter = messenger_map_->insert(
-          MessengerMap::value_type(cache_key, new SSLConnectJobMessenger()));
-      it = iter.first;
-    }
-    messenger = it->second;
-  }
-
-  return scoped_ptr<ConnectJob>(new SSLConnectJob(group_name,
-                                                  request.priority(),
-                                                  request.params(),
-                                                  ConnectionTimeout(),
-                                                  transport_pool_,
-                                                  socks_pool_,
-                                                  http_proxy_pool_,
-                                                  client_socket_factory_,
-                                                  host_resolver_,
-                                                  context_,
-                                                  messenger,
-                                                  delegate,
-                                                  net_log_));
-}
-
-base::TimeDelta
-SSLClientSocketPool::SSLConnectJobFactory::ConnectionTimeout() const {
+    ConnectJob::Delegate* delegate) {
+  return scoped_ptr<ConnectJob>(
+      new SSLConnectJob(group_name,
+                        request.priority(),
+                        request.params(),
+                        ConnectionTimeout(),
+                        transport_pool_,
+                        socks_pool_,
+                        http_proxy_pool_,
+                        client_socket_factory_,
+                        host_resolver_,
+                        context_,
+                        uncached_session_callback_,
+                        enable_ssl_connect_job_waiting_,
+                        delegate,
+                        net_log_));
+}
+
+base::TimeDelta SSLClientSocketPool::SSLConnectJobFactory::ConnectionTimeout()
+    const {
   return timeout_;
 }
 
@@ -822,6 +846,31 @@ bool SSLClientSocketPool::CloseOneIdleConnection() {
   return base_.CloseOneIdleConnectionInHigherLayeredPool();
 }
 
+SSLConnectJobMessenger* SSLClientSocketPool::AddSSLConnectJobMessenger(
+    std::string cache_key,
+    std::string group_name) {
+  MessengerMap::const_iterator it = messenger_map_.find(cache_key);
+  if (it == messenger_map_.end()) {
+    std::pair<MessengerMap::iterator, bool> iter =
+        messenger_map_.insert(MessengerMap::value_type(
+            cache_key,
+            new SSLConnectJobMessenger(
+                base::Bind(&SSLClientSocketPool::DeleteSSLConnectJobMessenger,
+                           base::Unretained(this)),
+                group_name)));
+    it = iter.first;
+  }
+  return it->second;
+}
+
+void SSLClientSocketPool::DeleteSSLConnectJobMessenger(std::string cache_key,
+                                                       std::string group_name) {
+  MessengerMap::iterator it = messenger_map_.find(cache_key);
+  delete it->second;
+  messenger_map_.erase(it);
+  base_.RemoveMessengersFromGroup(group_name, cache_key);
+}
+
 void SSLClientSocketPool::OnSSLConfigChanged() {
   FlushWithError(ERR_NETWORK_CHANGED);
 }