commit 578cf9fc53c5add11ffeb0a76068ff037ac0698d
author renjietang <renjietang@google.com> Fri Aug 23 11:57:52 2019 -0700
committer Copybara-Service <copybara-worker@google.com> Fri Aug 23 12:20:27 2019 -0700
tree ca518b389f2bda1c9b4f796e566c034b61ecc2ea
parent 5a2b3eb82c2179b7cffa90a31a52abd8223ef53b

Use real QPACK streams in QPACK stream tests instead of creating artificial ones.

gfe-relnote: test change only, not protected.
PiperOrigin-RevId: 265104614
Change-Id: Ide30c869535fc1ae5c03f6be110b9cfe3e55aded

quic/core/qpack/qpack_send_stream_test.cc

diff --git a/quic/core/qpack/qpack_send_stream_test.cc b/quic/core/qpack/qpack_send_stream_test.cc
index c297b56..8431afc 100644
--- a/quic/core/qpack/qpack_send_stream_test.cc
+++ b/quic/core/qpack/qpack_send_stream_test.cc
@@ -57,13 +57,8 @@
         session_(connection_) {
     session_.Initialize();
 
-    // TODO(b/112770235): Remove explicit QPACK stream construction in test once
-    // QPACK streams are created in QuicSpdySession initialization.
-    auto qpack_send = QuicMakeUnique<QpackSendStream>(
-        QuicSpdySessionPeer::GetNextOutgoingUnidirectionalStreamId(&session_),
-        &session_, kQpackEncoderStream);
-    qpack_send_stream_ = qpack_send.get();
-    session_.RegisterStaticStream(std::move(qpack_send), false);
+    qpack_send_stream_ =
+        QuicSpdySessionPeer::GetQpackDecoderSendStream(&session_);
 
     ON_CALL(session_, WritevData(_, _, _, _, _))
         .WillByDefault(Invoke(MockQuicSession::ConsumeData));