Add a variety of DCHECK in the toy-server class to ensure that the backend is
non-null and remove the guard added in cr/246231018.
The underlying issue was a bug in the Chromium QuicSimplerServer which
is fixed in:
https://chromium-review.googlesource.com/c/chromium/src/+/1594063
gfe-relnote: n/a - Not used in the GFE
PiperOrigin-RevId: 246429041
Change-Id: I76103b0d992950fef950ba5bb8d508fe40ae5436
diff --git a/quic/tools/quic_simple_server_stream.cc b/quic/tools/quic_simple_server_stream.cc
index e3fc507..55d93dc 100644
--- a/quic/tools/quic_simple_server_stream.cc
+++ b/quic/tools/quic_simple_server_stream.cc
@@ -29,7 +29,9 @@
QuicSimpleServerBackend* quic_simple_server_backend)
: QuicSpdyServerStreamBase(id, session, type),
content_length_(-1),
- quic_simple_server_backend_(quic_simple_server_backend) {}
+ quic_simple_server_backend_(quic_simple_server_backend) {
+ DCHECK(quic_simple_server_backend_);
+}
QuicSimpleServerStream::QuicSimpleServerStream(
PendingStream pending,
@@ -38,12 +40,12 @@
QuicSimpleServerBackend* quic_simple_server_backend)
: QuicSpdyServerStreamBase(std::move(pending), session, type),
content_length_(-1),
- quic_simple_server_backend_(quic_simple_server_backend) {}
+ quic_simple_server_backend_(quic_simple_server_backend) {
+ DCHECK(quic_simple_server_backend_);
+}
QuicSimpleServerStream::~QuicSimpleServerStream() {
- if (quic_simple_server_backend_) {
- quic_simple_server_backend_->CloseBackendResponseStream(this);
- }
+ quic_simple_server_backend_->CloseBackendResponseStream(this);
}
void QuicSimpleServerStream::OnInitialHeadersComplete(
