Chromium Code Reviews Chromium Code Reviews
Issue 1700743003:
Record whether and why the media default controls are being shown. (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@media-controls-usecount| Index: tools/metrics/histograms/histograms.xml |
| diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml |
| index 23b9020f207406be4fc41779d9ab357f4ff73a61..4ef428e9dbbc7725412c6e060b6fe72c5f9e5dc3 100644 |
| --- a/tools/metrics/histograms/histograms.xml |
| +++ b/tools/metrics/histograms/histograms.xml |
| @@ -19109,6 +19109,11 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries. |
| </summary> |
| </histogram> |
| +<histogram name="Media.Controls.Show" enum="MediaDefaultControlsReason"> |
| + <owner>mlamouri@chromium.org</owner> |
| + <summary>Whether default media controls were shown and why.</summary> |
|
Alexei Svitkine (slow)
2016/02/16 22:22:09
Mention when this is logged - once for each video?
mlamouri (slow - plz ping)
2016/02/18 15:49:31
Done.
|
| +</histogram> |
| + |
| <histogram name="Media.CrosBeamformingDeviceState" |
| enum="CrosBeamformingDeviceState"> |
| <owner>ajm@chromium.org</owner> |
| @@ -72173,6 +72178,13 @@ To add a new entry, add it with any value and run test to compute valid value. |
| <int value="39" label="SmoothStream"/> |
| </enum> |
| +<enum name="MediaDefaultControlsReason" type="int"> |
| + <int value="0" label="Default controls from attribute"/> |
| + <int value="1" label="Default controls because fullscreen"/> |
| + <int value="2" label="Default controls because no scripts allowed"/> |
| + <int value="3" label="No default controls"/> |
| +</enum> |
| + |
| <enum name="MediaElementAutoPlay" type="int"> |
| <int value="0" label="Media element with autoplay seen"/> |
| <int value="1" |
