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

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

Issue 2521573006: Server push cancellation: change the ownership of ServerPushDelegate to HttpNetworkSession. (Closed)
Patch Set: rebase Created 4 years 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/http/http_network_session.cc ('k') | net/quic/chromium/quic_chromium_client_session.h » ('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 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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/bidirectional_stream_quic_impl.h" 5 #include "net/quic/chromium/bidirectional_stream_quic_impl.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/callback_helpers.h" 9 #include "base/callback_helpers.h"
10 #include "base/memory/ptr_util.h" 10 #include "base/memory/ptr_util.h"
(...skipping 421 matching lines...) Expand 10 before | Expand all | Expand 10 after
432 session_.reset(new QuicChromiumClientSession( 432 session_.reset(new QuicChromiumClientSession(
433 connection_, std::move(socket), 433 connection_, std::move(socket),
434 /*stream_factory=*/nullptr, &crypto_client_stream_factory_, &clock_, 434 /*stream_factory=*/nullptr, &crypto_client_stream_factory_, &clock_,
435 &transport_security_state_, 435 &transport_security_state_,
436 base::WrapUnique(static_cast<QuicServerInfo*>(nullptr)), 436 base::WrapUnique(static_cast<QuicServerInfo*>(nullptr)),
437 QuicServerId(kDefaultServerHostName, kDefaultServerPort, 437 QuicServerId(kDefaultServerHostName, kDefaultServerPort,
438 PRIVACY_MODE_DISABLED), 438 PRIVACY_MODE_DISABLED),
439 kQuicYieldAfterPacketsRead, 439 kQuicYieldAfterPacketsRead,
440 QuicTime::Delta::FromMilliseconds(kQuicYieldAfterDurationMilliseconds), 440 QuicTime::Delta::FromMilliseconds(kQuicYieldAfterDurationMilliseconds),
441 /*cert_verify_flags=*/0, DefaultQuicConfig(), &crypto_config_, 441 /*cert_verify_flags=*/0, DefaultQuicConfig(), &crypto_config_,
442 "CONNECTION_UNKNOWN", dns_start, dns_end, &push_promise_index_, 442 "CONNECTION_UNKNOWN", dns_start, dns_end, &push_promise_index_, nullptr,
443 base::ThreadTaskRunnerHandle::Get().get(), 443 base::ThreadTaskRunnerHandle::Get().get(),
444 /*socket_performance_watcher=*/nullptr, net_log().bound().net_log())); 444 /*socket_performance_watcher=*/nullptr, net_log().bound().net_log()));
445 session_->Initialize(); 445 session_->Initialize();
446 TestCompletionCallback callback; 446 TestCompletionCallback callback;
447 session_->CryptoConnect(/*require_confirmation=*/false, 447 session_->CryptoConnect(/*require_confirmation=*/false,
448 callback.callback()); 448 callback.callback());
449 EXPECT_TRUE(session_->IsEncryptionEstablished()); 449 EXPECT_TRUE(session_->IsEncryptionEstablished());
450 } 450 }
451 451
452 void ConfirmHandshake() { 452 void ConfirmHandshake() {
(...skipping 1389 matching lines...) Expand 10 before | Expand all | Expand 10 after
1842 1842
1843 base::RunLoop().RunUntilIdle(); 1843 base::RunLoop().RunUntilIdle();
1844 1844
1845 EXPECT_EQ(1, delegate->on_data_read_count()); 1845 EXPECT_EQ(1, delegate->on_data_read_count());
1846 EXPECT_EQ(0, delegate->on_data_sent_count()); 1846 EXPECT_EQ(0, delegate->on_data_sent_count());
1847 } 1847 }
1848 1848
1849 } // namespace test 1849 } // namespace test
1850 1850
1851 } // namespace net 1851 } // namespace net
OLDNEW
« no previous file with comments | « net/http/http_network_session.cc ('k') | net/quic/chromium/quic_chromium_client_session.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698