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

Unified Diff: net/quic/quic_sent_packet_manager.cc

Issue 1330973002: relnote: Refactor QuicAckFrame::missing_packets to support a change to a (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@Tidy_up_DLOG_messages_101773586
Patch Set: Created 5 years, 3 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
Index: net/quic/quic_sent_packet_manager.cc
diff --git a/net/quic/quic_sent_packet_manager.cc b/net/quic/quic_sent_packet_manager.cc
index d4d4cf21c5d50d2ac3684c33725befb25205e19d..715bdf503f74528a2bee67c521583df3572576bb 100644
--- a/net/quic/quic_sent_packet_manager.cc
+++ b/net/quic/quic_sent_packet_manager.cc
@@ -271,10 +271,10 @@ void QuicSentPacketManager::OnIncomingAck(const QuicAckFrame& ack_frame,
void QuicSentPacketManager::UpdatePacketInformationReceivedByPeer(
const QuicAckFrame& ack_frame) {
- if (ack_frame.missing_packets.empty()) {
+ if (ack_frame.missing_packets.Empty()) {
least_packet_awaited_by_peer_ = ack_frame.largest_observed + 1;
} else {
- least_packet_awaited_by_peer_ = *(ack_frame.missing_packets.begin());
+ least_packet_awaited_by_peer_ = ack_frame.missing_packets.Min();
}
}
@@ -306,7 +306,7 @@ void QuicSentPacketManager::HandleAckForSentPackets(
break;
}
- if (ContainsKey(ack_frame.missing_packets, packet_number)) {
+ if (ack_frame.missing_packets.Contains(packet_number)) {
// Don't continue to increase the nack count for packets not in flight.
if (!it->in_flight) {
continue;

Powered by Google App Engine
This is Rietveld 408576698