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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: tools/metrics/histograms/histograms.xml

Side-by-side diff isn't available for this file because of its large size.
Issue 1897903002: Record how frequently EEP provides valid estimates (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed bengr comments, rebased Created 4 years, 7 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:
Download patch
« no previous file with comments | « net/base/network_quality_estimator_unittest.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index 34e02a3ff0a02d44f7eb9fdce70700eb3fbfc5fb..14602c4a32514e3e3d77d646abd402732700d681 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -33120,6 +33120,27 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</summary>
</histogram>
+<histogram name="NQE.ExternalEstimateProvider.DownlinkBandwidth" units="Kbps">
+ <owner>bengr@chromium.org</owner>
+ <owner>tbansal@chromium.org</owner>
+ <summary>
+ Records the estimate of the downlink bandwidth (in Kbps) provided by the
+ external estimate provided. Recorded every time the external estimate
+ provider provides a valid downlink bandwidth estimate to the network quality
+ estimator.
+ </summary>
+</histogram>
+
+<histogram name="NQE.ExternalEstimateProvider.RTT" units="ms">
+ <owner>bengr@chromium.org</owner>
+ <owner>tbansal@chromium.org</owner>
+ <summary>
+ Records the estimate of the RTT provided by the external estimate provided.
+ Recorded every time the external estimate provider provides a valid RTT
Alexei Svitkine (slow) 2016/04/26 21:27:05 Nit: Remove extra space.
tbansal1 2016/04/26 21:29:25 Done.
+ estimate to the network quality estimator.
+ </summary>
+</histogram>
+
<histogram name="NQE.ExternalEstimateProviderStatus"
enum="NQEExternalEstimateProviderStatus">
<owner>bengr@chromium.org</owner>
@@ -78717,6 +78738,10 @@ To add a new entry, add it with any value and run test to compute valid value.
<int value="2" label="External estimate provider was queried"/>
<int value="3" label="Query to external estimate provider was successful"/>
<int value="4" label="Callback received from external estimate provider"/>
+ <int value="5" label="RTT was available from the external estimate provider"/>
+ <int value="6"
+ label="Downlink bandwidth was available from the external estimate
+ provider"/>
</enum>
<enum name="NtpFollowAction" type="int">
« no previous file with comments | « net/base/network_quality_estimator_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698