Port QuicServer to Chrome network stack

net/quic/quic_server_packet_writer.h

Index: net/quic/quic_server_packet_writer.h
diff --git a/net/quic/quic_server_packet_writer.h b/net/quic/quic_server_packet_writer.h
index 63829935521019e5712e27410f70bbc38485ff40..c6a5a056add884919e0759751c68b5f26dc66b69 100644
--- a/net/quic/quic_server_packet_writer.h
+++ b/net/quic/quic_server_packet_writer.h
@@ -15,21 +15,30 @@
 
 namespace net {
 
+class QuicDispatcher;

[wtc, 2014/06/18 02:04:47]

Delete the unused forward declaration.

[dmziegler, 2014/06/18 20:13:42]

Done.

 struct WriteResult;

[wtc, 2014/06/18 02:04:47]

I think this forward declaration can be removed. I'd be surprised that a forward
declaration of struct WriteResult is enough. I think this header needs the
definition of this struct.

[dmziegler, 2014/06/18 20:13:41]

Done.

 
+class QuicServerPacketWriterVisitor {
+ public:
+  virtual ~QuicServerPacketWriterVisitor() {}
+
+  virtual void OnCanWrite() {}

[wtc, 2014/06/18 02:04:47]

Document this method.

This method should be abstract (= 0).

[dmziegler, 2014/06/18 20:13:42]

(This got deleted)

+};
+
 // Chrome specific packet writer which uses a UDPServerSocket for writing
 // data.
 class QuicServerPacketWriter : public QuicPacketWriter {
  public:
   QuicServerPacketWriter();
-  explicit QuicServerPacketWriter(UDPServerSocket* socket);
+  QuicServerPacketWriter(QuicServerPacketWriterVisitor* visitor,
+                         UDPServerSocket* socket);
   virtual ~QuicServerPacketWriter();
 
   // QuicPacketWriter
   virtual WriteResult WritePacket(const char* buffer,
                                   size_t buf_len,
-                                  const net::IPAddressNumber& self_address,
-                                  const net::IPEndPoint& peer_address) OVERRIDE;
+                                  const IPAddressNumber& self_address,
+                                  const IPEndPoint& peer_address) OVERRIDE;
   virtual bool IsWriteBlockedDataBuffered() const OVERRIDE;
   virtual bool IsWriteBlocked() const OVERRIDE;
   virtual void SetWritable() OVERRIDE;
@@ -42,6 +51,7 @@ class QuicServerPacketWriter : public QuicPacketWriter {
  private:
   base::WeakPtrFactory<QuicServerPacketWriter> weak_factory_;
   UDPServerSocket* socket_;
+  QuicServerPacketWriterVisitor* visitor_;

[wtc, 2014/06/18 02:04:47]

Nit: the order of these two members should match the order of the equivalent
constructor arguments.

[dmziegler, 2014/06/18 20:13:41]

No longer in the constructor.

   QuicConnection* connection_;
   bool write_blocked_;