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

Unified Diff: net/quic/quic_stream_factory_test.cc

Issue 12334063: Land recent QUIC changes. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: more EXPECT_FALSE Created 7 years, 10 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_session_test.cc ('k') | net/quic/quic_stream_sequencer.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/quic_stream_factory_test.cc
diff --git a/net/quic/quic_stream_factory_test.cc b/net/quic/quic_stream_factory_test.cc
index f18e3b3fee087fc8d6f5a70db39e6a2f2560e589..e8170215e9af9997af88bd66d2ec3ebfb07c5a50 100644
--- a/net/quic/quic_stream_factory_test.cc
+++ b/net/quic/quic_stream_factory_test.cc
@@ -38,14 +38,14 @@ class QuicStreamFactoryTest : public ::testing::Test {
host));
QuicFramer framer(QuicDecrypter::Create(kNULL),
QuicEncrypter::Create(kNULL));
- return scoped_ptr<QuicEncryptedPacket>(framer.EncryptPacket(*chlo));
+ return scoped_ptr<QuicEncryptedPacket>(framer.EncryptPacket(1, *chlo));
}
scoped_ptr<QuicEncryptedPacket> ConstructShlo() {
scoped_ptr<QuicPacket> shlo(ConstructHandshakePacket(0xDEADBEEF, kSHLO));
QuicFramer framer(QuicDecrypter::Create(kNULL),
QuicEncrypter::Create(kNULL));
- return scoped_ptr<QuicEncryptedPacket>(framer.EncryptPacket(*shlo));
+ return scoped_ptr<QuicEncryptedPacket>(framer.EncryptPacket(1, *shlo));
}
scoped_ptr<QuicEncryptedPacket> ConstructRstPacket(
@@ -53,12 +53,15 @@ class QuicStreamFactoryTest : public ::testing::Test {
QuicStreamId stream_id) {
QuicPacketHeader header;
header.public_header.guid = 0xDEADBEEF;
- header.public_header.flags = PACKET_PUBLIC_FLAGS_NONE;
+ header.public_header.reset_flag = false;
+ header.public_header.version_flag = false;
header.packet_sequence_number = num;
- header.private_flags = PACKET_PRIVATE_FLAGS_NONE;
+ header.entropy_flag = false;
+ header.fec_entropy_flag = false;
+ header.fec_flag = false;
header.fec_group = 0;
- QuicRstStreamFrame rst(stream_id, 0, QUIC_NO_ERROR);
+ QuicRstStreamFrame rst(stream_id, QUIC_NO_ERROR);
return scoped_ptr<QuicEncryptedPacket>(
ConstructPacket(header, QuicFrame(&rst)));
}
@@ -68,13 +71,15 @@ class QuicStreamFactoryTest : public ::testing::Test {
QuicPacketSequenceNumber least_unacked) {
QuicPacketHeader header;
header.public_header.guid = 0xDEADBEEF;
- header.public_header.flags = PACKET_PUBLIC_FLAGS_NONE;
+ header.public_header.reset_flag = false;
+ header.public_header.version_flag = false;
header.packet_sequence_number = 2;
- header.private_flags = PACKET_PRIVATE_FLAGS_NONE;
+ header.entropy_flag = false;
+ header.fec_entropy_flag = false;
+ header.fec_flag = false;
header.fec_group = 0;
QuicAckFrame ack(largest_received, least_unacked);
-
QuicCongestionFeedbackFrame feedback;
feedback.type = kTCP;
feedback.tcp.accumulated_number_of_lost_packets = 0;
@@ -86,8 +91,9 @@ class QuicStreamFactoryTest : public ::testing::Test {
frames.push_back(QuicFrame(&ack));
frames.push_back(QuicFrame(&feedback));
scoped_ptr<QuicPacket> packet(
- framer.ConstructFrameDataPacket(header, frames));
- return scoped_ptr<QuicEncryptedPacket>(framer.EncryptPacket(*packet));
+ framer.ConstructFrameDataPacket(header, frames).packet);
+ return scoped_ptr<QuicEncryptedPacket>(
+ framer.EncryptPacket(header.packet_sequence_number, *packet));
}
// Returns a newly created packet to send congestion feedback data.
@@ -95,9 +101,12 @@ class QuicStreamFactoryTest : public ::testing::Test {
QuicPacketSequenceNumber sequence_number) {
QuicPacketHeader header;
header.public_header.guid = 0xDEADBEEF;
- header.public_header.flags = PACKET_PUBLIC_FLAGS_NONE;
+ header.public_header.reset_flag = false;
+ header.public_header.version_flag = false;
header.packet_sequence_number = sequence_number;
- header.private_flags = PACKET_PRIVATE_FLAGS_NONE;
+ header.entropy_flag = false;
+ header.fec_entropy_flag = false;
+ header.fec_flag = false;
header.fec_group = 0;
QuicCongestionFeedbackFrame frame;
@@ -117,8 +126,9 @@ class QuicStreamFactoryTest : public ::testing::Test {
QuicFrames frames;
frames.push_back(frame);
scoped_ptr<QuicPacket> packet(
- framer.ConstructFrameDataPacket(header, frames));
- return scoped_ptr<QuicEncryptedPacket>(framer.EncryptPacket(*packet));
+ framer.ConstructFrameDataPacket(header, frames).packet);
+ return scoped_ptr<QuicEncryptedPacket>(
+ framer.EncryptPacket(header.packet_sequence_number, *packet));
}
MockHostResolver host_resolver_;
« no previous file with comments | « net/quic/quic_session_test.cc ('k') | net/quic/quic_stream_sequencer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698