Chromium Code Reviews Chromium Code Reviews
Issue 2933213003:
Handle zero timestamp in RTCVideoEncoder timestamp matching & add UMA (Closed)
| Index: tools/metrics/histograms/histograms.xml |
| diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml |
| index d49de6ca46708540be8ead92b5e64157f34035de..18700100b69e0bfdfae62d58a82484c9f6014845 100644 |
| --- a/tools/metrics/histograms/histograms.xml |
| +++ b/tools/metrics/histograms/histograms.xml |
| @@ -29552,6 +29552,15 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries. |
| <summary>Video codec profile used in RTC video encoder.</summary> |
| </histogram> |
| +<histogram name="Media.RTCVideoEncoderTimestampMatchSuccess" |
| + enum="BooleanSuccess"> |
| + <owner>emircan@chromium.org</owner> |
| + <summary> |
| + Indicates whether we were successful in preserving timestamps in hardware |
| + video encoder session. |
|
Alexei Svitkine (slow)
2017/06/13 15:15:36
Expand description to mention when it's logged (lo
emircan
2017/06/13 18:00:28
Done.
|
| + </summary> |
| +</histogram> |
| + |
| <histogram name="Media.Session.ActiveTime" units="ms"> |
| <owner>avayvod@chromium.org</owner> |
| <owner>mlamouri@chromium.org</owner> |
| @@ -76339,8 +76348,8 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries. |
| <owner>treib@chromium.org</owner> |
| <summary> |
| The time for the new tab page to fire the "load" event. Note: This |
| - is usually recorded with a suffix (.Local/Google/Other). The base version |
| - is recorded only on Android. Recorded only once per tab, i.e. excluding |
| + is usually recorded with a suffix (.Local/Google/Other). The base version is |
| + recorded only on Android. Recorded only once per tab, i.e. excluding |
| back/forward navigations. |
| </summary> |
| </histogram> |
