Pass more detailed audio hardware configuration information to the renderer

content/common/media/media_param_traits.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/common/media/media_param_traits.h"
 
 #include "base/stringprintf.h"
 #include "media/audio/audio_parameters.h"
 #include "media/base/limits.h"
 #include "media/video/capture/video_capture_types.h"
@@ skipping 26 matching lines @@
       !m->ReadInt(iter, &channel_layout) ||
       !m->ReadInt(iter, &sample_rate) ||
       !m->ReadInt(iter, &bits_per_sample) ||
       !m->ReadInt(iter, &frames_per_buffer) ||
       !m->ReadInt(iter, &channels) ||
       !m->ReadInt(iter, &input_channels))
     return false;
   r->Reset(static_cast<AudioParameters::Format>(format),
            static_cast<ChannelLayout>(channel_layout), input_channels,
            sample_rate, bits_per_sample, frames_per_buffer);
+  if (channel_layout == media::CHANNEL_LAYOUT_DISCRETE)

[DaleCurtis, 2013/03/07 02:41:39]

Does anything use Reset() that doesn't provide all values? If not, I'd just add
it to that function. It's a bit weird to do this check here.

[Chris Rogers, 2013/03/09 01:37:50]

Done.

+    r->SetDiscreteChannels(channels);
   if (!r->IsValid())
     return false;
   return true;
 }
 
 void ParamTraits<AudioParameters>::Log(const AudioParameters& p,
                                        std::string* l) {
   l->append(base::StringPrintf("<AudioParameters>"));
 }
 
@@ skipping 29 matching lines @@
 
   return true;
 }
 
 void ParamTraits<VideoCaptureParams>::Log(const VideoCaptureParams& p,
                                           std::string* l) {
   l->append(base::StringPrintf("<VideoCaptureParams>"));
 }
 
 }