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

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

Issue 2779273003: [Media Controls] Add UMA for timeline scrubber (Closed)
Patch Set: Address isherman's review comments Created 3 years, 9 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
Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index 72d58a4c0da5ee86f24467f740fa240fff893433..fbf348b48b0a9b69ac0117c1fc5956cfd8fa1023 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -27485,6 +27485,85 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</summary>
</histogram>
+<histogram base="true" name="Media.Timeline.DragGestureDuration" units="ms">
+<!-- Name completed by histogram_suffixes name="MediaTimelineWidths" -->
+
+ <owner>johnme@chromium.org</owner>
mlamouri (slow - plz ping) 2017/04/04 13:40:16 Actually, would you mind adding media-dev@chromium
johnme 2017/04/06 16:23:26 Done.
+ <summary>
+ The duration of the user gesture (how long their finger is down), recorded
+ at the end of each drag gesture. This histogram is split according to the
+ width of the slider track in CSS px - see histogram name suffix.
+ </summary>
+</histogram>
+
+<histogram base="true" name="Media.Timeline.DragPercent"
+ enum="MediaTimelinePercent">
+<!-- Name completed by histogram_suffixes name="MediaTimelineWidths" -->
+
+ <owner>johnme@chromium.org</owner>
+ <summary>
+ The net distance the media scrubber moved, recorded at the end of each drag
+ gesture, as a percentage of the width of the slider track. This histogram is
+ split according to the width of the slider track in CSS px - see histogram
+ name suffix.
+ </summary>
+</histogram>
+
+<histogram base="true" name="Media.Timeline.DragSumAbsTimeDelta"
+ enum="MediaTimelineAbsTimeDelta">
+<!-- Name completed by histogram_suffixes name="MediaTimelineWidths" -->
+
+ <owner>johnme@chromium.org</owner>
+ <summary>
+ The sum of absolute distances the media scrubber incrementally moved,
+ recorded at the end of each drag gesture, each measured as the change in
+ media current time. If the scrubber was dragged back and forth before being
+ released, this will be larger than the value recorded for
+ Media.Timeline.DragTimeDelta (and may even be larger than the media
+ duration), otherwise it will be the same as DragTimeDelta. This histogram is
+ split according to the width of the slider track in CSS px - see histogram
+ name suffix.
+ </summary>
+</histogram>
+
+<histogram base="true" name="Media.Timeline.DragTimeDelta"
+ enum="MediaTimelineTimeDelta">
+<!-- Name completed by histogram_suffixes name="MediaTimelineWidths" -->
+
+ <owner>johnme@chromium.org</owner>
+ <summary>
+ The net distance the media scrubber moved, recorded at the end of each drag
+ gesture, measured as the change in media current time. This histogram is
+ split according to the width of the slider track in CSS px - see histogram
+ name suffix.
+ </summary>
+</histogram>
+
+<histogram base="true" name="Media.Timeline.SeekType"
+ enum="MediaTimelineSeekType">
+<!-- Name completed by histogram_suffixes name="MediaTimelineWidths" -->
+
+ <owner>johnme@chromium.org</owner>
+ <summary>
+ The type of user gesture, recorded at the end of each input sequence. For
+ example holding down the right arrow key with the scrubber focused will only
+ be logged as a single event. This histogram is split according to the width
+ of the slider track in CSS px - see histogram name suffix.
+ </summary>
+</histogram>
+
+<histogram name="Media.Timeline.Width" units="CSS px">
+<!-- Name completed by histogram_suffixes name="MediaElementConfigurations" -->
+
+ <owner>johnme@chromium.org</owner>
+ <summary>
+ The width of the media timeline track in CSS pixels, recorded the first time
+ a media element with controls starts playing (strictly speaking, it's the
+ width in CSS pixels ignoring CSS transforms, multiplied by pageZoomFactor,
+ but deliberately ignoring pinch zoom's pageScaleFactor).
+ </summary>
+</histogram>
+
<histogram name="Media.TimeToPipelineStarted" units="ms">
<obsolete>
Removed from code 2014/6/18.
@@ -102303,6 +102382,149 @@ from previous Chrome versions.
<int value="2" label="Pending Media Tracks"/>
</enum>
+<enum name="MediaTimelineAbsTimeDelta" type="int">
+ <int value="0" label="[0, 0]"/>
+ <int value="1" label="(0, 16ms)"/>
Ilya Sherman 2017/04/03 21:39:26 nit: Is this accurately described as "[1, 16ms)"?
johnme 2017/04/06 16:23:26 Done, and changed [0, 0] to [0, 1ms) as well :)
+ <int value="2" label="[16ms, 32ms)"/>
+ <int value="3" label="[32ms, 64ms)"/>
+ <int value="4" label="[64ms, 128ms)"/>
+ <int value="5" label="[128ms, 256ms)"/>
+ <int value="6" label="[256ms, 512ms)"/>
+ <int value="7" label="[512ms, 1s)"/>
+ <int value="8" label="[1s, 2s)"/>
+ <int value="9" label="[2s, 4s)"/>
+ <int value="10" label="[4s, 8s)"/>
+ <int value="11" label="[8s, 15s)"/>
+ <int value="12" label="[15s, 30s)"/>
+ <int value="13" label="[30s, 1m)"/>
+ <int value="14" label="[1m, 2m)"/>
+ <int value="15" label="[2m, 4m)"/>
+ <int value="16" label="[4m, 8m)"/>
+ <int value="17" label="[8m, 15m)"/>
+ <int value="18" label="[15m, 30m)"/>
+ <int value="19" label="[30m, 1h)"/>
+ <int value="20" label="[1h, 2h)"/>
+ <int value="21" label="[2h, 4h)"/>
+ <int value="22" label="[4h, 8h)"/>
+ <int value="23" label="[8h, 16h)"/>
+ <int value="24" label="[16h, inf)"/>
+</enum>
+
+<enum name="MediaTimelinePercent" type="int">
+ <int value="0" label="[-100.0%, -90.0%]"/>
+ <int value="1" label="(-90.0%, -80.0%]"/>
+ <int value="2" label="(-80.0%, -70.0%]"/>
+ <int value="3" label="(-70.0%, -60.0%]"/>
+ <int value="4" label="(-60.0%, -50.0%]"/>
+ <int value="5" label="(-50.0%, -45.0%]"/>
+ <int value="6" label="(-45.0%, -40.0%]"/>
+ <int value="7" label="(-40.0%, -35.0%]"/>
+ <int value="8" label="(-35.0%, -30.0%]"/>
+ <int value="9" label="(-30.0%, -25.0%]"/>
+ <int value="10" label="(-25.0%, -20.0%]"/>
+ <int value="11" label="(-20.0%, -15.0%]"/>
+ <int value="12" label="(-15.0%, -10.0%]"/>
+ <int value="13" label="(-10.0%, -7.0%]"/>
+ <int value="14" label="(-7.0%, -5.0%]"/>
+ <int value="15" label="(-5.0%, -3.0%]"/>
+ <int value="16" label="(-3.0%, -2.0%]"/>
+ <int value="17" label="(-2.0%, -1.5%]"/>
+ <int value="18" label="(-1.5%, -1.0%]"/>
+ <int value="19" label="(-1.0%, -0.7%]"/>
+ <int value="20" label="(-0.7%, -0.5%]"/>
+ <int value="21" label="(-0.5%, -0.3%]"/>
+ <int value="22" label="(-0.3%, -0.2%]"/>
+ <int value="23" label="(-0.2%, -0.1%]"/>
+ <int value="24" label="(-0.1%, 0%)"/>
+ <int value="25" label="[0%, 0%]"/>
+ <int value="26" label="(0%, 0.1%)"/>
+ <int value="27" label="[0.1%, 0.2%)"/>
+ <int value="28" label="[0.2%, 0.3%)"/>
+ <int value="29" label="[0.3%, 0.5%)"/>
+ <int value="30" label="[0.5%, 0.7%)"/>
+ <int value="31" label="[0.7%, 1.0%)"/>
+ <int value="32" label="[1.0%, 1.5%)"/>
+ <int value="33" label="[1.5%, 2.0%)"/>
+ <int value="34" label="[2.0%, 3.0%)"/>
+ <int value="35" label="[3.0%, 5.0%)"/>
+ <int value="36" label="[5.0%, 7.0%)"/>
+ <int value="37" label="[7.0%, 10.0%)"/>
+ <int value="38" label="[10.0%, 15.0%)"/>
+ <int value="39" label="[15.0%, 20.0%)"/>
+ <int value="40" label="[20.0%, 25.0%)"/>
+ <int value="41" label="[25.0%, 30.0%)"/>
+ <int value="42" label="[30.0%, 35.0%)"/>
+ <int value="43" label="[35.0%, 40.0%)"/>
+ <int value="44" label="[40.0%, 45.0%)"/>
+ <int value="45" label="[45.0%, 50.0%)"/>
+ <int value="46" label="[50.0%, 60.0%)"/>
+ <int value="47" label="[60.0%, 70.0%)"/>
+ <int value="48" label="[70.0%, 80.0%)"/>
+ <int value="49" label="[80.0%, 90.0%)"/>
+ <int value="50" label="[90.0%, 100.0%]"/>
+</enum>
+
+<enum name="MediaTimelineSeekType" type="int">
+ <int value="0" label="Click"/>
+ <int value="1" label="Drag from current position"/>
+ <int value="2" label="Drag from elsewhere"/>
+ <int value="3" label="Keyboard arrow key"/>
+ <int value="4" label="Keyboard page up/down key"/>
+ <int value="5" label="Keyboard home/end key"/>
+</enum>
+
+<enum name="MediaTimelineTimeDelta" type="int">
+ <int value="0" label="(-inf, -16h]"/>
+ <int value="1" label="(-16h, -8h]"/>
+ <int value="2" label="(-8h, -4h]"/>
+ <int value="3" label="(-4h, -2h]"/>
+ <int value="4" label="(-2h, -1h]"/>
+ <int value="5" label="(-1h, -30m]"/>
+ <int value="6" label="(-30m, -15m]"/>
+ <int value="7" label="(-15m, -8m]"/>
+ <int value="8" label="(-8m, -4m]"/>
+ <int value="9" label="(-4m, -2m]"/>
+ <int value="10" label="(-2m, -1m]"/>
+ <int value="11" label="(-1m, -30s]"/>
+ <int value="12" label="(-30s, -15s]"/>
+ <int value="13" label="(-15s, -8s]"/>
+ <int value="14" label="(-8s, -4s]"/>
+ <int value="15" label="(-4s, -2s]"/>
+ <int value="16" label="(-2s, -1s]"/>
+ <int value="17" label="(-1s, -512ms]"/>
+ <int value="18" label="(-512ms, -256ms]"/>
+ <int value="19" label="(-256ms, -128ms]"/>
+ <int value="20" label="(-128ms, -64ms]"/>
+ <int value="21" label="(-64ms, -32ms]"/>
+ <int value="22" label="(-32ms, -16ms]"/>
+ <int value="23" label="(-16ms, 0)"/>
+ <int value="24" label="[0, 0]"/>
+ <int value="25" label="(0, 16ms)"/>
+ <int value="26" label="[16ms, 32ms)"/>
+ <int value="27" label="[32ms, 64ms)"/>
+ <int value="28" label="[64ms, 128ms)"/>
+ <int value="29" label="[128ms, 256ms)"/>
+ <int value="30" label="[256ms, 512ms)"/>
+ <int value="31" label="[512ms, 1s)"/>
+ <int value="32" label="[1s, 2s)"/>
+ <int value="33" label="[2s, 4s)"/>
+ <int value="34" label="[4s, 8s)"/>
+ <int value="35" label="[8s, 15s)"/>
+ <int value="36" label="[15s, 30s)"/>
+ <int value="37" label="[30s, 1m)"/>
+ <int value="38" label="[1m, 2m)"/>
+ <int value="39" label="[2m, 4m)"/>
+ <int value="40" label="[4m, 8m)"/>
+ <int value="41" label="[8m, 15m)"/>
+ <int value="42" label="[15m, 30m)"/>
+ <int value="43" label="[30m, 1h)"/>
+ <int value="44" label="[1h, 2h)"/>
+ <int value="45" label="[2h, 4h)"/>
+ <int value="46" label="[4h, 8h)"/>
+ <int value="47" label="[8h, 16h)"/>
+ <int value="48" label="[16h, inf)"/>
+</enum>
+
<enum name="MediaTypePredictionResult" type="int">
<int value="0" label="All correct"/>
<int value="1" label="All incorrect"/>
@@ -118643,6 +118865,14 @@ from previous Chrome versions.
<affected-histogram name="Media.AudioOutputController"/>
</histogram_suffixes>
+<histogram_suffixes name="MediaElementConfigurations" separator=".">
+ <suffix name="InlinePortrait"/>
+ <suffix name="InlineLandscape"/>
+ <suffix name="FullscreenPortrait"/>
+ <suffix name="FullscreenLandscape"/>
+ <affected-histogram name="Media.Timeline.Width"/>
+</histogram_suffixes>
+
<histogram_suffixes name="MediaPipelineStatusForStreams" separator=".">
<suffix name="AudioVideo.Other"
label="PipelineStatus for the codecs that dont have an explicit metric."/>
@@ -118693,6 +118923,20 @@ from previous Chrome versions.
<affected-histogram name="Media.PipelineStatus"/>
</histogram_suffixes>
+<histogram_suffixes name="MediaTimelineWidths" separator=".">
+ <suffix name="32_47"/>
+ <suffix name="48_79"/>
+ <suffix name="80_127"/>
+ <suffix name="128_255"/>
+ <suffix name="256_511"/>
+ <suffix name="512_inf"/>
+ <affected-histogram name="Media.Timeline.DragGestureDuration"/>
+ <affected-histogram name="Media.Timeline.DragPercent"/>
+ <affected-histogram name="Media.Timeline.DragSumAbsTimeDelta"/>
+ <affected-histogram name="Media.Timeline.DragTimeDelta"/>
+ <affected-histogram name="Media.Timeline.SeekType"/>
+</histogram_suffixes>
+
<histogram_suffixes name="MediaVideoCaptureManagerTime" separator=".">
<suffix name="StartDeviceTime"
label="Measures the time taken for DoStartDeviceOnDeviceThread()."/>

Powered by Google App Engine
This is Rietveld 408576698