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

Unified Diff: net/tools/quic/end_to_end_test.cc

Issue 398873003: Adds dynamic setting of FEC group size based on the connection's current (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 5 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/test_tools/quic_test_utils.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/tools/quic/end_to_end_test.cc
diff --git a/net/tools/quic/end_to_end_test.cc b/net/tools/quic/end_to_end_test.cc
index c11e06916e3de11df621c58559c8e4a454d44400..dd1b28be80526d1ed5ac07478ff1e156e860605e 100644
--- a/net/tools/quic/end_to_end_test.cc
+++ b/net/tools/quic/end_to_end_test.cc
@@ -683,7 +683,6 @@ TEST_P(EndToEndTest, LargePostFec) {
QuicPacketCreator* creator = QuicConnectionPeer::GetPacketCreator(
client_->client()->session()->connection());
EXPECT_TRUE(creator->IsFecEnabled());
- EXPECT_EQ(kMaxPacketsPerFecGroup, creator->max_packets_per_fec_group());
// Set FecPolicy to always protect data on all streams.
client_->SetFecPolicy(FEC_PROTECT_ALWAYS);
@@ -716,7 +715,6 @@ TEST_P(EndToEndTest, ClientSpecifiedFecProtectionForHeaders) {
QuicPacketCreator* creator = QuicConnectionPeer::GetPacketCreator(
client_->client()->session()->connection());
EXPECT_TRUE(creator->IsFecEnabled());
- EXPECT_EQ(kMaxPacketsPerFecGroup, creator->max_packets_per_fec_group());
// Verify that server headers stream is FEC protected.
server_thread_->Pause();
« no previous file with comments | « net/quic/test_tools/quic_test_utils.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698