[Media Controls] Add UMA for timeline scrubber

tools/metrics/histograms/histograms.xml

Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index 72d58a4c0da5ee86f24467f740fa240fff893433..4062dc267cd680339f94e98db58ed373668edb26 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -27485,6 +27485,82 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
   </summary>
 </histogram>
 
+<histogram name="Media.Timeline.DragGestureDuration" units="ms">

[Ilya Sherman, 2017/04/01 00:42:41]

nit: Please add base="true" for this histogram and others that are just
providing name fragments.

[johnme, 2017/04/03 16:16:48]

Done.

+<!-- Name completed by histogram_suffixes name="MediaTimelineWidths" -->
+
+  <owner>johnme@chromium.org</owner>
+  <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 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 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 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 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 +102379,149 @@ from previous Chrome versions.
   <int value="2" label="Pending Media Tracks"/>
 </enum>
 
+<enum name="MediaTimelineAbsTimeDelta" type="int">
+  <int value="0" label="[0, 0]"/>

[Ilya Sherman, 2017/04/01 00:42:41]

nit: We tend to have min be inclusive and max be inclusive on the UMA
dashboards.  You might want to follow that same pattern here, and update the
bucket boundaries accordingly.  It's just marginally less likely to confuse
people who are familiar with the way that UMA tends to work.

[johnme, 2017/04/03 16:16:48]

Done (assuming you mean min be inclusive and max be exclusive). I kept the
separate zero buckets (since those are likely quite common for TimeDelta &
Percent), and for negative ranges I did the opposite (min exclusive, max
inclusive) for symmetry.

+  <int value="1" label="(0, 16ms]"/>
+  <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 +118862,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 +118920,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()."/>