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

Unified Diff: net/quic/quic_sent_packet_manager.h

Issue 115463002: Land Recent QUIC Changes. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years 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 | « net/quic/quic_reliable_client_stream_test.cc ('k') | net/quic/quic_sent_packet_manager.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/quic_sent_packet_manager.h
diff --git a/net/quic/quic_sent_packet_manager.h b/net/quic/quic_sent_packet_manager.h
index 9e7bdb8fd32968d3d802af41b03cd57fb9fd0413..71c61e93c03659a97b5873c044b2f15a43810248 100644
--- a/net/quic/quic_sent_packet_manager.h
+++ b/net/quic/quic_sent_packet_manager.h
@@ -204,18 +204,23 @@ class NET_EXPORT_PRIVATE QuicSentPacketManager {
TransmissionInfo()
: retransmittable_frames(NULL),
sequence_number_length(PACKET_1BYTE_SEQUENCE_NUMBER),
- sent_time(QuicTime::Zero()) { }
+ sent_time(QuicTime::Zero()),
+ previous_transmissions(NULL) { }
TransmissionInfo(RetransmittableFrames* retransmittable_frames,
QuicSequenceNumberLength sequence_number_length)
: retransmittable_frames(retransmittable_frames),
sequence_number_length(sequence_number_length),
- sent_time(QuicTime::Zero()) {
+ sent_time(QuicTime::Zero()),
+ previous_transmissions(NULL) {
}
RetransmittableFrames* retransmittable_frames;
QuicSequenceNumberLength sequence_number_length;
// Zero when the packet is serialized, non-zero once it's sent.
QuicTime sent_time;
+ // Stores all previous transmissions if the packet has been retransmitted,
+ // and is NULL otherwise.
+ SequenceNumberSet* previous_transmissions;
};
typedef linked_hash_map<QuicPacketSequenceNumber,
@@ -252,11 +257,6 @@ class NET_EXPORT_PRIVATE QuicSentPacketManager {
QuicSequenceNumberLength GetSequenceNumberLength(
QuicPacketSequenceNumber sequence_number) const;
- // Returns the sequence number of the packet that |sequence_number| was
- // most recently transmitted as.
- QuicPacketSequenceNumber GetMostRecentTransmission(
- QuicPacketSequenceNumber sequence_number) const;
-
// Clears up to |num_to_clear| previous transmissions in order to make room
// in the ack frame for new acks.
void ClearPreviousRetransmissions(size_t num_to_clear);
@@ -276,12 +276,6 @@ class NET_EXPORT_PRIVATE QuicSentPacketManager {
// Pending retransmissions which have not been packetized and sent yet.
PendingRetransmissionMap pending_retransmissions_;
- // Map from sequence number to set of all sequence number that this packet has
- // been transmitted as. If a packet has not been retransmitted, it will not
- // have an entry in this map. If any transmission of a packet has been acked
- // it will not have an entry in this map.
- PreviousTransmissionMap previous_transmissions_map_;
-
// Tracks if the connection was created by the server.
bool is_server_;
« no previous file with comments | « net/quic/quic_reliable_client_stream_test.cc ('k') | net/quic/quic_sent_packet_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698