Port QuicServer to Chrome network stack

net/quic/quic_server_session.h

Index: net/quic/quic_server_session.h
diff --git a/net/quic/quic_server_session.h b/net/quic/quic_server_session.h
index 30ac08c80df802cfe3dea105300121f5e6cfeb3c..15b2ee9fdf742293f002bc48cf426a883d0058c1 100644
--- a/net/quic/quic_server_session.h
+++ b/net/quic/quic_server_session.h
@@ -13,6 +13,7 @@
 #include "base/basictypes.h"
 #include "base/containers/hash_tables.h"
 #include "base/memory/scoped_ptr.h"
+#include "net/quic/quic_connection_packet_writer_wrapper.h"
 #include "net/quic/quic_crypto_server_stream.h"
 #include "net/quic/quic_protocol.h"
 #include "net/quic/quic_session.h"
@@ -43,8 +44,10 @@ class QuicServerSessionVisitor {
 
 class QuicServerSession : public QuicSession {
  public:
+  // Takes ownership of connection_packet_writer

[wtc, 2014/06/20 21:40:50]

This is a strange design. The only reason we're passing
|connection_packet_writer| to the QuicServerSession constructor is for the
QuicServerSession object to own |connection_packet_writer|. The
QuicServerSession object itself doesn't use |connection_packet_writer| at all!

[dmziegler, 2014/06/23 18:31:25]

I will change this in a future CL.

   QuicServerSession(const QuicConfig& config,
                     QuicConnection* connection,
+                    QuicConnectionPacketWriterWrapper* connection_packet_writer,
                     QuicServerSessionVisitor* visitor);
 
   // Override the base class to notify the owner of the connection close.
@@ -77,6 +80,7 @@ class QuicServerSession : public QuicSession {
   friend class test::QuicServerSessionPeer;
 
   scoped_ptr<QuicCryptoServerStream> crypto_stream_;
+  scoped_ptr<QuicConnectionPacketWriterWrapper> connection_packet_writer_;
   QuicServerSessionVisitor* visitor_;
 
   DISALLOW_COPY_AND_ASSIGN(QuicServerSession);