Port QuicServer to Chrome network stack

net/quic/quic_per_connection_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/tools/quic/quic_packet_writer_wrapper.h"
+#include "net/quic/quic_per_connection_packet_writer.h"
 
-#include "net/quic/quic_types.h"
+#include "base/location.h"
+#include "base/logging.h"
+#include "base/message_loop/message_loop.h"
+#include "base/metrics/sparse_histogram.h"
+#include "net/base/io_buffer.h"
+#include "net/base/net_errors.h"
+#include "net/quic/crypto/quic_random.h"

[wtc, 2014/06/23 23:12:16]

We don't need many of these headers.

I remember you removed them before. Maybe there was a merge or rebase error?

[dmziegler, 2014/06/24 00:08:52]

Strange, seems so. Removed.

+#include "net/quic/quic_server_packet_writer.h"
 
 namespace net {
-namespace tools {
 
-QuicPacketWriterWrapper::QuicPacketWriterWrapper() {}
+QuicPerConnectionPacketWriter::QuicPerConnectionPacketWriter(
+    QuicServerPacketWriter* writer)
+    : weak_factory_(this), writer_(writer) {
+}
 
-QuicPacketWriterWrapper::QuicPacketWriterWrapper(QuicPacketWriter* writer)
-    : writer_(writer) {}
+QuicPerConnectionPacketWriter::~QuicPerConnectionPacketWriter() {
+}
 
-QuicPacketWriterWrapper::~QuicPacketWriterWrapper() {}
-
-WriteResult QuicPacketWriterWrapper::WritePacket(
+WriteResult QuicPerConnectionPacketWriter::WritePacket(
     const char* buffer,
     size_t buf_len,
-    const net::IPAddressNumber& self_address,
-    const net::IPEndPoint& peer_address) {
-  return writer_->WritePacket(buffer, buf_len, self_address, peer_address);
+    const IPAddressNumber& self_address,
+    const IPEndPoint& peer_address) {
+  return writer_->WritePacketWithCallback(
+      buffer,
+      buf_len,
+      self_address,
+      peer_address,
+      base::Bind(&QuicPerConnectionPacketWriter::OnWriteComplete,
+                 weak_factory_.GetWeakPtr()));
 }
 
-bool QuicPacketWriterWrapper::IsWriteBlockedDataBuffered() const {
+bool QuicPerConnectionPacketWriter::IsWriteBlockedDataBuffered() const {
   return writer_->IsWriteBlockedDataBuffered();
 }
 
-bool QuicPacketWriterWrapper::IsWriteBlocked() const {
+bool QuicPerConnectionPacketWriter::IsWriteBlocked() const {
   return writer_->IsWriteBlocked();
 }
 
-void QuicPacketWriterWrapper::SetWritable() {
+void QuicPerConnectionPacketWriter::SetWritable() {
   writer_->SetWritable();
 }
 
-void QuicPacketWriterWrapper::set_writer(QuicPacketWriter* writer) {
-  writer_.reset(writer);
+void QuicPerConnectionPacketWriter::OnWriteComplete(WriteResult wr) {

[wtc, 2014/06/23 23:12:15]

Rename "wr" to "result".

[dmziegler, 2014/06/24 00:08:52]

Done.

+  connection_->OnPacketSent(wr);
 }
 
-QuicPacketWriter* QuicPacketWriterWrapper::release_writer() {
-  return writer_.release();
-}
-
-}  // namespace tools
 }  // namespace net