Chromium Code Reviews| Index: tools/metrics/histograms/histograms.xml | 
| diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml | 
| index c7a6b843898f0197b5982f07a03c6fc62acfc264..574d69415a43932f8964ef259d230249d38e7ac6 100644 | 
| --- a/tools/metrics/histograms/histograms.xml | 
| +++ b/tools/metrics/histograms/histograms.xml | 
| @@ -17677,6 +17677,30 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries. | 
| </summary> | 
| </histogram> | 
| +<histogram name="Media.MSE.Duration" units="seconds"> | 
| + <owner>qinmin@chromium.org</owner> | 
| + <owner>timav@chromium.org</owner> | 
| + <summary>Duration of an uninterrupted MSE playback in seconds.</summary> | 
| 
 
xhwang
2015/09/30 21:02:57
+wolenetz as Media.MSE.Duration could be a general
 
Tima Vaisburd
2015/10/01 20:05:15
Added more explanation. This is the interval betwe
 
 | 
| +</histogram> | 
| + | 
| +<histogram name="Media.MSE.LateAudioFrames"> | 
| 
 
qinmin
2015/09/29 23:47:44
I think LateAudioFramesPerMillion is better
 
Ilya Sherman
2015/09/30 04:20:03
nit; You might also add units="frames per million"
 
Tima Vaisburd
2015/09/30 18:50:10
Added units but did not changes the histogram name
 
 | 
| + <owner>qinmin@chromium.org</owner> | 
| + <owner>timav@chromium.org</owner> | 
| + <summary> | 
| + Relative number of late audio frames wrt total number of audio frames in MSE | 
| 
 
xhwang
2015/09/30 21:02:57
Could you please elaborate what is exactly a "late
 
Tima Vaisburd
2015/10/01 20:05:15
Added an explanation, this is related to potential
 
 | 
| + playback, multiplied by one million. | 
| + </summary> | 
| +</histogram> | 
| + | 
| +<histogram name="Media.MSE.LateVideoFrames"> | 
| 
 
qinmin
2015/09/29 23:47:44
ditto
 
 | 
| + <owner>qinmin@chromium.org</owner> | 
| + <owner>timav@chromium.org</owner> | 
| + <summary> | 
| + Relative number of late video frames wrt total number of video frames in MSE | 
| 
 
xhwang
2015/09/30 21:02:57
ditto
 
Tima Vaisburd
2015/10/01 20:05:15
Added an explanation here as well.
 
 | 
| + playback, multiplied by one million. | 
| + </summary> | 
| +</histogram> | 
| + | 
| <histogram name="Media.MSE.NumberOfTracks"> | 
| <owner>acolwell@chromium.org</owner> | 
| <summary> | 
| @@ -17696,6 +17720,15 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries. | 
| </summary> | 
| </histogram> | 
| +<histogram name="Media.MSE.Starvations"> | 
| + <owner>qinmin@chromium.org</owner> | 
| + <owner>timav@chromium.org</owner> | 
| + <summary> | 
| + Relative number of starvations wrt total number of frames in MSE playback, | 
| 
 
xhwang
2015/09/30 21:02:57
ditto, define "starvation".
 
Tima Vaisburd
2015/10/01 20:05:15
Added here too.
 
 | 
| + multiplied by one million. | 
| + </summary> | 
| +</histogram> | 
| + | 
| <histogram name="Media.MSE.VideoCodec" enum="MSECodec"> | 
| <owner>acolwell@chromium.org</owner> | 
| <summary> |