Send the ClientHello handshake message. Fix a bug in

net/quic/quic_client_session.cc

Index: net/quic/quic_client_session.cc
===================================================================
--- net/quic/quic_client_session.cc	(revision 173961)
+++ net/quic/quic_client_session.cc	(working copy)
@@ -53,9 +53,9 @@
 };
 
 int QuicClientSession::CryptoConnect(const CompletionCallback& callback) {
-  CryptoHandshakeMessage message;
-  message.tag = kCHLO;
-  crypto_stream_.SendHandshakeMessage(message);
+  if (!crypto_stream_.CryptoConnect()) {
+    return ERR_FAILED;  // TODO(wtc): use a better error code.
+  }

[wtc, 2012/12/20 00:33:08]

Do you think |callback| should be passed along to crypto_stream_?

[Ryan Hamilton, 2012/12/20 18:51:06]

Can we avoid doing so?  The internal code does not use CompletionCallbacks.

 
   if (IsCryptoHandshakeComplete()) {
     return OK;