Add a new SpdyStream::Delegate method to handle trailers

net/spdy/spdy_http_stream.cc

 // Copyright (c) 2012 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/spdy/spdy_http_stream.h"
 
 #include <algorithm>
 #include <list>
 
 #include "base/bind.h"
@@ skipping 347 matching lines @@
       ScheduleBufferedReadCallback();
     }
   }
 }
 
 void SpdyHttpStream::OnDataSent() {
   request_body_buf_size_ = 0;
   ReadAndSendRequestBodyData();
 }
 
+void SpdyHttpStream::OnTrailers(const SpdyHeaderBlock& trailers) {}

[Ryan Hamilton, 2015/08/10 22:37:38]

nit: Can you add a TODO here to do something with them? Does chrome support
trailers for normal HTTP/1 responses? I assume so, but maybe not?

[xunjieli, 2015/08/11 15:07:14]

Done. Good idea! I added the bug number as well. According to crbug.com/422958,
Chrome does not support trailers for HTTP/1.

+
 void SpdyHttpStream::OnClose(int status) {
   if (stream_.get()) {
     stream_closed_ = true;
     closed_stream_status_ = status;
     closed_stream_id_ = stream_->stream_id();
     closed_stream_has_load_timing_info_ =
         stream_->GetLoadTimingInfo(&closed_stream_load_timing_info_);
     closed_stream_received_bytes_ = stream_->raw_received_bytes();
   }
   stream_.reset();
@@ skipping 155 matching lines @@
   Close(false);
   delete this;
 }
 
 void SpdyHttpStream::SetPriority(RequestPriority priority) {
   // TODO(akalin): Plumb this through to |stream_request_| and
   // |stream_|.
 }
 
 }  // namespace net