commit 3f4c9ecd14c5ae7513e1df2578e4e1d58a21bae9
author nharper <nharper@google.com> Mon Nov 02 15:17:47 2020 -0800
committer Copybara-Service <copybara-worker@google.com> Mon Nov 02 15:18:24 2020 -0800
tree ca6b0c91d59bbf69f75e67b962ca4f0aa18a0026
parent 1c6bea76ce890e894fc64989438b9aac11df79b0

Re-enable tests in Chrome.

When applying this change locally in my chromium checkout, all net_unittests
pass.

Protected by not protected (Chrome test-only change).

PiperOrigin-RevId: 340326746
Change-Id: Iad3e9452dc07f088bb2a1cc22b71263b6859a924

quic/core/http/end_to_end_test.cc

diff --git a/quic/core/http/end_to_end_test.cc b/quic/core/http/end_to_end_test.cc
index 0557b2b..91af8c6 100644
--- a/quic/core/http/end_to_end_test.cc
+++ b/quic/core/http/end_to_end_test.cc
@@ -1442,8 +1442,7 @@
             client_->SendCustomSynchronousRequest(headers, body));
 }
 
-// TODO(crbug.com/1140295): Re-enable this test.
-TEST_P(EndToEndTest, QUIC_TEST_DISABLED_IN_CHROME(LargePostZeroRTTFailure)) {
+TEST_P(EndToEndTest, LargePostZeroRTTFailure) {
   // Send a request and then disconnect. This prepares the client to attempt
   // a 0-RTT handshake for the next request.
   ASSERT_TRUE(Initialize());
@@ -1541,9 +1540,7 @@
   client_->Disconnect();
 }
 
-// TODO(crbug.com/1140295): Re-enable this test.
-TEST_P(EndToEndTest,
-       QUIC_TEST_DISABLED_IN_CHROME(SynchronousRequestZeroRTTFailure)) {
+TEST_P(EndToEndTest, SynchronousRequestZeroRTTFailure) {
   // Send a request and then disconnect. This prepares the client to attempt
   // a 0-RTT handshake for the next request.
   ASSERT_TRUE(Initialize());
@@ -1591,9 +1588,7 @@
   VerifyCleanConnection(false);
 }
 
-// TODO(crbug.com/1140295): Re-enable this test.
-TEST_P(EndToEndTest,
-       QUIC_TEST_DISABLED_IN_CHROME(LargePostSynchronousRequest)) {
+TEST_P(EndToEndTest, LargePostSynchronousRequest) {
   // Send a request and then disconnect. This prepares the client to attempt
   // a 0-RTT handshake for the next request.
   ASSERT_TRUE(Initialize());
@@ -1712,10 +1707,7 @@
 }
 
 // Regression test for b/166836136.
-// TODO(crbug.com/1140295): Re-enable this test.
-TEST_P(EndToEndTest,
-       QUIC_TEST_DISABLED_IN_CHROME(
-           RetransmissionAfterZeroRTTRejectBeforeOneRtt)) {
+TEST_P(EndToEndTest, RetransmissionAfterZeroRTTRejectBeforeOneRtt) {
   if (!version_.UsesTls()) {
     // This test is TLS specific.
     ASSERT_TRUE(Initialize());