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

Side by Side Diff: net/quic/core/quic_crypto_server_stream_test.cc

Issue 2459283003: Remove unused argument from CryptoTestUtils::SetupCryptoServerConfigForTest (Closed)
Patch Set: Created 4 years, 1 month 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
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 "net/quic/core/quic_crypto_server_stream.h" 5 #include "net/quic/core/quic_crypto_server_stream.h"
6 6
7 #include <map> 7 #include <map>
8 #include <memory> 8 #include <memory>
9 #include <vector> 9 #include <vector>
10 10
(...skipping 96 matching lines...) Expand 10 before | Expand all | Expand 10 after
107 server_id_, QuicTime::Delta::FromSeconds(100000), supported_versions_, 107 server_id_, QuicTime::Delta::FromSeconds(100000), supported_versions_,
108 helpers_.back().get(), alarm_factories_.back().get(), 108 helpers_.back().get(), alarm_factories_.back().get(),
109 &server_crypto_config_, &server_compressed_certs_cache_, 109 &server_crypto_config_, &server_compressed_certs_cache_,
110 &server_connection_, &server_session); 110 &server_connection_, &server_session);
111 CHECK(server_session); 111 CHECK(server_session);
112 server_session_.reset(server_session); 112 server_session_.reset(server_session);
113 CryptoTestUtils::FakeServerOptions options; 113 CryptoTestUtils::FakeServerOptions options;
114 options.token_binding_params = QuicTagVector{kTB10}; 114 options.token_binding_params = QuicTagVector{kTB10};
115 CryptoTestUtils::SetupCryptoServerConfigForTest( 115 CryptoTestUtils::SetupCryptoServerConfigForTest(
116 server_connection_->clock(), server_connection_->random_generator(), 116 server_connection_->clock(), server_connection_->random_generator(),
117 server_session_->config(), &server_crypto_config_, options); 117 &server_crypto_config_, options);
118 } 118 }
119 119
120 QuicCryptoServerStream* server_stream() { 120 QuicCryptoServerStream* server_stream() {
121 return server_session_->GetCryptoStream(); 121 return server_session_->GetCryptoStream();
122 } 122 }
123 123
124 QuicCryptoClientStream* client_stream() { 124 QuicCryptoClientStream* client_stream() {
125 return client_session_->GetCryptoStream(); 125 return client_session_->GetCryptoStream();
126 } 126 }
127 127
(...skipping 490 matching lines...) Expand 10 before | Expand all | Expand 10 after
618 // Regression test for b/31521252, in which a crash would happen here. 618 // Regression test for b/31521252, in which a crash would happen here.
619 AdvanceHandshakeWithFakeClient(); 619 AdvanceHandshakeWithFakeClient();
620 EXPECT_FALSE(server_stream()->encryption_established()); 620 EXPECT_FALSE(server_stream()->encryption_established());
621 EXPECT_FALSE(server_stream()->handshake_confirmed()); 621 EXPECT_FALSE(server_stream()->handshake_confirmed());
622 } 622 }
623 623
624 } // namespace 624 } // namespace
625 625
626 } // namespace test 626 } // namespace test
627 } // namespace net 627 } // namespace net
OLDNEW
« no previous file with comments | « net/quic/core/quic_crypto_client_stream_test.cc ('k') | net/quic/test_tools/crypto_test_utils.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698