Chromium Code Reviews| Index: media/audio/audio_output_resampler.cc |
| diff --git a/media/audio/audio_output_resampler.cc b/media/audio/audio_output_resampler.cc |
| index e6fcc56560484308bbb495984a63f8e3309a6985..648cf1317f33543333071a0bd8517336a6111f83 100644 |
| --- a/media/audio/audio_output_resampler.cc |
| +++ b/media/audio/audio_output_resampler.cc |
| @@ -77,7 +77,7 @@ static void RecordStats(const AudioParameters& output_params) { |
| limits::kMaxBitsPerSample); |
| UMA_HISTOGRAM_ENUMERATION( |
| "Media.HardwareAudioChannelLayout", output_params.channel_layout(), |
| - CHANNEL_LAYOUT_MAX); |
| + CHANNEL_LAYOUT_MAX + 1); |
| UMA_HISTOGRAM_ENUMERATION( |
|
rileya (GONE FROM CHROMIUM)
2014/01/29 21:12:20
Same here. More in the Fallback function below.
|
| "Media.HardwareAudioChannelCount", output_params.channels(), |
| limits::kMaxChannels); |
| @@ -85,7 +85,7 @@ static void RecordStats(const AudioParameters& output_params) { |
| AudioSampleRate asr = media::AsAudioSampleRate(output_params.sample_rate()); |
| if (asr != kUnexpectedAudioSampleRate) { |
| UMA_HISTOGRAM_ENUMERATION( |
| - "Media.HardwareAudioSamplesPerSecond", asr, kUnexpectedAudioSampleRate); |
| + "Media.HardwareAudioSamplesPerSecond", asr, kAudioSampleRateMax + 1); |
| } else { |
| UMA_HISTOGRAM_COUNTS( |
| "Media.HardwareAudioSamplesPerSecondUnexpected", |
| @@ -101,7 +101,7 @@ static void RecordFallbackStats(const AudioParameters& output_params) { |
| output_params.bits_per_sample(), limits::kMaxBitsPerSample); |
| UMA_HISTOGRAM_ENUMERATION( |
| "Media.FallbackHardwareAudioChannelLayout", |
| - output_params.channel_layout(), CHANNEL_LAYOUT_MAX); |
| + output_params.channel_layout(), CHANNEL_LAYOUT_MAX + 1); |
| UMA_HISTOGRAM_ENUMERATION( |
| "Media.FallbackHardwareAudioChannelCount", |
| output_params.channels(), limits::kMaxChannels); |
| @@ -110,7 +110,7 @@ static void RecordFallbackStats(const AudioParameters& output_params) { |
| if (asr != kUnexpectedAudioSampleRate) { |
| UMA_HISTOGRAM_ENUMERATION( |
| "Media.FallbackHardwareAudioSamplesPerSecond", |
| - asr, kUnexpectedAudioSampleRate); |
| + asr, kAudioSampleRateMax); |
|
Ami GONE FROM CHROMIUM
2014/01/29 21:29:02
If you had a PRESUBMIT it would bitch about this.
|
| } else { |
| UMA_HISTOGRAM_COUNTS( |
| "Media.FallbackHardwareAudioSamplesPerSecondUnexpected", |