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

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

Issue 1978953003: Attempts to reduce test flakiness by setting more lenient handshake timeouts. Similar to internal c… (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@121671298
Patch Set: Created 4 years, 7 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 <sys/epoll.h> 6 #include <sys/epoll.h>
7 7
8 #include <list> 8 #include <list>
9 #include <memory> 9 #include <memory>
10 #include <string> 10 #include <string>
(...skipping 282 matching lines...) Expand 10 before | Expand all | Expand 10 after
293 // Use different flow control windows for client/server. 293 // Use different flow control windows for client/server.
294 client_config_.SetInitialStreamFlowControlWindowToSend( 294 client_config_.SetInitialStreamFlowControlWindowToSend(
295 2 * kInitialStreamFlowControlWindowForTest); 295 2 * kInitialStreamFlowControlWindowForTest);
296 client_config_.SetInitialSessionFlowControlWindowToSend( 296 client_config_.SetInitialSessionFlowControlWindowToSend(
297 2 * kInitialSessionFlowControlWindowForTest); 297 2 * kInitialSessionFlowControlWindowForTest);
298 server_config_.SetInitialStreamFlowControlWindowToSend( 298 server_config_.SetInitialStreamFlowControlWindowToSend(
299 3 * kInitialStreamFlowControlWindowForTest); 299 3 * kInitialStreamFlowControlWindowForTest);
300 server_config_.SetInitialSessionFlowControlWindowToSend( 300 server_config_.SetInitialSessionFlowControlWindowToSend(
301 3 * kInitialSessionFlowControlWindowForTest); 301 3 * kInitialSessionFlowControlWindowForTest);
302 302
303 // The default idle timeouts can be too strict when running on a busy
304 // machine.
305 const QuicTime::Delta timeout = QuicTime::Delta::FromSeconds(30);
306 client_config_.set_max_time_before_crypto_handshake(timeout);
307 client_config_.set_max_idle_time_before_crypto_handshake(timeout);
308 server_config_.set_max_time_before_crypto_handshake(timeout);
309 server_config_.set_max_idle_time_before_crypto_handshake(timeout);
310
303 QuicInMemoryCachePeer::ResetForTests(); 311 QuicInMemoryCachePeer::ResetForTests();
304 AddToCache("/foo", 200, kFooResponseBody); 312 AddToCache("/foo", 200, kFooResponseBody);
305 AddToCache("/bar", 200, kBarResponseBody); 313 AddToCache("/bar", 200, kBarResponseBody);
306 } 314 }
307 315
308 ~EndToEndTest() override { 316 ~EndToEndTest() override {
309 // TODO(rtenneti): port RecycleUnusedPort if needed. 317 // TODO(rtenneti): port RecycleUnusedPort if needed.
310 // RecycleUnusedPort(server_address_.port()); 318 // RecycleUnusedPort(server_address_.port());
311 QuicInMemoryCachePeer::ResetForTests(); 319 QuicInMemoryCachePeer::ResetForTests();
312 } 320 }
(...skipping 2374 matching lines...) Expand 10 before | Expand all | Expand 10 after
2687 client_->WaitForResponse(); 2695 client_->WaitForResponse();
2688 // TODO(fayang): Fix this test to work with stateless rejects. 2696 // TODO(fayang): Fix this test to work with stateless rejects.
2689 if (!BothSidesSupportStatelessRejects()) { 2697 if (!BothSidesSupportStatelessRejects()) {
2690 VerifyCleanConnection(false); 2698 VerifyCleanConnection(false);
2691 } 2699 }
2692 } 2700 }
2693 2701
2694 } // namespace 2702 } // namespace
2695 } // namespace test 2703 } // namespace test
2696 } // namespace net 2704 } // 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