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..00a35f75b4a5979d596ead5a5f82a1f175fc2372 100644
--- a/net/socket/ssl_client_socket_openssl.h
+++ b/net/socket/ssl_client_socket_openssl.h
@@ -57,6 +57,9 @@ class SSLClientSocketOpenSSL : public SSLClientSocket {
   }
 
   // SSLClientSocket implementation.
+  virtual bool InSessionCache() const OVERRIDE;
+  virtual void OnSessionComplete(const base::Closure& cb) const OVERRIDE;
+  virtual void OnSocketFailure(const base::Closure& cb) OVERRIDE;

[Ryan Sleevi, 2014/06/26 01:47:16]

For API simplicitly, we may want to combine these into a
base::CompletionCallback, where 0/OK = success, and other statuses indicate an
error.

That said, it's more of a note than a request for change (at present) - still
want to see how things shake out.

[mshelley, 2014/07/01 02:35:22]

Done.

   virtual void GetSSLCertRequestInfo(
       SSLCertRequestInfo* cert_request_info) OVERRIDE;
   virtual NextProtoStatus GetNextProto(std::string* proto,
@@ -208,6 +211,9 @@ 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_;
+
   // OpenSSL stuff
   SSL* ssl_;
   BIO* transport_bio_;