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

Unified Diff: net/base/network_quality_estimator.cc

Issue 1897903002: Record how frequently EEP provides valid estimates (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 8 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/base/network_quality_estimator.cc
diff --git a/net/base/network_quality_estimator.cc b/net/base/network_quality_estimator.cc
index 9a6d8d60f0660841119ea7af663753b3f42fddc7..852c81ec89f322e6132c5abfb861d960ad9e9ac3 100644
--- a/net/base/network_quality_estimator.cc
+++ b/net/base/network_quality_estimator.cc
@@ -945,6 +945,8 @@ void NetworkQualityEstimator::QueryExternalEstimateProvider() {
EXTERNAL_ESTIMATE_PROVIDER_STATUS_QUERY_SUCCESSFUL);
base::TimeDelta rtt;
if (external_estimate_provider_->GetRTT(&rtt)) {
+ RecordExternalEstimateProviderMetrics(
+ EXTERNAL_ESTIMATE_PROVIDER_STATUS_RTT_AVAILABLE);
rtt_observations_.AddObservation(
RttObservation(rtt, base::TimeTicks::Now(), EXTERNAL_ESTIMATE));
}
@@ -952,6 +954,8 @@ void NetworkQualityEstimator::QueryExternalEstimateProvider() {
int32_t downstream_throughput_kbps;
if (external_estimate_provider_->GetDownstreamThroughputKbps(
&downstream_throughput_kbps)) {
+ RecordExternalEstimateProviderMetrics(
+ EXTERNAL_ESTIMATE_PROVIDER_STATUS_DOWNLINK_BANDWIDTH_AVAILABLE);
downstream_throughput_kbps_observations_.AddObservation(
ThroughputObservation(downstream_throughput_kbps,
base::TimeTicks::Now(), EXTERNAL_ESTIMATE));

Powered by Google App Engine
This is Rietveld 408576698