Add UMA reporting for audio hardware parameters.

media/audio/audio_output_resampler.cc

Index: media/audio/audio_output_resampler.cc
diff --git a/media/audio/audio_output_resampler.cc b/media/audio/audio_output_resampler.cc
index e67e2771649443f2c6c494fef07940c4162ca19a..a861beff70c18e868a051e8a5e0ba4478dfdad32 100644
--- a/media/audio/audio_output_resampler.cc
+++ b/media/audio/audio_output_resampler.cc
@@ -8,16 +8,56 @@
 #include "base/bind_helpers.h"
 #include "base/compiler_specific.h"
 #include "base/message_loop.h"
+#include "base/metrics/histogram.h"
 #include "base/time.h"
 #include "media/audio/audio_io.h"
 #include "media/audio/audio_output_dispatcher_impl.h"
 #include "media/audio/audio_output_proxy.h"
 #include "media/audio/audio_util.h"
+#include "media/audio/sample_rates.h"
 #include "media/base/audio_pull_fifo.h"
+#include "media/base/limits.h"
 #include "media/base/multi_channel_resampler.h"
 
+#if defined(OS_WIN)
+#include "media/audio/win/audio_low_latency_output_win.h"
+#endif
+
 namespace media {
 
+static void RecordStats(const AudioParameters& output_params) {
+  UMA_HISTOGRAM_ENUMERATION(
+      "Media.HardwareAudioBitsPerChannel", output_params.bits_per_sample(),

[scherkus (not reviewing), 2012/09/12 13:34:50]

don't you have a WASAPI version of this?

[DaleCurtis, 2012/09/12 14:03:52]

Not sure I'm landing that. I don't think it will work w/o supporting 24-bit
interleave on ToInterleaved() first.

+      limits::kMaxBitsPerSample);
+#if defined(OS_WIN)
+  // TODO(dalecurtis): Since channel mixing is handle by the output device right

[scherkus (not reviewing), 2012/09/12 13:35:13]

s/handle/handled/

[DaleCurtis, 2012/09/12 14:03:52]

Done.

+  // now and not by AudioOutputResampler, we need to query for hardware channel
+  // information.  Remove once AOR handles this, http://crbug.com/138762
+  UMA_HISTOGRAM_ENUMERATION(
+      "Media.HardwareAudioChannelLayout",
+      WASAPIAudioOutputStream::HardwareChannelLayout(), CHANNEL_LAYOUT_MAX);
+  UMA_HISTOGRAM_ENUMERATION(
+      "Media.HardwareAudioChannelCount",
+      WASAPIAudioOutputStream::HardwareChannelCount(), limits::kMaxChannels);
+#else
+  UMA_HISTOGRAM_ENUMERATION(
+      "Media.HardwareAudioChannelLayout", output_params.channel_layout(),
+      CHANNEL_LAYOUT_MAX);
+  UMA_HISTOGRAM_ENUMERATION(
+      "Media.HardwareAudioChannelCount", output_params.channels(),
+      limits::kMaxChannels);
+#endif
+  AudioSampleRate asr = media::AsAudioSampleRate(output_params.sample_rate());
+  if (asr != kUnexpectedAudioSampleRate) {
+    UMA_HISTOGRAM_ENUMERATION(
+        "Media.HardwareAudioSamplesPerSecond", asr, kUnexpectedAudioSampleRate);
+  } else {
+    UMA_HISTOGRAM_COUNTS(
+        "Media.HardwareAudioSamplesPerSecondUnexpected",
+        output_params.sample_rate());
+  }
+}
+
 AudioOutputResampler::AudioOutputResampler(AudioManager* audio_manager,
                                            const AudioParameters& input_params,
                                            const AudioParameters& output_params,
@@ -77,6 +117,9 @@ AudioOutputResampler::AudioOutputResampler(AudioManager* audio_manager,
   // we've stabilized the issues there.
   dispatcher_ = new AudioOutputDispatcherImpl(
       audio_manager, output_params, close_delay);
+
+  // Record UMA statistics for the hardware configuration.
+  RecordStats(output_params);
 }
 
 AudioOutputResampler::~AudioOutputResampler() {}