Chromium Code Reviews Chromium Code Reviews
Issue 452643002:
Add UMA reporting to CdmPromise. (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@master| Index: tools/metrics/histograms/histograms.xml |
| diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml |
| index 48c4b5647bffe386679dcc80f81e46d79291c622..2cf9d251927b197238223d83b09deb9cca6dfbda 100644 |
| --- a/tools/metrics/histograms/histograms.xml |
| +++ b/tools/metrics/histograms/histograms.xml |
| @@ -10366,6 +10366,11 @@ Therefore, the affected-histogram name has to have at least one dot in it. |
| <summary>KeyError event count using the Clear Key key system.</summary> |
| </histogram> |
| +<histogram name="Media.EME.ClearKey.NewSession" enum="CdmPromiseResult"> |
|
ddorwin
2014/08/08 03:35:40
Update the name and comments.
sandersd (OOO until July 31)
2014/08/08 17:22:24
Done.
|
| + <owner>sandersd@chromium.org</owner> |
| + <summary>NewSession result using the Clear Key key system.</summary> |
| +</histogram> |
| + |
| <histogram name="Media.EME.NeedKey"> |
| <owner>xhwang@chromium.org</owner> |
| <summary>EME NeedKey event count.</summary> |
| @@ -10409,6 +10414,11 @@ Therefore, the affected-histogram name has to have at least one dot in it. |
| <summary>KeyError event count using an unknown key system.</summary> |
| </histogram> |
| +<histogram name="Media.EME.Unknown.NewSession" enum="CdmPromiseResult"> |
| + <owner>sandersd@chromium.org</owner> |
| + <summary>NewSession result using an unknown key system.</summary> |
| +</histogram> |
| + |
| <histogram name="Media.EME.Widevine.addKey" enum="MediaKeyException"> |
| <owner>xhwang@chromium.org</owner> |
| <summary>addKey result using the Widevine key system.</summary> |
| @@ -10440,6 +10450,11 @@ Therefore, the affected-histogram name has to have at least one dot in it. |
| <summary>KeyError event count using the Widevine key system.</summary> |
| </histogram> |
| +<histogram name="Media.EME.Widevine.NewSession" enum="CdmPromiseResult"> |
| + <owner>sandersd@chromium.org</owner> |
| + <summary>NewSession result using the Widevine key system.</summary> |
| +</histogram> |
| + |
| <histogram name="Media.FallbackHardwareAudioBitsPerChannel"> |
| <owner>Please list the metric's owners. Add more owner tags as needed.</owner> |
| <summary> |
| @@ -36712,6 +36727,17 @@ Therefore, the affected-histogram name has to have at least one dot in it. |
| <int value="20" label="FutureCat (>10.10), 8-bit (?)"/> |
| </enum> |
| +<enum name="CdmPromiseResult" type="int"> |
| + <int value="0" label="Success"/> |
| + <int value="1" label="Not Supported Error"/> |
|
ddorwin
2014/08/08 03:35:41
These are really exception names, so we could remo
sandersd (OOO until July 31)
2014/08/08 17:22:24
Done.
|
| + <int value="2" label="Invalid State Error"/> |
| + <int value="3" label="Invalid Access Error"/> |
| + <int value="4" label="Quota Exceeded Error"/> |
| + <int value="5" label="Unknown Error"/> |
| + <int value="6" label="Client Error"/> |
| + <int value="7" label="Output Error"/> |
| +</enum> |
| + |
| <enum name="CertificateChainPosition" type="int"> |
| <int value="0" label="Root Certificate"/> |
| </enum> |
