diff options
| author | fischman@chromium.org <fischman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-02-23 21:22:45 +0000 |
|---|---|---|
| committer | fischman@chromium.org <fischman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-02-23 21:22:45 +0000 |
| commit | 521ff2402cf238c0d6b3ec156f35add7f431a368 (patch) | |
| tree | 29525ef978e1035e025582b5e976a11047ccd48a /media/base | |
| parent | b8fe4c57d9b2f796d05011e280a91012d46a0f2a (diff) | |
| download | chromium_src-521ff2402cf238c0d6b3ec156f35add7f431a368.zip chromium_src-521ff2402cf238c0d6b3ec156f35add7f431a368.tar.gz chromium_src-521ff2402cf238c0d6b3ec156f35add7f431a368.tar.bz2 | |
Don't histogram negative VideoCodecProfile values.
Review URL: http://codereview.chromium.org/9430065
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@123334 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'media/base')
| -rw-r--r-- | media/base/video_decoder_config.cc | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/media/base/video_decoder_config.cc b/media/base/video_decoder_config.cc index 4a26922..6b1cf14 100644 --- a/media/base/video_decoder_config.cc +++ b/media/base/video_decoder_config.cc @@ -79,8 +79,11 @@ void VideoDecoderConfig::Initialize(VideoCodec codec, if (record_stats) { UMA_HISTOGRAM_ENUMERATION("Media.VideoCodec", codec, kVideoCodecMax + 1); - UMA_HISTOGRAM_ENUMERATION("Media.VideoCodecProfile", profile, - VIDEO_CODEC_PROFILE_MAX + 1); + // Drop UNKNOWN because U_H_E() uses one bucket for all values less than 1. + if (profile >= 0) { + UMA_HISTOGRAM_ENUMERATION("Media.VideoCodecProfile", profile, + VIDEO_CODEC_PROFILE_MAX + 1); + } UMA_HISTOGRAM_COUNTS_10000("Media.VideoCodedWidth", coded_size.width()); UmaHistogramAspectRatio("Media.VideoCodedAspectRatio", coded_size); UMA_HISTOGRAM_COUNTS_10000("Media.VideoVisibleWidth", visible_rect.width()); |