QuicChromiumClientStream should only do OnStream after notifying delegate about trailers

net/quic/quic_spdy_stream.cc

Index: net/quic/quic_spdy_stream.cc
diff --git a/net/quic/quic_spdy_stream.cc b/net/quic/quic_spdy_stream.cc
index 75b160e2d72501343580e81d2fc9c2445427e4d2..8ffd2e0a6272062aa018b5e532e2f7a3bdef7619 100644
--- a/net/quic/quic_spdy_stream.cc
+++ b/net/quic/quic_spdy_stream.cc
@@ -213,7 +213,6 @@ void QuicSpdyStream::OnTrailingHeadersComplete(bool fin, size_t /*frame_len*/) {
     return;
   }
 
-  OnStreamFrame(QuicStreamFrame(id(), fin, stream_bytes_read(), StringPiece()));

[Ryan Hamilton, 2016/03/08 04:12:43]

I thought that you and robbie agreed that this was the right place for this
change?

Did you ever merge this back to google3?

[xunjieli, 2016/03/08 14:46:02]

Robbie and I agreed that we need to add a OnStreamFrame. However, if we put
OnStreamFrame here, it will call OnDataAvailable with Fin flag before we notify
delegate of trailers. This isn't correct, because once delegate consumes FIN
flag, it thinks that everything is done. (The code is fine for initial headers,
because there's some logic to block the sequencer, so we delay OnDataAvailable
until the initial headers). 

This is discovered when I did more testing.

Sorry, I thought you'd help me with merging with Google3, so I didn't do
anything. Could you help me?

   trailers_decompressed_ = true;
 }