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

Unified Diff: net/http/http_stream_factory_impl_job.cc

Issue 675783005: QUIC - Always pesist 200 MRU alternate protocols to preferences file. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Persist only 200 MRU AlternateProtocolHostPortPairs Created 6 years, 2 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/http/http_stream_factory_impl_job.cc
diff --git a/net/http/http_stream_factory_impl_job.cc b/net/http/http_stream_factory_impl_job.cc
index 45d152a107945d57b624b3199a8a1baaaf6ce851..a11dd2e99724b1d24a168d2919515cb63876b5f9 100644
--- a/net/http/http_stream_factory_impl_job.cc
+++ b/net/http/http_stream_factory_impl_job.cc
@@ -1430,28 +1430,17 @@ bool HttpStreamFactoryImpl::Job::IsOrphaned() const {
}
void HttpStreamFactoryImpl::Job::ReportJobSuccededForRequest() {
- net::AlternateProtocolExperiment alternate_protocol_experiment =
- ALTERNATE_PROTOCOL_NOT_PART_OF_EXPERIMENT;
- base::WeakPtr<HttpServerProperties> http_server_properties =
- session_->http_server_properties();
- if (http_server_properties) {
- alternate_protocol_experiment =
- http_server_properties->GetAlternateProtocolExperiment();
- }
if (using_existing_quic_session_) {
// If an existing session was used, then no TCP connection was
// started.
- HistogramAlternateProtocolUsage(ALTERNATE_PROTOCOL_USAGE_NO_RACE,
- alternate_protocol_experiment);
+ HistogramAlternateProtocolUsage(ALTERNATE_PROTOCOL_USAGE_NO_RACE);
} else if (original_url_) {
// This job was the alternate protocol job, and hence won the race.
- HistogramAlternateProtocolUsage(ALTERNATE_PROTOCOL_USAGE_WON_RACE,
- alternate_protocol_experiment);
+ HistogramAlternateProtocolUsage(ALTERNATE_PROTOCOL_USAGE_WON_RACE);
} else {
// This job was the normal job, and hence the alternate protocol job lost
// the race.
- HistogramAlternateProtocolUsage(ALTERNATE_PROTOCOL_USAGE_LOST_RACE,
- alternate_protocol_experiment);
+ HistogramAlternateProtocolUsage(ALTERNATE_PROTOCOL_USAGE_LOST_RACE);
}
}

Powered by Google App Engine
This is Rietveld 408576698