commit c8c02a5f788cddb615e5f0fb2636dea67cad2396
author renjietang <renjietang@google.com> Thu Aug 22 10:38:37 2019 -0700
committer Copybara-Service <copybara-worker@google.com> Thu Aug 22 10:40:12 2019 -0700
tree 7dc12387fd59510d4e76bcdd80f4cc8f24dd94be
parent a7fe5779a2836219e20f7018411f32f5770d517c

Make sure QPACK stream types are sent along with SETTINGS in one packet.

gfe-relnote: v99 only, not protected.

Merge intruction: I wrote https://chromium-review.googlesource.com/c/chromium/src/+/1764346 to modify TestPakcetMaker so that v99 tests can be run again.

PiperOrigin-RevId: 264867694
Change-Id: I4d2453abb2f608d39f705c268ff7da265c69e22f

quic/core/qpack/qpack_send_stream.cc

diff --git a/quic/core/qpack/qpack_send_stream.cc b/quic/core/qpack/qpack_send_stream.cc
index b978985..50fe687 100644
--- a/quic/core/qpack/qpack_send_stream.cc
+++ b/quic/core/qpack/qpack_send_stream.cc
@@ -25,11 +25,11 @@
 
 void QpackSendStream::WriteStreamData(QuicStringPiece data) {
   QuicConnection::ScopedPacketFlusher flusher(session()->connection());
-  SendStreamType();
+  MaybeSendStreamType();
   WriteOrBufferData(data, false, nullptr);
 }
 
-void QpackSendStream::SendStreamType() {
+void QpackSendStream::MaybeSendStreamType() {
   if (!stream_type_sent_) {
     char type[sizeof(http3_stream_type_)];
     QuicDataWriter writer(QUIC_ARRAYSIZE(type), type);