commit 647b3cf62e9d2599ee9564f29675308e9b24b585
author renjietang <renjietang@google.com> Tue Aug 04 13:23:12 2020 -0700
committer Copybara-Service <copybara-worker@google.com> Tue Aug 04 13:23:49 2020 -0700
tree 865fa785a8c43bb859aa71c19462e9e36c6df419
parent b7abebbf5c181b6ecf6f5241152c9e54935d7f56

Remove zombie stream map in QuicSession.

Those streams will live in stream_map_ instead.

This CL also removes some duplicated tests.

Protected by gfe2_reloadable_flag_quic_remove_zombie_streams

PiperOrigin-RevId: 324877471
Change-Id: I6d3df84048c35e7ca52ec030e019bbc0def4d3ce

quic/core/quic_stream.cc

diff --git a/quic/core/quic_stream.cc b/quic/core/quic_stream.cc
index 9e695d4..ee3f983 100644
--- a/quic/core/quic_stream.cc
+++ b/quic/core/quic_stream.cc
@@ -979,7 +979,8 @@
     fin_outstanding_ = false;
     fin_lost_ = false;
   }
-  if (!IsWaitingForAcks()) {
+  if (!IsWaitingForAcks() && (!session()->remove_zombie_streams() ||
+                              (read_side_closed_ && write_side_closed_))) {
     session_->OnStreamDoneWaitingForAcks(id_);
   }
   return new_data_acked;