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

Unified Diff: net/nqe/network_quality_estimator.cc

Issue 2340193003: NQE: Set RTT thresholds for effective connection type of 3G (Closed)
Patch Set: Created 4 years, 3 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 | « no previous file | net/nqe/network_quality_estimator_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/nqe/network_quality_estimator.cc
diff --git a/net/nqe/network_quality_estimator.cc b/net/nqe/network_quality_estimator.cc
index 359b437ebfcf8521d6b2b9f947834418337650eb..607b5db42d82b9e343b270fbf9f487c65f0f1917 100644
--- a/net/nqe/network_quality_estimator.cc
+++ b/net/nqe/network_quality_estimator.cc
@@ -537,6 +537,16 @@ void NetworkQualityEstimator::ObtainEffectiveConnectionTypeModelParams(
base::TimeDelta::FromMilliseconds(1280),
nqe::internal::kInvalidThroughput);
+ default_effective_connection_type_thresholds_[EFFECTIVE_CONNECTION_TYPE_3G] =
+ nqe::internal::NetworkQuality(
+ // Set to 273 milliseconds, which corresponds to 50th percentile of
+ // 3G HTTP RTT observations on Android.
+ base::TimeDelta::FromMilliseconds(273),
+ // Set to 204 milliseconds, which corresponds to 50th percentile of
+ // 3G transport RTT observations on Android.
+ base::TimeDelta::FromMilliseconds(204),
+ nqe::internal::kInvalidThroughput);
+
for (size_t i = 0; i < EFFECTIVE_CONNECTION_TYPE_LAST; ++i) {
EffectiveConnectionType effective_connection_type =
static_cast<EffectiveConnectionType>(i);
« no previous file with comments | « no previous file | net/nqe/network_quality_estimator_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698