Remove QuicSpdySession::CanCreatePushStreamWithId().
PiperOrigin-RevId: 431032880
diff --git a/quic/core/http/quic_spdy_client_session_base.cc b/quic/core/http/quic_spdy_client_session_base.cc
index 56a9764..b761a19 100644
--- a/quic/core/http/quic_spdy_client_session_base.cc
+++ b/quic/core/http/quic_spdy_client_session_base.cc
@@ -87,8 +87,9 @@
return;
}
- if (VersionUsesHttp3(transport_version()) &&
- !CanCreatePushStreamWithId(promised_stream_id)) {
+ if (VersionUsesHttp3(transport_version())) {
+ // Received push stream id is higher than MAX_PUSH_ID
+ // because no MAX_PUSH_ID frame is ever sent.
connection()->CloseConnection(
QUIC_INVALID_STREAM_ID,
"Received push stream id higher than MAX_PUSH_ID.",
diff --git a/quic/core/http/quic_spdy_session.cc b/quic/core/http/quic_spdy_session.cc
index 7b63328..6a57107 100644
--- a/quic/core/http/quic_spdy_session.cc
+++ b/quic/core/http/quic_spdy_session.cc
@@ -1592,13 +1592,6 @@
: transport_goaway_sent();
}
-bool QuicSpdySession::CanCreatePushStreamWithId(PushId /* push_id */) {
- // TODO(b/171463363): Remove this method.
- QUICHE_DCHECK(VersionUsesHttp3(transport_version()));
-
- return false;
-}
-
void QuicSpdySession::CloseConnectionOnDuplicateHttp3UnidirectionalStreams(
absl::string_view type) {
QUIC_PEER_BUG(quic_peer_bug_10360_9) << absl::StrCat(
diff --git a/quic/core/http/quic_spdy_session.h b/quic/core/http/quic_spdy_session.h
index 50603de..9756e24 100644
--- a/quic/core/http/quic_spdy_session.h
+++ b/quic/core/http/quic_spdy_session.h
@@ -331,10 +331,6 @@
// TODO(b/171463363): Remove.
bool OnMaxPushIdFrame(PushId max_push_id);
- // TODO(b/171463363): Remove.
- // Returns false.
- bool CanCreatePushStreamWithId(PushId push_id);
-
int32_t destruction_indicator() const { return destruction_indicator_; }
void set_debug_visitor(Http3DebugVisitor* debug_visitor) {