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

Unified Diff: net/quic/quic_packet_creator.cc

Issue 11377096: Change from re-transmitting an packet with a retransmit number to sending a new packet with a new s… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 1 month 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_packet_creator.h ('k') | net/quic/quic_protocol.h » ('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 deef83de853284bf3c8d9b9012dd522203f64436..2b00e2e48bb50ef74eb2e90d0104099c72f7a1bc 100644
--- a/net/quic/quic_packet_creator.cc
+++ b/net/quic/quic_packet_creator.cc
@@ -81,7 +81,7 @@ void QuicPacketCreator::DataToStream(QuicStreamId id,
data_to_send -= frame_len;
// Produce the data packet (which might fin the stream).
- framer_->ConstructFragementDataPacket(header, frames, &packet);
+ framer_->ConstructFrameDataPacket(header, frames, &packet);
DCHECK_GE(options_.max_packet_length, packet->length());
packets->push_back(make_pair(header.packet_sequence_number, packet));
frames.clear();
@@ -93,7 +93,7 @@ void QuicPacketCreator::DataToStream(QuicStreamId id,
FillPacketHeader(current_fec_group, PACKET_FLAGS_NONE, &header);
QuicStreamFrame frame(id, true, offset, "");
frames.push_back(QuicFrame(&frame));
- framer_->ConstructFragementDataPacket(header, frames, &packet);
+ framer_->ConstructFrameDataPacket(header, frames, &packet);
packets->push_back(make_pair(header.packet_sequence_number, packet));
frames.clear();
}
@@ -138,7 +138,7 @@ QuicPacketCreator::PacketPair QuicPacketCreator::ResetStream(
QuicPacket* packet;
QuicFrames frames;
frames.push_back(QuicFrame(&close_frame));
- framer_->ConstructFragementDataPacket(header, frames, &packet);
+ framer_->ConstructFrameDataPacket(header, frames, &packet);
return make_pair(header.packet_sequence_number, packet);
}
@@ -151,7 +151,7 @@ QuicPacketCreator::PacketPair QuicPacketCreator::CloseConnection(
QuicPacket* packet;
QuicFrames frames;
frames.push_back(QuicFrame(close_frame));
- framer_->ConstructFragementDataPacket(header, frames, &packet);
+ framer_->ConstructFrameDataPacket(header, frames, &packet);
return make_pair(header.packet_sequence_number, packet);
}
@@ -164,10 +164,17 @@ QuicPacketCreator::PacketPair QuicPacketCreator::AckPacket(
QuicPacket* packet;
QuicFrames frames;
frames.push_back(QuicFrame(ack_frame));
- framer_->ConstructFragementDataPacket(header, frames, &packet);
+ framer_->ConstructFrameDataPacket(header, frames, &packet);
return make_pair(header.packet_sequence_number, packet);
}
+QuicPacketSequenceNumber QuicPacketCreator::SetNewSequenceNumber(
+ QuicPacket* packet) {
+ ++sequence_number_;
+ framer_->WriteSequenceNumber(sequence_number_, packet);
+ return sequence_number_;
+}
+
void QuicPacketCreator::FillPacketHeader(QuicFecGroupNumber fec_group,
QuicPacketFlags flags,
QuicPacketHeader* header) {
@@ -176,9 +183,6 @@ void QuicPacketCreator::FillPacketHeader(QuicFecGroupNumber fec_group,
header->packet_sequence_number = ++sequence_number_;
header->fec_group = fec_group;
- // Default to zero - the sender should increment this as packets are
- // retransmitted.
- header->retransmission_count = 0;
header->transmission_time = 0;
}
« no previous file with comments | « net/quic/quic_packet_creator.h ('k') | net/quic/quic_protocol.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698