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

Unified Diff: media/blink/webmediaplayer_impl.cc

Issue 2885303002: media: Add Media.UnderflowDuration.EME UMA (Closed)
Patch Set: Created 3 years, 7 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:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/blink/webmediaplayer_impl.cc
diff --git a/media/blink/webmediaplayer_impl.cc b/media/blink/webmediaplayer_impl.cc
index 2b8427ab3bc28b4b39d61611b536c33f78c06821..98db8f7b033b9f1240edac01a250b97d540f60fa 100644
--- a/media/blink/webmediaplayer_impl.cc
+++ b/media/blink/webmediaplayer_impl.cc
@@ -2390,10 +2390,14 @@ void WebMediaPlayerImpl::SwitchRenderer(bool is_rendered_remotely) {
void WebMediaPlayerImpl::RecordUnderflowDuration(base::TimeDelta duration) {
DCHECK(data_source_ || chunk_demuxer_);
+
if (data_source_)
UMA_HISTOGRAM_TIMES("Media.UnderflowDuration", duration);
else
UMA_HISTOGRAM_TIMES("Media.UnderflowDuration.MSE", duration);
+
+ if (is_encrypted_)
+ UMA_HISTOGRAM_TIMES("Media.UnderflowDuration.EME", duration);
}
#define UMA_HISTOGRAM_VIDEO_HEIGHT(name, sample) \
« no previous file with comments | « no previous file | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698