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", |