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

Side by Side Diff: net/tools/quic/end_to_end_test.cc

Issue 488233004: QUIC - disabled EndToEndTest.LargePostZeroRTTFailure to fix flakiness (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 3 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <stddef.h> 5 #include <stddef.h>
6 #include <string> 6 #include <string>
7 #include <sys/epoll.h> 7 #include <sys/epoll.h>
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/basictypes.h" 10 #include "base/basictypes.h"
(...skipping 654 matching lines...) Expand 10 before | Expand all | Expand 10 after
665 string body; 665 string body;
666 GenerateBody(&body, 1024 * 1024); 666 GenerateBody(&body, 1024 * 1024);
667 667
668 HTTPMessage request(HttpConstants::HTTP_1_1, 668 HTTPMessage request(HttpConstants::HTTP_1_1,
669 HttpConstants::POST, "/foo"); 669 HttpConstants::POST, "/foo");
670 request.AddBody(body, true); 670 request.AddBody(body, true);
671 671
672 EXPECT_EQ(kFooResponseBody, client_->SendCustomSynchronousRequest(request)); 672 EXPECT_EQ(kFooResponseBody, client_->SendCustomSynchronousRequest(request));
673 } 673 }
674 674
675 TEST_P(EndToEndTest, LargePostZeroRTTFailure) { 675 // TODO(rtenneti): This is flaky on ChromiumOS bots.
676 // http://crbug.com/407756
677 TEST_P(EndToEndTest, DISABLED_LargePostZeroRTTFailure) {
676 // Have the server accept 0-RTT without waiting a startup period. 678 // Have the server accept 0-RTT without waiting a startup period.
677 strike_register_no_startup_period_ = true; 679 strike_register_no_startup_period_ = true;
678 680
679 // Send a request and then disconnect. This prepares the client to attempt 681 // Send a request and then disconnect. This prepares the client to attempt
680 // a 0-RTT handshake for the next request. 682 // a 0-RTT handshake for the next request.
681 ASSERT_TRUE(Initialize()); 683 ASSERT_TRUE(Initialize());
682 684
683 string body; 685 string body;
684 GenerateBody(&body, 20480); 686 GenerateBody(&body, 20480);
685 687
(...skipping 702 matching lines...) Expand 10 before | Expand all | Expand 10 after
1388 QuicSession* session = dispatcher->session_map().begin()->second; 1390 QuicSession* session = dispatcher->session_map().begin()->second;
1389 EXPECT_EQ(0u, QuicSessionPeer::GetLocallyClosedStreamsHighestOffset( 1391 EXPECT_EQ(0u, QuicSessionPeer::GetLocallyClosedStreamsHighestOffset(
1390 session).size()); 1392 session).size());
1391 server_thread_->Resume(); 1393 server_thread_->Resume();
1392 } 1394 }
1393 1395
1394 } // namespace 1396 } // namespace
1395 } // namespace test 1397 } // namespace test
1396 } // namespace tools 1398 } // namespace tools
1397 } // namespace net 1399 } // namespace net
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698