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

Unified Diff: net/quic/quic_session.cc

Issue 786953009: Removing quic version 21 (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@Fixes_typo_83546153
Patch Set: Created 5 years, 11 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
« no previous file with comments | « net/quic/quic_server_session.cc ('k') | net/tools/quic/quic_server_session.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/quic_session.cc
diff --git a/net/quic/quic_session.cc b/net/quic/quic_session.cc
index 865af07cd0dbddbb2ecdd1c70d083c6459c381ff..bddc18ef2e8b28203e1a842c1dc0788030adbca0 100644
--- a/net/quic/quic_session.cc
+++ b/net/quic/quic_session.cc
@@ -255,13 +255,6 @@ void QuicSession::OnWindowUpdateFrames(
continue;
}
- if (connection_->version() < QUIC_VERSION_21 &&
- (stream_id == kCryptoStreamId || stream_id == kHeadersStreamId)) {
- DLOG(DFATAL) << "WindowUpdate for stream " << stream_id << " in version "
- << QuicVersionToString(connection_->version());
- return;
- }
-
ReliableQuicStream* stream = GetStream(stream_id);
if (stream) {
stream->OnWindowUpdateFrame(frames[i]);
@@ -498,10 +491,8 @@ void QuicSession::OnNewStreamFlowControlWindow(QuicStreamOffset new_window) {
}
// Inform all existing streams about the new window.
- if (connection_->version() >= QUIC_VERSION_21) {
- GetCryptoStream()->UpdateSendWindowOffset(new_window);
- headers_stream_->UpdateSendWindowOffset(new_window);
- }
+ GetCryptoStream()->UpdateSendWindowOffset(new_window);
+ headers_stream_->UpdateSendWindowOffset(new_window);
for (DataStreamMap::iterator it = stream_map_.begin();
it != stream_map_.end(); ++it) {
it->second->UpdateSendWindowOffset(new_window);
« no previous file with comments | « net/quic/quic_server_session.cc ('k') | net/tools/quic/quic_server_session.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698