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

Unified Diff: net/quic/quic_packet_creator.cc

Issue 1413443005: Remove QuicFecGroup::MinProtectedPacket which is no longer needed. No behavior change expected. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@105506241
Patch Set: Created 5 years, 2 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 | « net/quic/quic_fec_group_interface.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/quic_packet_creator.cc
diff --git a/net/quic/quic_packet_creator.cc b/net/quic/quic_packet_creator.cc
index 23388b5af574056fb9b77498413e34923f61ad49..f3800449c5bcf3683b721fa334df6063e94fe469 100644
--- a/net/quic/quic_packet_creator.cc
+++ b/net/quic/quic_packet_creator.cc
@@ -531,8 +531,7 @@ SerializedPacket QuicPacketCreator::SerializeFec(char* buffer,
fec_group_.reset(nullptr);
packet_size_ = 0;
LOG_IF(DFATAL, packet == nullptr)
- << "Failed to serialize fec packet for group:"
- << fec_group_->MinProtectedPacket();
+ << "Failed to serialize fec packet for group:" << fec_group_number_;
DCHECK_GE(max_packet_length_, packet->length());
// Immediately encrypt the packet, to ensure we don't encrypt the same packet
// packet number multiple times.
« no previous file with comments | « net/quic/quic_fec_group_interface.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698