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

Unified Diff: net/tools/quic/end_to_end_test.cc

Issue 2823223003: Rename QuicSession::GetCryptoStream() to GetMutableCryptoStream(). Add a const GetCryptoStream(). M… (Closed)
Patch Set: Created 3 years, 8 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « net/quic/test_tools/quic_test_utils.cc ('k') | net/tools/quic/quic_client_session.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/tools/quic/end_to_end_test.cc
diff --git a/net/tools/quic/end_to_end_test.cc b/net/tools/quic/end_to_end_test.cc
index 3ad5c0533215ec41fa5dc18bf0c6c1e6f754b5a7..aad4fe01dd798739bbd243bb9bb250edcbc719e0 100644
--- a/net/tools/quic/end_to_end_test.cc
+++ b/net/tools/quic/end_to_end_test.cc
@@ -633,7 +633,7 @@ TEST_P(EndToEndTest, HandshakeSuccessful) {
ASSERT_TRUE(Initialize());
EXPECT_TRUE(client_->client()->WaitForCryptoHandshakeConfirmed());
QuicCryptoStream* crypto_stream =
- QuicSessionPeer::GetCryptoStream(client_->client()->session());
+ QuicSessionPeer::GetMutableCryptoStream(client_->client()->session());
QuicStreamSequencer* sequencer = QuicStreamPeer::sequencer(crypto_stream);
EXPECT_NE(FLAGS_quic_reloadable_flag_quic_release_crypto_stream_buffer,
QuicStreamSequencerPeer::IsUnderlyingBufferAllocated(sequencer));
@@ -641,7 +641,7 @@ TEST_P(EndToEndTest, HandshakeSuccessful) {
QuicDispatcher* dispatcher =
QuicServerPeer::GetDispatcher(server_thread_->server());
QuicSession* server_session = dispatcher->session_map().begin()->second.get();
- crypto_stream = QuicSessionPeer::GetCryptoStream(server_session);
+ crypto_stream = QuicSessionPeer::GetMutableCryptoStream(server_session);
sequencer = QuicStreamPeer::sequencer(crypto_stream);
EXPECT_NE(FLAGS_quic_reloadable_flag_quic_release_crypto_stream_buffer,
QuicStreamSequencerPeer::IsUnderlyingBufferAllocated(sequencer));
@@ -1764,7 +1764,7 @@ TEST_P(EndToEndTest, HeadersAndCryptoStreamsNoConnectionFlowControl) {
server_thread_->WaitForCryptoHandshakeConfirmed();
QuicCryptoStream* crypto_stream =
- QuicSessionPeer::GetCryptoStream(client_->client()->session());
+ QuicSessionPeer::GetMutableCryptoStream(client_->client()->session());
EXPECT_LT(
QuicFlowControllerPeer::SendWindowSize(crypto_stream->flow_controller()),
kStreamIFCW);
@@ -1815,8 +1815,10 @@ TEST_P(EndToEndTest, FlowControlsSynced) {
ExpectFlowControlsSynced(client_session->flow_controller(),
server_session->flow_controller());
ExpectFlowControlsSynced(
- QuicSessionPeer::GetCryptoStream(client_session)->flow_controller(),
- QuicSessionPeer::GetCryptoStream(server_session)->flow_controller());
+ QuicSessionPeer::GetMutableCryptoStream(client_session)
+ ->flow_controller(),
+ QuicSessionPeer::GetMutableCryptoStream(server_session)
+ ->flow_controller());
SpdyFramer spdy_framer(SpdyFramer::ENABLE_COMPRESSION);
SpdySettingsIR settings_frame;
settings_frame.AddSetting(SETTINGS_MAX_HEADER_LIST_SIZE,
« no previous file with comments | « net/quic/test_tools/quic_test_utils.cc ('k') | net/tools/quic/quic_client_session.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698