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

Unified Diff: trunk/src/net/quic/reliable_quic_stream.cc

Issue 16374004: Revert 204046 "Land Recent QUIC changes." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 7 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
« no previous file with comments | « trunk/src/net/quic/quic_stream_factory_test.cc ('k') | trunk/src/net/quic/reliable_quic_stream_test.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: trunk/src/net/quic/reliable_quic_stream.cc
===================================================================
--- trunk/src/net/quic/reliable_quic_stream.cc (revision 204061)
+++ trunk/src/net/quic/reliable_quic_stream.cc (working copy)
@@ -95,12 +95,7 @@
void ReliableQuicStream::Close(QuicRstStreamErrorCode error) {
stream_error_ = error;
- if (error != QUIC_STREAM_NO_ERROR) {
- // Sending a RstStream results in calling CloseStream.
- session()->SendRstStream(id(), error);
- } else {
- session_->CloseStream(id());
- }
+ session()->SendRstStream(id(), error);
}
int ReliableQuicStream::Readv(const struct iovec* iov, size_t iov_len) {
@@ -160,7 +155,6 @@
}
QuicConsumedData ReliableQuicStream::WriteData(StringPiece data, bool fin) {
- DCHECK(data.size() > 0 || fin);
return WriteOrBuffer(data, fin);
}
@@ -218,8 +212,6 @@
if (fin && consumed_data.fin_consumed) {
fin_sent_ = true;
CloseWriteSide();
- } else if (fin && !consumed_data.fin_consumed) {
- session_->MarkWriteBlocked(id());
}
} else {
session_->MarkWriteBlocked(id());
« no previous file with comments | « trunk/src/net/quic/quic_stream_factory_test.cc ('k') | trunk/src/net/quic/reliable_quic_stream_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698