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

Unified Diff: net/quic/quic_ack_notifier.h

Issue 798873005: Remove unused original transmission stats from QuicAckNotifier. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@removing_quic_version_21_83551025
Patch Set: Created 5 years, 11 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | net/quic/quic_ack_notifier.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/quic_ack_notifier.h
diff --git a/net/quic/quic_ack_notifier.h b/net/quic/quic_ack_notifier.h
index f19d0f8c6b995de6d157b3535cc94019de3452ca..f8d813ec7c1b13592c0438e665a4d75e50f06a5d 100644
--- a/net/quic/quic_ack_notifier.h
+++ b/net/quic/quic_ack_notifier.h
@@ -17,21 +17,18 @@ namespace net {
// trigger a call to a provided Closure.
class NET_EXPORT_PRIVATE QuicAckNotifier {
public:
- class NET_EXPORT_PRIVATE DelegateInterface
- : public base::RefCounted<DelegateInterface> {
+ class NET_EXPORT_PRIVATE DelegateInterface
+ : public base::RefCounted<DelegateInterface> {
public:
DelegateInterface();
// Args:
- // num_original_packets - Number of packets in the original transmission.
- // num_original_bytes - Number of packets in the original transmission.
// num_retransmitted_packets - Number of packets that had to be
// retransmitted.
// num_retransmitted_bytes - Number of bytes that had to be retransmitted.
- virtual void OnAckNotification(int num_original_packets,
- int num_original_bytes,
- int num_retransmitted_packets,
+ virtual void OnAckNotification(int num_retransmitted_packets,
int num_retransmitted_bytes,
QuicTime::Delta delta_largest_observed) = 0;
+
protected:
friend class base::RefCounted<DelegateInterface>;
@@ -83,11 +80,6 @@ class NET_EXPORT_PRIVATE QuicAckNotifier {
// delegate will not be called until this is empty.
base::hash_map<QuicPacketSequenceNumber, PacketInfo> sequence_numbers_;
- // Transmission and retransmission stats.
- // Number of packets in the original transmission.
- int original_packet_count_;
- // Number of packets in the original transmission.
- int original_byte_count_;
// Number of packets that had to be retransmitted.
int retransmitted_packet_count_;
// Number of bytes that had to be retransmitted.
« no previous file with comments | « no previous file | net/quic/quic_ack_notifier.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698