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

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

Issue 198353003: Land Recent QUIC Changes (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 9 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 | Annotate | Revision Log
« no previous file with comments | « net/quic/quic_ack_notifier.h ('k') | net/quic/quic_ack_notifier_manager.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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_ack_notifier.h" 5 #include "net/quic/quic_ack_notifier.h"
6 6
7 #include <set> 7 #include <set>
8 8
9 #include "base/logging.h" 9 #include "base/logging.h"
10 #include "base/stl_util.h" 10 #include "base/stl_util.h"
11 11
12 using base::hash_map;
13 using std::make_pair;
14
12 namespace net { 15 namespace net {
13 16
17 QuicAckNotifier::PacketInfo::PacketInfo() : packet_payload_size(0) {
18 }
19
20 QuicAckNotifier::PacketInfo::PacketInfo(int payload_size)
21 : packet_payload_size(payload_size) {
22 }
23
14 QuicAckNotifier::DelegateInterface::DelegateInterface() {} 24 QuicAckNotifier::DelegateInterface::DelegateInterface() {}
15 25
16 QuicAckNotifier::DelegateInterface::~DelegateInterface() {} 26 QuicAckNotifier::DelegateInterface::~DelegateInterface() {}
17 27
18 QuicAckNotifier::QuicAckNotifier(DelegateInterface* delegate) 28 QuicAckNotifier::QuicAckNotifier(DelegateInterface* delegate)
19 : delegate_(delegate) { 29 : delegate_(delegate),
30 original_packet_count_(0),
31 original_byte_count_(0),
32 retransmitted_packet_count_(0),
33 retransmitted_byte_count_(0) {
20 DCHECK(delegate); 34 DCHECK(delegate);
21 } 35 }
22 36
23 QuicAckNotifier::~QuicAckNotifier() { 37 QuicAckNotifier::~QuicAckNotifier() {
24 } 38 }
25 39
26 void QuicAckNotifier::AddSequenceNumber( 40 void QuicAckNotifier::AddSequenceNumber(
27 const QuicPacketSequenceNumber& sequence_number) { 41 const QuicPacketSequenceNumber& sequence_number,
28 sequence_numbers_.insert(sequence_number); 42 int packet_payload_size) {
29 } 43 sequence_numbers_.insert(make_pair(sequence_number,
30 44 PacketInfo(packet_payload_size)));
31 void QuicAckNotifier::AddSequenceNumbers( 45 ++original_packet_count_;
32 const SequenceNumberSet& sequence_numbers) { 46 original_byte_count_ += packet_payload_size;
33 for (SequenceNumberSet::const_iterator it = sequence_numbers.begin();
34 it != sequence_numbers.end(); ++it) {
35 AddSequenceNumber(*it);
36 }
37 } 47 }
38 48
39 bool QuicAckNotifier::OnAck(QuicPacketSequenceNumber sequence_number) { 49 bool QuicAckNotifier::OnAck(QuicPacketSequenceNumber sequence_number) {
40 DCHECK(ContainsKey(sequence_numbers_, sequence_number)); 50 DCHECK(ContainsKey(sequence_numbers_, sequence_number));
41 sequence_numbers_.erase(sequence_number); 51 sequence_numbers_.erase(sequence_number);
42 if (IsEmpty()) { 52 if (IsEmpty()) {
43 // We have seen all the sequence numbers we were waiting for, trigger 53 // We have seen all the sequence numbers we were waiting for, trigger
44 // callback notification. 54 // callback notification.
45 delegate_->OnAckNotification(); 55 delegate_->OnAckNotification(
56 original_packet_count_, original_byte_count_,
57 retransmitted_packet_count_, retransmitted_byte_count_);
46 return true; 58 return true;
47 } 59 }
48 return false; 60 return false;
49 } 61 }
50 62
51 void QuicAckNotifier::UpdateSequenceNumber( 63 void QuicAckNotifier::UpdateSequenceNumber(
52 QuicPacketSequenceNumber old_sequence_number, 64 QuicPacketSequenceNumber old_sequence_number,
53 QuicPacketSequenceNumber new_sequence_number) { 65 QuicPacketSequenceNumber new_sequence_number) {
54 sequence_numbers_.erase(old_sequence_number); 66 DCHECK(!ContainsKey(sequence_numbers_, new_sequence_number));
55 sequence_numbers_.insert(new_sequence_number); 67
68 PacketInfo packet_info;
69 hash_map<QuicPacketSequenceNumber, PacketInfo>::iterator it =
70 sequence_numbers_.find(old_sequence_number);
71 if (it != sequence_numbers_.end()) {
72 packet_info = it->second;
73 sequence_numbers_.erase(it);
74 } else {
75 DLOG(DFATAL) << "Old sequence number not found.";
76 }
77
78 ++retransmitted_packet_count_;
79 retransmitted_byte_count_ += packet_info.packet_payload_size;
80 sequence_numbers_.insert(make_pair(new_sequence_number, packet_info));
56 } 81 }
57 82
58 }; // namespace net 83 }; // namespace net
OLDNEW
« no previous file with comments | « net/quic/quic_ack_notifier.h ('k') | net/quic/quic_ack_notifier_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698