Chromium Code Reviews| Index: media/blink/watch_time_reporter.h |
| diff --git a/media/blink/watch_time_reporter.h b/media/blink/watch_time_reporter.h |
| index 61ae1dc2c99bec283800fe12dc258251eb66e12e..d72e4cfd37be6a84672f50f27277c3cce698b5ba 100644 |
| --- a/media/blink/watch_time_reporter.h |
| +++ b/media/blink/watch_time_reporter.h |
| @@ -17,20 +17,18 @@ |
| namespace media { |
| // Class for monitoring and reporting watch time in response to various state |
| -// changes during the playback of media. At present we are only recording the |
| -// watch time for audio+video playbacks. |
| -// TODO(dalecurtis): We want to introduce a similar "listening time" metric in |
| -// the near future to track audio only cases. |
| +// changes during the playback of media. We record metrics for audio only |
| +// playbacks as well as audio+video playbacks of sufficient size. |
| // |
| // Watch time for our purposes is defined as the amount of elapsed media time |
| -// for audio+video media. A minimum of 7 seconds of unmuted, foreground media |
| -// must be watched to start watch time monitoring. Watch time is checked every |
| -// 5 seconds from then on and reported to multiple buckets: All, MSE, SRC, EME, |
| -// AC, and battery. |
| +// for audio only or audio+video media. A minimum of 7 seconds of unmuted, |
| +// foreground (video only) media must be watched to start watch time monitoring. |
|
sandersd (OOO until July 31)
2016/11/17 21:10:37
Nit: Perhaps say 'when there is video' instead of
DaleCurtis
2016/11/17 22:17:34
Done.
|
| +// Watch time is checked every 5 seconds from then on and reported to multiple |
| +// buckets: All, MSE, SRC, EME, AC, and battery. |
| // |
| -// Any one of paused, hidden, or muted is sufficient to stop watch time metric |
| -// reports. Each of these has a hysteresis where if the state change is undone |
| -// within 5 seconds, the watch time will be counted as uninterrupted. |
| +// Any one of paused, hidden (video only), or muted is sufficient to stop watch |
| +// time metric reports. Each of these has a hysteresis where if the state change |
| +// is undone within 5 seconds, the watch time will be counted as uninterrupted. |
| // |
| // Power events (on/off battery power) have a similar hysteresis, but unlike |
| // the aforementioned properties, will not stop metric collection. |