commit 40038f52168279670e1e2fe7865e1dd4c30fceca
author renjietang <renjietang@google.com> Wed Apr 08 11:30:31 2020 -0700
committer Copybara-Service <copybara-worker@google.com> Wed Apr 08 11:31:26 2020 -0700
tree 1a936e6049e34a57952d8bb50f77d3d40e0cbd4f
parent 18aef545a83c01790869257820fb11e3d5efefa2

Remove QuicStreamIdManager::IsIncomingStream() as it's hard coded. Also QuicSession doesn't need the indirectness.

QuicUtils should be the single source of truth.

gfe-relnote: no behavior change. not protected.
PiperOrigin-RevId: 305518573
Change-Id: I926c93ccd33f12c694b823d764a88d13df3e6c94

quic/core/quic_session.cc

diff --git a/quic/core/quic_session.cc b/quic/core/quic_session.cc
index d190b6c..b57ddd7 100644
--- a/quic/core/quic_session.cc
+++ b/quic/core/quic_session.cc
@@ -1840,7 +1840,7 @@
 
 bool QuicSession::IsIncomingStream(QuicStreamId id) const {
   if (VersionHasIetfQuicFrames(transport_version())) {
-    return v99_streamid_manager_.IsIncomingStream(id);
+    return !QuicUtils::IsOutgoingStreamId(version(), id, perspective_);
   }
   return stream_id_manager_.IsIncomingStream(id);
 }