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

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

Issue 2076353002: Record NQE ECT at main frame (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebased Created 4 years, 6 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/nqe/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 baddd7b6aa073d4f15726031a4e505ca6e7cf049..39e624c9e6da132ae524c8bdddf89f344ee2a664 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -35055,6 +35055,18 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</summary>
</histogram>
+<histogram name="NQE.MainFrame.EffectiveConnectionType"
+ enum="NQEEffectiveConnectionType">
+ <owner>tbansal@chromium.org</owner>
+ <owner>bengr@chromium.org</owner>
+ <summary>
+ Effective connection type estimated by network quality estimator. Suffixed
+ with the connection type reported by the operating system.
+
+ This metric is recorded on main-frame requests.
+ </summary>
+</histogram>
+
<histogram name="NQE.MainFrame.Kbps" units="Kbps">
<owner>bengr@chromium.org</owner>
<owner>tbansal@chromium.org</owner>
@@ -83075,6 +83087,16 @@ To add a new entry, add it with any value and run test to compute valid value.
<int value="2" label="Enabled">NPAPI is enabled</int>
</enum>
+<enum name="NQEEffectiveConnectionType" type="int">
+ <int value="0" label="Unknown"/>
+ <int value="1" label="Offline"/>
+ <int value="2" label="Slow 2G"/>
+ <int value="3" label="2G"/>
+ <int value="4" label="3G"/>
+ <int value="5" label="4G"/>
+ <int value="6" label="Broadband"/>
+</enum>
+
<enum name="NQEExternalEstimateProviderStatus" type="int">
<int value="0" label="External estimate provider was not available"/>
<int value="1" label="External estimate provider was available"/>
@@ -96630,6 +96652,7 @@ To add a new entry, add it with any value and run test to compute valid value.
<suffix name="None" label="With no detected network"/>
<suffix name="Bluetooth" label="On Bluetooth network"/>
<affected-histogram name="NQE.FastestRTT"/>
+ <affected-histogram name="NQE.MainFrame.EffectiveConnectionType"/>
<affected-histogram name="NQE.MainFrame.Kbps.Percentile50"/>
<affected-histogram name="NQE.MainFrame.RTT.Percentile50"/>
<affected-histogram name="NQE.MainFrame.TransportRTT.Percentile50"/>
« no previous file with comments | « net/nqe/network_quality_estimator_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698