Implements new, more robust design for communicating between SSLConnectJobs.

net/socket/ssl_client_socket_openssl.h

Index: net/socket/ssl_client_socket_openssl.h
diff --git a/net/socket/ssl_client_socket_openssl.h b/net/socket/ssl_client_socket_openssl.h
index 5d70c0523fabe0230b2361f7418688f1215fe0df..7cf975c34d852df26c6d5d925392842b8233f972 100644
--- a/net/socket/ssl_client_socket_openssl.h
+++ b/net/socket/ssl_client_socket_openssl.h
@@ -55,8 +55,15 @@ class SSLClientSocketOpenSSL : public SSLClientSocket {
   const std::string& ssl_session_cache_shard() const {
     return ssl_session_cache_shard_;
   }

[wtc, 2014/07/08 01:25:42]

Rename this method "GetSessionCacheKey". The original name was appropriate when
this was a free function (as opposed to a class method).

[mshelley, 2014/07/09 19:51:00]

Done.

+  std::string GetSocketSessionCacheKey() const;
 
   // SSLClientSocket implementation.
+  virtual bool InSessionCache() const OVERRIDE;
+  virtual void WatchSessionForCompletion(
+      const base::Closure& callback) const OVERRIDE;
+  virtual void SetSocketFailureCallback(const base::Closure& callback) OVERRIDE;
+  virtual void SetIsLeader();

[wtc, 2014/07/08 01:25:42]

Add "OVERRIDE".

[mshelley, 2014/07/09 19:51:00]

Done.

+  virtual void OnSocketFailure() OVERRIDE;
   virtual void GetSSLCertRequestInfo(
       SSLCertRequestInfo* cert_request_info) OVERRIDE;
   virtual NextProtoStatus GetNextProto(std::string* proto,
@@ -153,6 +160,12 @@ class SSLClientSocketOpenSSL : public SSLClientSocket {
   bool transport_recv_busy_;
   bool transport_recv_eof_;
 
+  // True if the socket has been read from before.
+  bool has_read_;
+
+  // True if the socket has been written to before.
+  bool has_written_;
+
   scoped_refptr<DrainableIOBuffer> send_buffer_;
   scoped_refptr<IOBuffer> recv_buffer_;
 
@@ -208,6 +221,12 @@ class SSLClientSocketOpenSSL : public SSLClientSocket {
   // The service for retrieving Channel ID keys.  May be NULL.
   ServerBoundCertService* server_bound_cert_service_;
 
+  // Callback that is invoked when the connection encounters an error.
+  base::Closure error_callback_;
+
+  // True if there may be pending sockets waiting for this socket to connect.
+  bool is_leader_;
+
   // OpenSSL stuff
   SSL* ssl_;
   BIO* transport_bio_;