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

Unified Diff: net/quic/quic_packet_generator_test.cc

Issue 12806002: Land Recent QUIC Changes (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: minor comment fix Created 7 years, 9 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_packet_generator.cc ('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_generator_test.cc
diff --git a/net/quic/quic_packet_generator_test.cc b/net/quic/quic_packet_generator_test.cc
index 3b97472b63c9865df35a9424bd8146b3cc8fc4ef..5ed1df591879cfd1bb5449c4a69a86054de2f094 100644
--- a/net/quic/quic_packet_generator_test.cc
+++ b/net/quic/quic_packet_generator_test.cc
@@ -31,14 +31,15 @@ class MockDelegate : public QuicPacketGenerator::DelegateInterface {
MockDelegate() {}
virtual ~MockDelegate() {}
- MOCK_METHOD1(CanWrite, bool(bool is_retransmission));
+ MOCK_METHOD2(CanWrite, bool(bool is_retransmission,
+ bool has_retransmittable_data));
MOCK_METHOD0(CreateAckFrame, QuicAckFrame*());
MOCK_METHOD0(CreateFeedbackFrame, QuicCongestionFeedbackFrame*());
MOCK_METHOD1(OnSerializedPacket, bool(const SerializedPacket& packet));
void SetCanWrite(bool can_write) {
- EXPECT_CALL(*this, CanWrite(false)).WillRepeatedly(Return(can_write));
+ EXPECT_CALL(*this, CanWrite(false, _)).WillRepeatedly(Return(can_write));
}
private:
@@ -76,8 +77,9 @@ class QuicPacketGeneratorTest : public ::testing::Test {
QuicPacketGeneratorTest()
: framer_(kQuicVersion1,
QuicDecrypter::Create(kNULL),
- QuicEncrypter::Create(kNULL)),
- creator_(42, &framer_, &random_),
+ QuicEncrypter::Create(kNULL),
+ false),
+ creator_(42, &framer_, &random_, false),
generator_(&delegate_, &creator_),
packet_(0, NULL, 0, NULL),
packet2_(0, NULL, 0, NULL),
@@ -101,7 +103,7 @@ class QuicPacketGeneratorTest : public ::testing::Test {
QuicAckFrame* CreateAckFrame() {
// TODO(rch): Initialize this so it can be verified later.
- return new QuicAckFrame(0, 0);
+ return new QuicAckFrame(0, QuicTime::Zero(), 0);
}
QuicCongestionFeedbackFrame* CreateFeedbackFrame() {
« no previous file with comments | « net/quic/quic_packet_generator.cc ('k') | net/quic/quic_protocol.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698