Send the ClientHello handshake message. Fix a bug in

net/quic/test_tools/quic_test_utils.h

Index: net/quic/test_tools/quic_test_utils.h
===================================================================
--- net/quic/test_tools/quic_test_utils.h	(revision 174485)
+++ net/quic/test_tools/quic_test_utils.h	(working copy)
@@ -32,6 +32,11 @@
 // Constructs a basic crypto handshake message
 QuicPacket* ConstructHandshakePacket(QuicGuid guid, CryptoTag tag);
 
+// Constructs a ClientHello crypto handshake message
+QuicPacket* ConstructClientHelloPacket(QuicGuid guid,
+                                       const QuicClock* clock,
+                                       QuicRandom* random_generator);
+
 class MockFramerVisitor : public QuicFramerVisitorInterface {
  public:
   MockFramerVisitor();
@@ -148,6 +153,8 @@
 class MockConnection : public QuicConnection {
  public:
   MockConnection(QuicGuid guid, IPEndPoint address);
+  MockConnection(QuicGuid guid, IPEndPoint address,
+                 QuicConnectionHelperInterface* helper);

[wtc, 2013/01/15 23:11:03]

This formatting is allowed by the current Style Guide. If
you prefer one parameter per line, please let me know.

[Ryan Hamilton, 2013/01/16 00:29:36]

Are you sure?  jar complains about this every time. 

[wtc, 2013/01/16 00:43:49]

Yes, I am sure. See the second code example at
http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml#Function_Decla...

  If you have too much text to fit on one line:

    ReturnType ClassName::ReallyLongFunctionName(Type par_name1, Type par_name2,
                                                 Type par_name3) {
      DoSomething();
      ...
    }

I changed the formatting to preserve the old style.

   virtual ~MockConnection();
 
   MOCK_METHOD3(ProcessUdpPacket, void(const IPEndPoint& self_address,
@@ -197,8 +204,6 @@
                ReliableQuicStream*(QuicStreamId id));
   MOCK_METHOD0(GetCryptoStream, QuicCryptoStream*());
   MOCK_METHOD0(CreateOutgoingReliableStream, ReliableQuicStream*());
-  MOCK_METHOD3(WriteData,
-               void(QuicStreamId id, base::StringPiece data, bool fin));
   MOCK_METHOD4(WriteData, int(QuicStreamId id, base::StringPiece data,
                               QuicStreamOffset offset, bool fin));
   MOCK_METHOD0(IsHandshakeComplete, bool());