Create a visitor which can allow using both a trace... visitor and the

Create a visitor which can allow using both a trace... visitor and the
existing stats visitor.

Changed QuicConnection's QuicConnectionDebugVisitor to a scoped_ptr.

Merge internal change: 71863508

R=rch@chromium.org

[Ryan Hamilton, 2014-07-29 19:16:51]

lgtm

https://codereview.chromium.org/429453003/diff/1/net/quic/quic_client_session.h
File net/quic/quic_client_session.h (right):

https://codereview.chromium.org/429453003/diff/1/net/quic/quic_client_session...
net/quic/quic_client_session.h:239: QuicConnectionLogger* logger_;
I take it this is no longer owned by the client session? Can you add a client
about this?

https://codereview.chromium.org/429453003/diff/1/net/quic/quic_connection_tes...
File net/quic/quic_connection_test.cc (right):

https://codereview.chromium.org/429453003/diff/1/net/quic/quic_connection_tes...
net/quic/quic_connection_test.cc:617: visitor_(),
nit: this shouldn't be required, should it?

[ramant (doing other things), 2014-07-29 21:28:58]

Thanks Ryan. Made the changes we had talked offline.

https://codereview.chromium.org/429453003/diff/1/net/quic/quic_client_session.h
File net/quic/quic_client_session.h (right):

https://codereview.chromium.org/429453003/diff/1/net/quic/quic_client_session...
net/quic/quic_client_session.h:239: QuicConnectionLogger* logger_;
On 2014/07/29 19:16:50, Ryan Hamilton wrote:
> I take it this is no longer owned by the client session? Can you add a client
> about this?

Added a comment.
Done.

https://codereview.chromium.org/429453003/diff/1/net/quic/quic_connection_tes...
File net/quic/quic_connection_test.cc (right):

https://codereview.chromium.org/429453003/diff/1/net/quic/quic_connection_tes...
net/quic/quic_connection_test.cc:617: visitor_(),
On 2014/07/29 19:16:50, Ryan Hamilton wrote:
> nit: this shouldn't be required, should it?

Done.

[Ryan Hamilton, 2014-07-29 22:04:21]

lgtm