commit e5c1238dd96a6a0258430f02cdf069f0d985b187
author renjietang <renjietang@google.com> Thu Nov 07 10:13:14 2019 -0800
committer Copybara-Service <copybara-worker@google.com> Thu Nov 07 10:13:50 2019 -0800
tree 24f2fd6fd09f12113984a6ef8cf4d862add71ca5
parent d8bd48b01415b23d96096de03002ef3986f5a3ad

Cleanup unused code in QuicStream.

gfe-relnote: unused code. Not protected.
PiperOrigin-RevId: 279112591
Change-Id: Iff0606671902211f8db67b4fd1e9fe668651e8d8

quic/core/quic_session_test.cc

diff --git a/quic/core/quic_session_test.cc b/quic/core/quic_session_test.cc
index c148961..d5d8321 100644
--- a/quic/core/quic_session_test.cc
+++ b/quic/core/quic_session_test.cc
@@ -143,7 +143,6 @@
                bool(QuicStreamOffset, QuicByteCount, bool));
 
   MOCK_CONST_METHOD0(HasPendingRetransmission, bool());
-  MOCK_METHOD1(OnStopSending, void(uint16_t code));
 };
 
 class TestSession : public QuicSession {
@@ -2698,7 +2697,6 @@
 
   QuicStreamId stream_id = stream->id();
   QuicStopSendingFrame frame(1, stream_id, 123);
-  EXPECT_CALL(*stream, OnStopSending(123));
   // Expect a reset to come back out.
   EXPECT_CALL(*connection_, SendControlFrame(_));
   EXPECT_CALL(
@@ -2706,8 +2704,7 @@
       OnStreamReset(stream_id, static_cast<QuicRstStreamErrorCode>(123)));
   EXPECT_CALL(*connection_, CloseConnection(_, _, _)).Times(0);
   session_.OnStopSendingFrame(frame);
-  // When the STOP_SENDING is received, the node generates a RST_STREAM,
-  // which closes the stream in the write direction. Ensure this.
+
   EXPECT_FALSE(QuicStreamPeer::read_side_closed(stream));
   EXPECT_TRUE(stream->write_side_closed());
 }