Index: tools/metrics/histograms/histograms.xml |
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml |
index f8ce2d3bf745ff971181508ece164e0d9741b4de..19518e7ef35432a93b564bc6d030107f04c2fd33 100644 |
--- a/tools/metrics/histograms/histograms.xml |
+++ b/tools/metrics/histograms/histograms.xml |
@@ -28408,6 +28408,22 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries. |
<summary>Pixel format used in HTML5 video. Emitted on video load.</summary> |
</histogram> |
+<histogram name="Media.VideoHeight.Average" units="pixels"> |
Ilya Sherman
2017/04/17 20:10:14
nit: Please add the attribute base="true", since t
xhwang
2017/04/18 05:52:37
Done.
|
+ <owner>media-dev@chromium.org</owner> |
+ <summary> |
+ The height of the video frame in an HTML5 video. Reported whenever a new |
+ video frame size is available (e.g. during MSE config change). |
Ilya Sherman
2017/04/17 20:10:14
Hmm, the average is updated whenever the size chan
xhwang
2017/04/18 05:52:37
See previous discussion on this in the code review
Ilya Sherman
2017/04/18 05:58:54
Could you please help me to understand how you pla
xhwang
2017/04/18 06:39:50
The original intention was to use this as the indi
DaleCurtis
2017/04/18 17:46:22
we could use a base::flat_set to ensure we only re
Ilya Sherman
2017/04/19 00:35:10
Sampling seems like a reasonable solution to me.
|
+ </summary> |
+</histogram> |
+ |
+<histogram name="Media.VideoHeight.Initial" units="pixels"> |
Ilya Sherman
2017/04/17 20:10:14
nit: Please add the attribute base="true", since t
xhwang
2017/04/18 05:52:37
Done.
|
+ <owner>xhwang@chromium.org</owner> |
+ <summary> |
+ The height of the first video frame in an HTML5 video. Reported when the |
+ first video frame is available. |
+ </summary> |
+</histogram> |
+ |
<histogram name="Media.VideoPersistence.AttemptResult" |
enum="VideoPersistenceAttemptResult"> |
<owner>mlamouri@chromium.org</owner> |
@@ -120470,6 +120486,15 @@ from previous Chrome versions. |
<affected-histogram name="Media.VideoCaptureManager"/> |
</histogram_suffixes> |
+<histogram_suffixes name="MediaVideoCategories" separator="."> |
+ <suffix name="All" label="All media with a video track."/> |
+ <suffix name="EME" label="EME media with a video track."/> |
+ <suffix name="MSE" label="MSE media with a video track."/> |
+ <suffix name="SRC" label="SRC media with a video track."/> |
+ <affected-histogram name="Media.VideoHeight.Average"/> |
+ <affected-histogram name="Media.VideoHeight.Initial"/> |
Ilya Sherman
2017/04/17 20:10:14
You are currently adding almost 1000 buckets-worth
xhwang
2017/04/18 05:52:36
When I said ~50 buckets, it's for UMA_HISTOGRAM_CU
Ilya Sherman
2017/04/18 05:58:54
It makes a fairly significant difference, yes. Wh
xhwang
2017/04/18 06:39:50
Because without seeing the real world data, it's h
Ilya Sherman
2017/04/19 00:32:50
Thanks. To be clear, if 120 buckets gives signifi
|
+</histogram_suffixes> |
+ |
<histogram_suffixes name="MediaWatchTimeCategories" separator="."> |
<suffix name="Audio.All" |
label="Watch time for all media with only an audio track."/> |