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

Unified Diff: net/quic/quic_headers_stream.cc

Issue 368803003: QUIC_VERSION_21: headers and crypto streams are now flow controlled at (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 6 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_headers_stream.cc
diff --git a/net/quic/quic_headers_stream.cc b/net/quic/quic_headers_stream.cc
index aab8571553d2bf4ea5525fb8465723de28907260..92f24f45b0145b83fb033a99986db236f5b635a5 100644
--- a/net/quic/quic_headers_stream.cc
+++ b/net/quic/quic_headers_stream.cc
@@ -174,7 +174,13 @@ QuicHeadersStream::QuicHeadersStream(QuicSession* session)
spdy_framer_visitor_(new SpdyFramerVisitor(this)) {
spdy_framer_.set_visitor(spdy_framer_visitor_.get());
spdy_framer_.set_debug_visitor(spdy_framer_visitor_.get());
- DisableFlowControl();
+ if (version() <= QUIC_VERSION_20) {
+ // Prior to QUIC_VERSION_21 the headers stream is not subject to any flow
+ // control.
+ DisableFlowControl();
+ }
+ // The headers stream is exempt from connection level flow control.
+ DisableConnectionFlowControlForThisStream();
}
QuicHeadersStream::~QuicHeadersStream() {}

Powered by Google App Engine
This is Rietveld 408576698