commit fe8f9c18f5d39b665d13e0cd6212bc8eb747799e
author fayang <fayang@google.com> Wed Mar 18 16:17:38 2020 -0700
committer Copybara-Service <copybara-worker@google.com> Wed Mar 18 16:18:16 2020 -0700
tree 8992bd0419b3a7b9813f73070334ff824cbf9bad
parent 501f768a5ce074c91389dc28298c13488f8087fa

gfe-relnote: In QUIC, remove is_config_negotiated_ from QuicStreamIdManager. No functional change expected, not protected.

PiperOrigin-RevId: 301691294
Change-Id: I422896ecea4648b6f53ddd64c76923117fccb5b9

quic/core/quic_session.cc

diff --git a/quic/core/quic_session.cc b/quic/core/quic_session.cc
index a10f415..004e334 100644
--- a/quic/core/quic_session.cc
+++ b/quic/core/quic_session.cc
@@ -840,11 +840,19 @@
 
 void QuicSession::SendMaxStreams(QuicStreamCount stream_count,
                                  bool unidirectional) {
+  if (!is_configured_) {
+    QUIC_BUG << "Try to send max streams before config negotiated.";
+    return;
+  }
   control_frame_manager_.WriteOrBufferMaxStreams(stream_count, unidirectional);
 }
 
 void QuicSession::SendStreamsBlocked(QuicStreamCount stream_count,
                                      bool unidirectional) {
+  if (!is_configured_) {
+    QUIC_BUG << "Try to send stream blocked before config negotiated.";
+    return;
+  }
   control_frame_manager_.WriteOrBufferStreamsBlocked(stream_count,
                                                      unidirectional);
 }
@@ -1137,14 +1145,6 @@
   is_configured_ = true;
   connection()->OnConfigNegotiated();
 
-  // Inform stream ID manager so that it can reevaluate any deferred
-  // STREAMS_BLOCKED or MAX_STREAMS frames against the config and either send
-  // the frames or discard them.
-  if (VersionHasIetfQuicFrames(connection_->transport_version())) {
-    QuicConnection::ScopedPacketFlusher flusher(connection());
-    v99_streamid_manager_.OnConfigNegotiated();
-  }
-
   // Ask flow controllers to try again since the config could have unblocked us.
   if (connection_->version().AllowsLowFlowControlLimits()) {
     OnCanWrite();