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

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: address comments 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
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 1320 matching lines...) Expand 10 before | Expand all | Expand 10 after
1773 1773
1774 base::RunLoop().RunUntilIdle(); 1774 base::RunLoop().RunUntilIdle();
1775 1775
1776 EXPECT_EQ(1, delegate->on_data_read_count()); 1776 EXPECT_EQ(1, delegate->on_data_read_count());
1777 EXPECT_EQ(0, delegate->on_data_sent_count()); 1777 EXPECT_EQ(0, delegate->on_data_sent_count());
1778 } 1778 }
1779 1779
1780 } // namespace test 1780 } // namespace test
1781 1781
1782 } // namespace net 1782 } // namespace net
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698