Send the ClientHello handshake message. Fix a bug in

net/quic/crypto/crypto_protocol.cc

 // 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.
 
 #include "net/quic/crypto/crypto_protocol.h"
 
 namespace net {
 
 CryptoHandshakeMessage::CryptoHandshakeMessage() {}
 CryptoHandshakeMessage::~CryptoHandshakeMessage() {}
 
+QuicClientHelloConfig::QuicClientHelloConfig() {

[Ryan Hamilton, 2012/12/22 22:31:32]

These seem like sensible values, but I confess, I was surprised to see the
default constructor initialize them to these values.  I might be tempted to have
the default constructor initialize them to 0 or "empty" values, and have a
distinct method to populate them with defaults.

+  // Version must be 0.
+  version = 0;
+
+  // Key exchange methods.
+  key_exchange.resize(2);
+  key_exchange[0] = kC255;
+  key_exchange[1] = kP256;
+
+  // Authenticated encryption algorithms.
+  aead.resize(2);
+  aead[0] = kAESG;
+  aead[1] = kAESH;
+
+  // Congestion control feedback types.
+  congestion_control.resize(1);
+  congestion_control[0] = kQBIC;
+
+  // Idle connection state lifetime.
+  idle_connection_state_lifetime = 300;  // 300 seconds.
+
+  // Keepalive timeout.
+  keepalive_timeout = 0;  // Don't send keepalive probes.
+}
+
 }  // namespace net