Enhance net internals/net log output for QUIC.

net/quic/quic_reliable_client_stream_test.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/quic_reliable_client_stream.h"
 
 #include "net/base/net_errors.h"
 #include "net/quic/quic_client_session.h"
 #include "net/quic/quic_utils.h"
 #include "net/quic/test_tools/quic_test_utils.h"
@@ skipping 17 matching lines @@
   MOCK_METHOD1(OnError, void(int));
 
  private:
   DISALLOW_COPY_AND_ASSIGN(MockDelegate);
 };
 
 class QuicReliableClientStreamTest : public ::testing::Test {
  public:
   QuicReliableClientStreamTest()
       : session_(new MockConnection(1, IPEndPoint()), false),
-        stream_(1, &session_) {
+        stream_(1, &session_, net_log_) {

[eroman, 2013/02/05 23:17:30]

If the test doesn't use it, you can also pass in BoundNetLog() directly, and
avoid adding a new member to the test fixture.

(Since this isn't tied to any actual NetLog, it won't actually be doing any
logging).

If you wanted to tied it to something you could use CapturingBoundNetLog

[Ryan Hamilton, 2013/02/05 23:35:29]

Done.

     stream_.SetDelegate(&delegate_);
   }
 
   testing::StrictMock<MockDelegate> delegate_;
+  BoundNetLog net_log_;
   MockSession session_;
   QuicReliableClientStream stream_;
 };
 
 TEST_F(QuicReliableClientStreamTest, TerminateFromPeer) {
   EXPECT_CALL(delegate_, OnClose(QUIC_NO_ERROR));
 
   stream_.TerminateFromPeer(true);
 }
 
@@ skipping 19 matching lines @@
 TEST_F(QuicReliableClientStreamTest, OnError) {
   EXPECT_CALL(delegate_, OnError(ERR_INTERNET_DISCONNECTED));
 
   stream_.OnError(ERR_INTERNET_DISCONNECTED);
   EXPECT_FALSE(stream_.GetDelegate());
 }
 
 }  // namespace
 }  // namespace test
 }  // namespace net