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

Unified Diff: net/quic/quic_packet_creator.cc

Issue 1541263002: Landing Recent QUIC changes until 12/18/2015 13:57 UTC (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: replace -1 with 0xff for InvalidPathId Created 5 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_multipath_received_packet_manager_test.cc ('k') | net/quic/quic_packet_creator_test.cc » ('j') | 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 a9739e649ecfd801d64966f18b70ee73d3375f8c..7305feac8e7ac25eb5588fa479eb6fa0e6bb123d 100644
--- a/net/quic/quic_packet_creator.cc
+++ b/net/quic/quic_packet_creator.cc
@@ -643,12 +643,13 @@ SerializedPacket QuicPacketCreator::SerializePacket(
packet_size_ = 0;
queued_frames_.clear();
needs_padding_ = false;
- return SerializedPacket(
- header.packet_number, header.public_header.packet_number_length,
- encrypted_buffer, encrypted_length, /* owns_buffer*/ false,
- QuicFramer::GetPacketEntropyHash(header),
- queued_retransmittable_frames_.release(), has_ack_, has_stop_waiting_,
- encryption_level_);
+ return SerializedPacket(current_path_, header.packet_number,
+ header.public_header.packet_number_length,
+ encrypted_buffer, encrypted_length,
+ /* owns_buffer*/ false,
+ QuicFramer::GetPacketEntropyHash(header),
+ queued_retransmittable_frames_.release(), has_ack_,
+ has_stop_waiting_, encryption_level_);
}
SerializedPacket QuicPacketCreator::SerializeFec(char* buffer,
@@ -678,7 +679,7 @@ SerializedPacket QuicPacketCreator::SerializeFec(char* buffer,
LOG(DFATAL) << "Failed to encrypt packet number " << packet_number_;
return NoPacket();
}
- SerializedPacket serialized(header.packet_number,
+ SerializedPacket serialized(current_path_, header.packet_number,
header.public_header.packet_number_length, buffer,
encrypted_length, /* owns_buffer */ false,
QuicFramer::GetPacketEntropyHash(header), nullptr,
@@ -698,8 +699,8 @@ QuicEncryptedPacket* QuicPacketCreator::SerializeVersionNegotiationPacket(
}
SerializedPacket QuicPacketCreator::NoPacket() {
- return SerializedPacket(0, PACKET_1BYTE_PACKET_NUMBER, nullptr, 0, nullptr,
- false, false);
+ return SerializedPacket(kInvalidPathId, 0, PACKET_1BYTE_PACKET_NUMBER,
+ nullptr, 0, nullptr, false, false);
}
void QuicPacketCreator::FillPacketHeader(QuicFecGroupNumber fec_group,
« no previous file with comments | « net/quic/quic_multipath_received_packet_manager_test.cc ('k') | net/quic/quic_packet_creator_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698