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

Unified Diff: net/quic/quic_session.cc

Issue 1761263002: Landing recent QUIC changes until 7:19 PM, Feb 26, 2016 UTC-5. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add ActivateStream() call to QuicChromiumClientSession to fix server push failure Created 4 years, 10 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_session.h ('k') | net/quic/quic_session_test.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 e3aad3f6dc5618ca7145f5ace1db7df21a5fc525..d8ec06c2f21273da9eb764c6eb96fba5fbcd0169 100644
--- a/net/quic/quic_session.cc
+++ b/net/quic/quic_session.cc
@@ -675,12 +675,7 @@ ReliableQuicStream* QuicSession::GetOrCreateDynamicStream(
}
// Check if the new number of open streams would cause the number of
// open streams to exceed the limit.
- size_t num_open_incoming_streams =
- FLAGS_quic_distinguish_incoming_outgoing_streams
- ? GetNumOpenIncomingStreams()
- : dynamic_stream_map_.size() - draining_streams_.size() +
- locally_closed_streams_highest_offset_.size();
- if (num_open_incoming_streams >= max_open_incoming_streams()) {
+ if (GetNumOpenIncomingStreams() >= max_open_incoming_streams()) {
if (connection()->version() <= QUIC_VERSION_27) {
connection()->SendConnectionCloseWithDetails(
QUIC_TOO_MANY_OPEN_STREAMS, "Old style stream rejection");
@@ -691,12 +686,7 @@ ReliableQuicStream* QuicSession::GetOrCreateDynamicStream(
return nullptr;
}
- ReliableQuicStream* stream = CreateIncomingDynamicStream(stream_id);
- if (stream == nullptr) {
- return nullptr;
- }
- ActivateStream(stream);
- return stream;
+ return CreateIncomingDynamicStream(stream_id);
}
void QuicSession::set_max_open_incoming_streams(
« no previous file with comments | « net/quic/quic_session.h ('k') | net/quic/quic_session_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698