| 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 38bb4c4f5589c953fc7c456cf78cf9a016b72143..e59d19f3ba065a949866f490555037b05a850fc3 100644
|
| --- a/net/tools/quic/end_to_end_test.cc
|
| +++ b/net/tools/quic/end_to_end_test.cc
|
| @@ -449,13 +449,12 @@ class EndToEndTest : public ::testing::TestWithParam<TestParams> {
|
| // being discarded, based on connection stats.
|
| // Calls server_thread_ Pause() and Resume(), which may only be called once
|
| // per test.
|
| - void VerifyCleanConnection(bool /*had_packet_loss*/) {
|
| + void VerifyCleanConnection(bool had_packet_loss) {
|
| QuicConnectionStats client_stats =
|
| client_->client()->session()->connection()->GetStats();
|
| - // TODO(ianswett): Re-enable this check once b/19572432 is fixed.
|
| - // if (!had_packet_loss) {
|
| - // EXPECT_EQ(0u, client_stats.packets_lost);
|
| - // }
|
| + if (FLAGS_quic_reply_to_rej && !had_packet_loss) {
|
| + EXPECT_EQ(0u, client_stats.packets_lost);
|
| + }
|
| EXPECT_EQ(0u, client_stats.packets_discarded);
|
| // When doing 0-RTT with stateless rejects, the encrypted requests cause
|
| // a retranmission of the SREJ packets which are dropped by the client.
|
| @@ -478,10 +477,9 @@ class EndToEndTest : public ::testing::TestWithParam<TestParams> {
|
| ASSERT_EQ(1u, dispatcher->session_map().size());
|
| QuicSession* session = dispatcher->session_map().begin()->second;
|
| QuicConnectionStats server_stats = session->connection()->GetStats();
|
| - // TODO(ianswett): Re-enable this check once b/19572432 is fixed.
|
| - // if (!had_packet_loss) {
|
| - // EXPECT_EQ(0u, server_stats.packets_lost);
|
| - // }
|
| + if (FLAGS_quic_reply_to_rej && !had_packet_loss) {
|
| + EXPECT_EQ(0u, server_stats.packets_lost);
|
| + }
|
| EXPECT_EQ(0u, server_stats.packets_discarded);
|
| // TODO(ianswett): Restore the check for packets_dropped equals 0.
|
| // The expect for packets received is equal to packets processed fails
|
|
|