commit df2240d195094ff96457181586a6408308397f37
author renjietang <renjietang@google.com> Fri Jul 17 12:17:00 2020 -0700
committer Copybara-Service <copybara-worker@google.com> Fri Jul 17 12:17:37 2020 -0700
tree db7fa795e59d5540f97010850d71506eb29c462e
parent 76690759ed2275b0a465659c951a34c5c8ea471d

Remove streams_waiting_for_acks_ in QuicSession because it stores duplicate information and only reduces minimal time complexity.

Session shouldn't care this much about streams' states.

Protected by gfe2_reloadable_flag_quic_remove_streams_waiting_for_acks

PiperOrigin-RevId: 321829178
Change-Id: I3589810eb553b1b743799b7ab51990fbd42e1008

quic/core/quic_stream.h

diff --git a/quic/core/quic_stream.h b/quic/core/quic_stream.h
index 41484a3..2a7d036 100644
--- a/quic/core/quic_stream.h
+++ b/quic/core/quic_stream.h
@@ -218,9 +218,6 @@
   size_t busy_counter() const { return busy_counter_; }
   void set_busy_counter(size_t busy_counter) { busy_counter_ = busy_counter; }
 
-  void set_rst_received(bool rst_received) { rst_received_ = rst_received; }
-  void set_stream_error(QuicRstStreamErrorCode error) { stream_error_ = error; }
-
   // Adjust the flow control window according to new offset in |frame|.
   virtual void OnWindowUpdateFrame(const QuicWindowUpdateFrame& frame);
 
@@ -408,6 +405,9 @@
   // empty.
   void SetFinSent();
 
+  void set_rst_received(bool rst_received) { rst_received_ = rst_received; }
+  void set_stream_error(QuicRstStreamErrorCode error) { stream_error_ = error; }
+
   StreamDelegateInterface* stream_delegate() { return stream_delegate_; }
 
   bool fin_buffered() const { return fin_buffered_; }