OnStreamEnd is now called instead of the sentinel call of OnStreamFrameData(stream_id, nullptr, 0, …

net/quic/quic_headers_stream.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "net/quic/quic_headers_stream.h"
 
 #include "base/macros.h"
 #include "base/metrics/histogram_macros.h"
 #include "base/strings/stringprintf.h"
 #include "net/quic/quic_bug_tracker.h"
@@ skipping 48 matching lines @@
     if (fin && len == 0) {
       // The framer invokes OnStreamFrameData with zero-length data and
       // fin = true after processing a SYN_STREAM or SYN_REPLY frame
       // that had the fin bit set.
       return;
     }
     CloseConnection("SPDY DATA frame received.");
   }
 
   void OnStreamEnd(SpdyStreamId stream_id) override {
-    LOG(DFATAL) << "Unimplemented.";
+    // The framer invokes OnStreamEnd after processing a SYN_STREAM
+    // or SYN_REPLY frame that had the fin bit set.

[Ryan Hamilton, 2016/03/10 23:21:14]

Hm. I think we actually need to do something here. If you look at
QuicHeadersStream::OnControlFrameHeaderData, it looks to me like that code
expects len == 0. Do the tests pass with this CL (and flag = true)?

   }
 
   void OnStreamPadding(SpdyStreamId stream_id, size_t len) override {
     CloseConnection("SPDY frame padding received.");
   }
 
   SpdyHeadersHandlerInterface* OnHeaderFrameStart(
       SpdyStreamId stream_id) override {
     LOG(FATAL);
     return nullptr;
@@ skipping 279 matching lines @@
 
 void QuicHeadersStream::OnCompressedFrameSize(size_t frame_len) {
   frame_len_ += frame_len;
 }
 
 bool QuicHeadersStream::IsConnected() {
   return session()->connection()->connected();
 }
 
 }  // namespace net