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

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

Issue 2707973005: Remove Media.UnderflowCount, add Media.UnderflowDuration for MSE. (Closed)
Patch Set: Created 3 years, 10 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
« no previous file with comments | « media/blink/webmediaplayer_impl.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index 86869a435ce1fb5ee9a688b389ca56f5a3779b27..e94c81ce2a7307b92a9cfa85a89d150028123adc 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -26473,6 +26473,9 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</histogram>
<histogram name="Media.UnderflowCount">
+ <obsolete>
+ Removed Feb 2017. Media.UnderflowDuration provides more useful counts.
+ </obsolete>
<owner>dalecurtis@chromium.org</owner>
<summary>
The number of times a src= playback has underflowed; i.e. ran out of data.
@@ -26482,7 +26485,16 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
<histogram name="Media.UnderflowDuration" units="ms">
<owner>dalecurtis@chromium.org</owner>
<summary>
- The amount of time taken to leave the underflow state; i.e. resume playback.
+ The amount of time taken to leave the underflow state (i.e. resume playback)
+ for src= playbacks.
+ </summary>
+</histogram>
+
+<histogram name="Media.UnderflowDuration.MSE" units="ms">
+ <owner>dalecurtis@chromium.org</owner>
+ <summary>
+ The amount of time taken to leave the underflow state (i.e. resume playback)
+ for Media Source Extensions (MSE) based playbacks.
</summary>
</histogram>
« no previous file with comments | « media/blink/webmediaplayer_impl.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698