Makes waiting SSLConnectJobs use the message loops to resume their connection.

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>
 
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/scoped_vector.h"
+#include "base/thread_task_runner_handle.h"

[Ryan Sleevi, 2014/07/10 00:03:22]

This should be in the .cc, not the .h

In .h files, we want to keep the number of headers included as small as
possible.

[mshelley, 2014/07/10 00:44:20]

Done.

 #include "base/time/time.h"
 #include "net/base/privacy_mode.h"
 #include "net/dns/host_resolver.h"
 #include "net/http/http_response_info.h"
 #include "net/socket/client_socket_pool.h"
 #include "net/socket/client_socket_pool_base.h"
 #include "net/socket/client_socket_pool_histograms.h"
 #include "net/socket/ssl_client_socket.h"
 #include "net/ssl/ssl_config_service.h"
 
@@ skipping 84 matching lines @@
     SSLClientSocket* socket;
     base::Closure callback;
 
     SocketAndCallback(SSLClientSocket* ssl_socket,
                       base::Closure job_resumption_callback)
         : socket(ssl_socket), callback(job_resumption_callback) {}
   };
 
   typedef std::vector<SocketAndCallback> SSLPendingSocketsAndCallbacks;
 
+  SSLConnectJobMessenger();
+
   // Returns true if the given |ssl_socket| should continue its
   // SSL connection.
   bool CanProceed(SSLClientSocket* ssl_socket);
 
   // Informs the session cache that it should notify the SSLConnectJobMessenger
   // upon the completion of |ssl_socket|'s connection.
   void MonitorConnectionResult(SSLClientSocket* ssl_socket);
 
   // Adds the socket and its associated callback to the SSLConnectJobMessenger's
   // list of pending sockets and callbacks.
   void AddPendingSocket(SSLClientSocket* socket, const base::Closure& callback);
 
   // Processes pending callbacks when a socket successfully completes
   // its connection.
   void OnJobSucceeded();
 
-  // Processes pending callbacks when a socket encounters an error
+  // Posts a task to process pending callbacks when a socket encounters an error
   // while completing its connection.
   void OnJobFailed();
 
+  // Determines which pending socket should be then next leading connection,
+  // and runs that socket's resumption callback.
+  void ConnectNewLeader();
+
  private:
   // Runs all callbacks stored in |pending_sockets_and_callbacks_|.
-  void RunAllJobs(std::vector<SocketAndCallback>& pending_socket_and_callbacks);
+  void RunAllJobs(
+      std::vector<SocketAndCallback>& pending_sockets_and_callbacks);
 
+  base::WeakPtrFactory<SSLConnectJobMessenger> weak_factory_;
   SSLPendingSocketsAndCallbacks pending_sockets_and_callbacks_;
   std::vector<SSLClientSocket*> connecting_sockets_;
 };
 
 // SSLConnectJob handles the SSL handshake after setting up the underlying
 // connection as specified in the params.
 class SSLConnectJob : public ConnectJob {
  public:
   // Note: the SSLConnectJob does not own |messenger|.
   // so it must outlive the job.
@@ skipping 214 matching lines @@
   HttpProxyClientSocketPool* const http_proxy_pool_;
   PoolBase base_;
   const scoped_refptr<SSLConfigService> ssl_config_service_;
 
   DISALLOW_COPY_AND_ASSIGN(SSLClientSocketPool);
 };
 
 }  // namespace net
 
 #endif  // NET_SOCKET_SSL_CLIENT_SOCKET_POOL_H_