commit ff4240d00f3efda299ed34c997db1d1abfb8f44c
author renjietang <renjietang@google.com> Wed Jul 01 06:19:55 2020 -0700
committer Copybara-Service <copybara-worker@google.com> Wed Jul 01 06:20:30 2020 -0700
tree 10c981c0bb2dcd09bfdde42418b340a25f4e7f10
parent ed1fe4f83c350bfccf42e612c1111956847721fb

Make QuicSession::ResetStream() smarter.

If the stream reseted doesn't exist, we always send out 0 as the bytes written. And if it doesn, we always send what the stream has written. So we don't need the explicit parameter for bytes_written.

No behavior change. not protected.

PiperOrigin-RevId: 319211675
Change-Id: I6c96d94e2d0ec1eca0c86a4740a7384aa15c8529

quic/test_tools/quic_test_client.cc

diff --git a/quic/test_tools/quic_test_client.cc b/quic/test_tools/quic_test_client.cc
index 503f585..f876532 100644
--- a/quic/test_tools/quic_test_client.cc
+++ b/quic/test_tools/quic_test_client.cc
@@ -423,9 +423,7 @@
 
   QuicStreamId stream_id = GetNthClientInitiatedBidirectionalStreamId(
       session->transport_version(), 0);
-  QuicStream* stream = session->GetOrCreateStream(stream_id);
-  session->ResetStream(stream_id, QUIC_STREAM_CANCELLED,
-                       stream->stream_bytes_written());
+  session->ResetStream(stream_id, QUIC_STREAM_CANCELLED);
   return ret;
 }