Port QuicServer to Chrome network stack

net/quic/quic_server_packet_writer.cc

 // Copyright 2014 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_server_packet_writer.h"
 
 #include "base/location.h"
 #include "base/logging.h"
 #include "base/metrics/sparse_histogram.h"
 #include "net/base/io_buffer.h"
 #include "net/base/net_errors.h"
+#include "net/quic/quic_dispatcher.h"

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

Remove this header?

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

Done.

 
 namespace net {
 
 QuicServerPacketWriter::QuicServerPacketWriter() : weak_factory_(this) {
 }
 
-QuicServerPacketWriter::QuicServerPacketWriter(UDPServerSocket* socket)
+QuicServerPacketWriter::QuicServerPacketWriter(
+    QuicServerPacketWriterVisitor* visitor,
+    UDPServerSocket* socket)
     : weak_factory_(this),
       socket_(socket),
+      visitor_(visitor),
       write_blocked_(false) {
 }
 
 QuicServerPacketWriter::~QuicServerPacketWriter() {
 }
 
 WriteResult QuicServerPacketWriter::WritePacket(
     const char* buffer, size_t buf_len,
     const net::IPAddressNumber& self_address,
     const net::IPEndPoint& peer_address) {
@@ skipping 31 matching lines @@
 void QuicServerPacketWriter::SetWritable() {
   write_blocked_ = false;
 }
 
 void QuicServerPacketWriter::OnWriteComplete(int rv) {
   DCHECK_NE(rv, ERR_IO_PENDING);
   write_blocked_ = false;
   WriteResult result(rv < 0 ? WRITE_STATUS_ERROR : WRITE_STATUS_OK, rv);
   connection_->OnPacketSent(result);
   connection_->OnCanWrite();
+  visitor_->OnCanWrite();

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

It is strange that we are invoking the OnCanWrite() methods on two objects, but
only one of them is considered a "visitor".

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

Now there's one Callback and one QuicBlockedWriterInterface.

 }
 
 }  // namespace net