commit e238a1377a67a6e5dc04c73601cc0b50fa725f1b
author renjietang <renjietang@google.com> Wed Aug 12 16:46:41 2020 -0700
committer Copybara-Service <copybara-worker@google.com> Wed Aug 12 16:47:24 2020 -0700
tree 110ebf65ecd1492b8e0cb13b4fb1f3c7c152a229
parent 8957c6e926375f405747af0522d9f71c70445e5c

Remove an redundant if clause in QuicSession::OnNewStreamFlowControlWindow().

This code is reached only in QUIC with QUIC crypto, thus !connection_->version().AllowsLowFlowControlLimits() is always true.

No behavior change. not protected.

PiperOrigin-RevId: 326341408
Change-Id: I0969f18acc73272d3b8dcfc153896a2eaca31e99

quic/core/quic_session.cc

diff --git a/quic/core/quic_session.cc b/quic/core/quic_session.cc
index 59bf083..715b518 100644
--- a/quic/core/quic_session.cc
+++ b/quic/core/quic_session.cc
@@ -1348,8 +1348,7 @@
 void QuicSession::OnNewStreamFlowControlWindow(QuicStreamOffset new_window) {
   DCHECK(version().UsesQuicCrypto());
   QUIC_DVLOG(1) << ENDPOINT << "OnNewStreamFlowControlWindow " << new_window;
-  if (new_window < kMinimumFlowControlSendWindow &&
-      !connection_->version().AllowsLowFlowControlLimits()) {
+  if (new_window < kMinimumFlowControlSendWindow) {
     QUIC_LOG_FIRST_N(ERROR, 1)
         << "Peer sent us an invalid stream flow control send window: "
         << new_window << ", below minimum: " << kMinimumFlowControlSendWindow;