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

Unified Diff: net/quic/quic_spdy_stream.cc

Issue 2176323002: Deprecate FLAGS_quic_disable_pre_30. Remove QUIC versions [25-29]. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@127879468
Patch Set: Created 4 years, 5 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_stream.cc
diff --git a/net/quic/quic_spdy_stream.cc b/net/quic/quic_spdy_stream.cc
index 3cbceb64ea6f28f7886fd29bb77d714d558b484c..b4cc7b6baad44d7071dea92c3589886dd2badaa9 100644
--- a/net/quic/quic_spdy_stream.cc
+++ b/net/quic/quic_spdy_stream.cc
@@ -47,8 +47,8 @@ QuicSpdyStream::~QuicSpdyStream() {
}
void QuicSpdyStream::CloseWriteSide() {
- if (version() > QUIC_VERSION_28 && !fin_received() && !rst_received() &&
- sequencer()->ignore_read_data() && !rst_sent()) {
+ if (!fin_received() && !rst_received() && sequencer()->ignore_read_data() &&
+ !rst_sent()) {
DCHECK(fin_sent());
// Tell the peer to stop sending further data.
DVLOG(1) << ENDPOINT << "Send QUIC_STREAM_NO_ERROR on stream " << id();
@@ -59,8 +59,8 @@ void QuicSpdyStream::CloseWriteSide() {
}
void QuicSpdyStream::StopReading() {
- if (version() > QUIC_VERSION_28 && !fin_received() && !rst_received() &&
- write_side_closed() && !rst_sent()) {
+ if (!fin_received() && !rst_received() && write_side_closed() &&
+ !rst_sent()) {
DCHECK(fin_sent());
// Tell the peer to stop sending further data.
DVLOG(1) << ENDPOINT << "Send QUIC_STREAM_NO_ERROR on stream " << id();
@@ -332,8 +332,7 @@ void QuicSpdyStream::OnTrailingHeadersComplete(
}
void QuicSpdyStream::OnStreamReset(const QuicRstStreamFrame& frame) {
- if (frame.error_code != QUIC_STREAM_NO_ERROR ||
- version() <= QUIC_VERSION_28) {
+ if (frame.error_code != QUIC_STREAM_NO_ERROR) {
ReliableQuicStream::OnStreamReset(frame);
return;
}

Powered by Google App Engine
This is Rietveld 408576698