Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(336)

Side by Side Diff: net/quic/quic_per_connection_packet_writer.cc

Issue 467963002: Refactoring: Create per-connection packet writers in QuicDispatcher. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: More review comments Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "net/quic/quic_per_connection_packet_writer.h" 5 #include "net/quic/quic_per_connection_packet_writer.h"
6 6
7 #include "net/quic/quic_server_packet_writer.h" 7 #include "net/quic/quic_server_packet_writer.h"
8 #include "net/quic/quic_types.h" 8 #include "net/quic/quic_types.h"
9 9
10 namespace net { 10 namespace net {
11 11
12 QuicPerConnectionPacketWriter::QuicPerConnectionPacketWriter( 12 QuicPerConnectionPacketWriter::QuicPerConnectionPacketWriter(
13 QuicServerPacketWriter* writer) 13 QuicServerPacketWriter* shared_writer,
14 : weak_factory_(this), writer_(writer) { 14 QuicConnection* connection)
15 : weak_factory_(this),
16 shared_writer_(shared_writer),
17 connection_(connection) {
15 } 18 }
16 19
17 QuicPerConnectionPacketWriter::~QuicPerConnectionPacketWriter() { 20 QuicPerConnectionPacketWriter::~QuicPerConnectionPacketWriter() {
18 } 21 }
19 22
20 WriteResult QuicPerConnectionPacketWriter::WritePacket( 23 WriteResult QuicPerConnectionPacketWriter::WritePacket(
21 const char* buffer, 24 const char* buffer,
22 size_t buf_len, 25 size_t buf_len,
23 const IPAddressNumber& self_address, 26 const IPAddressNumber& self_address,
24 const IPEndPoint& peer_address) { 27 const IPEndPoint& peer_address) {
25 return writer_->WritePacketWithCallback( 28 return shared_writer_->WritePacketWithCallback(
26 buffer, 29 buffer,
27 buf_len, 30 buf_len,
28 self_address, 31 self_address,
29 peer_address, 32 peer_address,
30 base::Bind(&QuicPerConnectionPacketWriter::OnWriteComplete, 33 base::Bind(&QuicPerConnectionPacketWriter::OnWriteComplete,
31 weak_factory_.GetWeakPtr())); 34 weak_factory_.GetWeakPtr()));
32 } 35 }
33 36
34 bool QuicPerConnectionPacketWriter::IsWriteBlockedDataBuffered() const { 37 bool QuicPerConnectionPacketWriter::IsWriteBlockedDataBuffered() const {
35 return writer_->IsWriteBlockedDataBuffered(); 38 return shared_writer_->IsWriteBlockedDataBuffered();
36 } 39 }
37 40
38 bool QuicPerConnectionPacketWriter::IsWriteBlocked() const { 41 bool QuicPerConnectionPacketWriter::IsWriteBlocked() const {
39 return writer_->IsWriteBlocked(); 42 return shared_writer_->IsWriteBlocked();
40 } 43 }
41 44
42 void QuicPerConnectionPacketWriter::SetWritable() { 45 void QuicPerConnectionPacketWriter::SetWritable() {
43 writer_->SetWritable(); 46 shared_writer_->SetWritable();
44 } 47 }
45 48
46 void QuicPerConnectionPacketWriter::OnWriteComplete(WriteResult result) { 49 void QuicPerConnectionPacketWriter::OnWriteComplete(WriteResult result) {
47 connection_->OnPacketSent(result); 50 connection_->OnPacketSent(result);
48 } 51 }
49 52
50 } // namespace net 53 } // namespace net
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698