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

Unified Diff: net/spdy/spdy_session.cc

Issue 2165023002: Remove enable_priority_dependencies. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
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
« no previous file with comments | « net/spdy/spdy_session.h ('k') | net/spdy/spdy_session_pool.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/spdy/spdy_session.cc
diff --git a/net/spdy/spdy_session.cc b/net/spdy/spdy_session.cc
index c1d49ffd0151adb2d742d5ba8338515a4937ad2c..49338ac571b1d1134c3ff8fe1c61e5492ffb603c 100644
--- a/net/spdy/spdy_session.cc
+++ b/net/spdy/spdy_session.cc
@@ -627,7 +627,6 @@ SpdySession::SpdySession(const SpdySessionKey& spdy_session_key,
bool verify_domain_authentication,
bool enable_sending_initial_data,
bool enable_ping_based_connection_checking,
- bool enable_priority_dependencies,
size_t session_max_recv_window_size,
size_t stream_max_recv_window_size,
TimeFunc time_func,
@@ -683,7 +682,6 @@ SpdySession::SpdySession(const SpdySessionKey& spdy_session_key,
hung_interval_(base::TimeDelta::FromSeconds(kHungIntervalSeconds)),
proxy_delegate_(proxy_delegate),
time_func_(time_func),
- priority_dependencies_enabled_(enable_priority_dependencies),
weak_factory_(this) {
net_log_.BeginEvent(
NetLog::TYPE_HTTP2_SESSION,
@@ -1018,10 +1016,8 @@ std::unique_ptr<SpdySerializedFrame> SpdySession::CreateSynStream(
SpdyStreamId dependent_stream_id = 0;
bool exclusive = false;
- if (priority_dependencies_enabled_) {
- priority_dependency_state_.OnStreamSynSent(
- stream_id, spdy_priority, &dependent_stream_id, &exclusive);
- }
+ priority_dependency_state_.OnStreamSynSent(stream_id, spdy_priority,
+ &dependent_stream_id, &exclusive);
if (net_log().IsCapturing()) {
net_log().AddEvent(
@@ -1213,8 +1209,7 @@ void SpdySession::CloseActiveStreamIterator(ActiveStreamMap::iterator it,
std::unique_ptr<SpdyStream> owned_stream(it->second.stream);
active_streams_.erase(it);
- if (priority_dependencies_enabled_)
- priority_dependency_state_.OnStreamDestruction(owned_stream->stream_id());
+ priority_dependency_state_.OnStreamDestruction(owned_stream->stream_id());
// TODO(akalin): When SpdyStream was ref-counted (and
// |unclaimed_pushed_streams_| held scoped_refptr<SpdyStream>), this
« no previous file with comments | « net/spdy/spdy_session.h ('k') | net/spdy/spdy_session_pool.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698