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

Side by Side Diff: net/quic/chromium/quic_stream_factory_test.cc

Issue 2889273003: Fix some non-const ref function argument types. (Closed)
Patch Set: Created 3 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 | « net/nqe/observation_buffer.h ('k') | net/ssl/ssl_config_unittest.cc » ('j') | 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 "net/quic/chromium/quic_stream_factory.h" 5 #include "net/quic/chromium/quic_stream_factory.h"
6 6
7 #include <memory> 7 #include <memory>
8 #include <ostream> 8 #include <ostream>
9 #include <utility> 9 #include <utility>
10 10
(...skipping 384 matching lines...) Expand 10 before | Expand all | Expand 10 after
395 return client_maker_.MakeInitialSettingsPacket(1, nullptr); 395 return client_maker_.MakeInitialSettingsPacket(1, nullptr);
396 } 396 }
397 397
398 std::unique_ptr<QuicReceivedPacket> ConstructInitialSettingsPacket( 398 std::unique_ptr<QuicReceivedPacket> ConstructInitialSettingsPacket(
399 QuicPacketNumber packet_number, 399 QuicPacketNumber packet_number,
400 QuicStreamOffset* offset) { 400 QuicStreamOffset* offset) {
401 return client_maker_.MakeInitialSettingsPacket(packet_number, offset); 401 return client_maker_.MakeInitialSettingsPacket(packet_number, offset);
402 } 402 }
403 403
404 // Helper method for server migration tests. 404 // Helper method for server migration tests.
405 void VerifyServerMigration(QuicConfig& config, IPEndPoint expected_address) { 405 void VerifyServerMigration(const QuicConfig& config,
406 IPEndPoint expected_address) {
406 allow_server_migration_ = true; 407 allow_server_migration_ = true;
407 Initialize(); 408 Initialize();
408 409
409 ProofVerifyDetailsChromium verify_details = DefaultProofVerifyDetails(); 410 ProofVerifyDetailsChromium verify_details = DefaultProofVerifyDetails();
410 crypto_client_stream_factory_.AddProofVerifyDetails(&verify_details); 411 crypto_client_stream_factory_.AddProofVerifyDetails(&verify_details);
411 crypto_client_stream_factory_.SetConfig(config); 412 crypto_client_stream_factory_.SetConfig(config);
412 413
413 // Set up first socket data provider. 414 // Set up first socket data provider.
414 MockQuicData socket_data1; 415 MockQuicData socket_data1;
415 socket_data1.AddRead(SYNCHRONOUS, ERR_IO_PENDING); 416 socket_data1.AddRead(SYNCHRONOUS, ERR_IO_PENDING);
(...skipping 4655 matching lines...) Expand 10 before | Expand all | Expand 10 after
5071 // Clear all cached states. 5072 // Clear all cached states.
5072 factory_->ClearCachedStatesInCryptoConfig( 5073 factory_->ClearCachedStatesInCryptoConfig(
5073 base::Callback<bool(const GURL&)>()); 5074 base::Callback<bool(const GURL&)>());
5074 EXPECT_TRUE(test_cases[0].state->certs().empty()); 5075 EXPECT_TRUE(test_cases[0].state->certs().empty());
5075 EXPECT_TRUE(test_cases[1].state->certs().empty()); 5076 EXPECT_TRUE(test_cases[1].state->certs().empty());
5076 EXPECT_TRUE(test_cases[2].state->certs().empty()); 5077 EXPECT_TRUE(test_cases[2].state->certs().empty());
5077 } 5078 }
5078 5079
5079 } // namespace test 5080 } // namespace test
5080 } // namespace net 5081 } // namespace net
OLDNEW
« no previous file with comments | « net/nqe/observation_buffer.h ('k') | net/ssl/ssl_config_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698