Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(313)

Unified Diff: net/quic/quic_spdy_session.cc

Issue 1870833005: relnote: Implements OnHeaderFrameStart and OnHeaderFrameEnd in QuicHeadersStream. Not used in produ… (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@06_CL_119188441
Patch Set: Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: net/quic/quic_spdy_session.cc
diff --git a/net/quic/quic_spdy_session.cc b/net/quic/quic_spdy_session.cc
index 07b202abdd23d327c62fcac905df545cf598e4d0..364b4425ec19046b657d75d8985bf302443ae822 100644
--- a/net/quic/quic_spdy_session.cc
+++ b/net/quic/quic_spdy_session.cc
@@ -63,6 +63,18 @@ void QuicSpdySession::OnStreamHeadersComplete(QuicStreamId stream_id,
stream->OnStreamHeadersComplete(fin, frame_len);
}
+void QuicSpdySession::OnStreamHeaderList(QuicStreamId stream_id,
+ bool fin,
+ size_t frame_len,
+ const QuicHeaderList& header_list) {
+ QuicSpdyStream* stream = GetSpdyDataStream(stream_id);
+ if (!stream) {
+ // It's quite possible to receive headers after a stream has been reset.
+ return;
+ }
+ stream->OnStreamHeaderList(fin, frame_len, header_list);
+}
+
size_t QuicSpdySession::WriteHeaders(
QuicStreamId id,
const SpdyHeaderBlock& headers,
@@ -113,6 +125,16 @@ void QuicSpdySession::OnPromiseHeadersComplete(QuicStreamId stream_id,
ConnectionCloseBehavior::SILENT_CLOSE);
}
+void QuicSpdySession::OnPromiseHeaderList(QuicStreamId stream_id,
+ QuicStreamId promised_stream_id,
+ size_t frame_len,
+ const QuicHeaderList& header_list) {
+ string error = "OnPromiseHeaderList should be overriden in client code.";
+ QUIC_BUG << error;
+ connection()->CloseConnection(QUIC_INTERNAL_ERROR, error,
+ ConnectionCloseBehavior::SILENT_CLOSE);
+}
+
void QuicSpdySession::OnConfigNegotiated() {
QuicSession::OnConfigNegotiated();
if (FLAGS_quic_disable_hpack_dynamic_table &&

Powered by Google App Engine
This is Rietveld 408576698