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

Unified Diff: net/nqe/network_quality_estimator.cc

Issue 2663963002: Add effective connection type to throughput mapping (Closed)
Patch Set: Created 3 years, 11 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_params.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 daceb853d09f31e5d54337fba7475001eb46b0a5..3a0aa4e73323c319e588e677ae93a0dded493222 100644
--- a/net/nqe/network_quality_estimator.cc
+++ b/net/nqe/network_quality_estimator.cc
@@ -1382,17 +1382,6 @@ bool NetworkQualityEstimator::ReadCachedNetworkQualityEstimate() {
const base::TimeTicks now = tick_clock_->NowTicks();
- if (effective_connection_type_ == EFFECTIVE_CONNECTION_TYPE_UNKNOWN) {
- // Read the effective connection type from the cached estimate.
- last_effective_connection_type_computation_ = now;
- network_quality_ = cached_network_quality.network_quality();
- effective_connection_type_ =
- cached_network_quality.effective_connection_type();
-
- if (effective_connection_type_ != EFFECTIVE_CONNECTION_TYPE_UNKNOWN)
- NotifyObserversOfEffectiveConnectionTypeChanged();
- }
-
if (cached_network_quality.network_quality().downstream_throughput_kbps() !=
nqe::internal::kInvalidThroughput) {
ThroughputObservation througphput_observation(
@@ -1422,6 +1411,7 @@ bool NetworkQualityEstimator::ReadCachedNetworkQualityEstimate() {
rtt_observations_.AddObservation(rtt_observation);
NotifyObserversOfRTT(rtt_observation);
}
+ ComputeEffectiveConnectionType();
return true;
}
« no previous file with comments | « no previous file | net/nqe/network_quality_estimator_params.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698