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

Side by Side Diff: media/base/video_decoder_config.cc

Issue 148553003: Clean up histogram'd media enum max values. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: address comments Created 6 years, 10 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "media/base/video_decoder_config.h" 5 #include "media/base/video_decoder_config.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 #include "base/metrics/histogram.h" 8 #include "base/metrics/histogram.h"
9 9
10 namespace media { 10 namespace media {
(...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after
70 // Drop UNKNOWN because U_H_E() uses one bucket for all values less than 1. 70 // Drop UNKNOWN because U_H_E() uses one bucket for all values less than 1.
71 if (profile >= 0) { 71 if (profile >= 0) {
72 UMA_HISTOGRAM_ENUMERATION("Media.VideoCodecProfile", profile, 72 UMA_HISTOGRAM_ENUMERATION("Media.VideoCodecProfile", profile,
73 VIDEO_CODEC_PROFILE_MAX + 1); 73 VIDEO_CODEC_PROFILE_MAX + 1);
74 } 74 }
75 UMA_HISTOGRAM_COUNTS_10000("Media.VideoCodedWidth", coded_size.width()); 75 UMA_HISTOGRAM_COUNTS_10000("Media.VideoCodedWidth", coded_size.width());
76 UmaHistogramAspectRatio("Media.VideoCodedAspectRatio", coded_size); 76 UmaHistogramAspectRatio("Media.VideoCodedAspectRatio", coded_size);
77 UMA_HISTOGRAM_COUNTS_10000("Media.VideoVisibleWidth", visible_rect.width()); 77 UMA_HISTOGRAM_COUNTS_10000("Media.VideoVisibleWidth", visible_rect.width());
78 UmaHistogramAspectRatio("Media.VideoVisibleAspectRatio", visible_rect); 78 UmaHistogramAspectRatio("Media.VideoVisibleAspectRatio", visible_rect);
79 UMA_HISTOGRAM_ENUMERATION( 79 UMA_HISTOGRAM_ENUMERATION(
80 "Media.VideoPixelFormat", format, VideoFrame::HISTOGRAM_MAX); 80 "Media.VideoPixelFormat", format, VideoFrame::FORMAT_MAX + 1);
81 } 81 }
82 82
83 codec_ = codec; 83 codec_ = codec;
84 profile_ = profile; 84 profile_ = profile;
85 format_ = format; 85 format_ = format;
86 coded_size_ = coded_size; 86 coded_size_ = coded_size;
87 visible_rect_ = visible_rect; 87 visible_rect_ = visible_rect;
88 natural_size_ = natural_size; 88 natural_size_ = natural_size;
89 extra_data_.assign(extra_data, extra_data + extra_data_size); 89 extra_data_.assign(extra_data, extra_data + extra_data_size);
90 is_encrypted_ = is_encrypted; 90 is_encrypted_ = is_encrypted;
(...skipping 70 matching lines...) Expand 10 before | Expand all | Expand 10 after
161 161
162 size_t VideoDecoderConfig::extra_data_size() const { 162 size_t VideoDecoderConfig::extra_data_size() const {
163 return extra_data_.size(); 163 return extra_data_.size();
164 } 164 }
165 165
166 bool VideoDecoderConfig::is_encrypted() const { 166 bool VideoDecoderConfig::is_encrypted() const {
167 return is_encrypted_; 167 return is_encrypted_;
168 } 168 }
169 169
170 } // namespace media 170 } // namespace media
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698