Moved AudioUtil static functions to AudioManager interfaces

media/audio/audio_manager_base.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 "media/audio/audio_manager_base.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/command_line.h"
 #include "base/message_loop_proxy.h"
@@ skipping 367 matching lines @@
 
 AudioParameters AudioManagerBase::GetPreferredLowLatencyOutputStreamParameters(
     const AudioParameters& input_params) {
 #if defined(OS_IOS)
   // IOS implements audio input only.
   NOTIMPLEMENTED();
   return AudioParameters();
 #else
   // TODO(dalecurtis): This should include bits per channel and channel layout
   // eventually.
+  AudioParameters default_params = GetDefaultOutputStreamParameters();
   return AudioParameters(
       AudioParameters::AUDIO_PCM_LOW_LATENCY,
       input_params.channel_layout(), input_params.input_channels(),
-      GetAudioHardwareSampleRate(), 16, GetAudioHardwareBufferSize());
+      default_params.sample_rate(), 16, default_params.frames_per_buffer());
 #endif  // defined(OS_IOS)
 }
 
 void AudioManagerBase::AddOutputDeviceChangeListener(
     AudioDeviceListener* listener) {
   DCHECK(message_loop_->BelongsToCurrentThread());
   output_listeners_.AddObserver(listener);
 }
 
 void AudioManagerBase::RemoveOutputDeviceChangeListener(
     AudioDeviceListener* listener) {
   DCHECK(message_loop_->BelongsToCurrentThread());
   output_listeners_.RemoveObserver(listener);
 }
 
+AudioParameters AudioManagerBase::GetDefaultOutputStreamParameters() {
+  return AudioParameters();

[Chris Rogers, 2013/02/27 20:06:14]

seems a bit dangerous -- should probably add NOTREACHED()

[tommi (sloooow) - chröme, 2013/03/01 11:29:29]

Can we make these methods pure virtual?

[no longer working on chromium, 2013/03/01 16:44:33]

Done.

[no longer working on chromium, 2013/03/01 16:44:33]

I make it pure virtual to for the developer to implement their own
GetDefaultOutputStreamParameters on all AudioManagers

+}
+
+AudioParameters AudioManagerBase::GetDefaultInputStreamParameters(
+    const std::string& device_id) {
+  return AudioParameters();

[Chris Rogers, 2013/02/27 20:06:14]

seems a bit dangerous -- should probably add NOTREACHED()

[no longer working on chromium, 2013/03/01 16:44:33]

Done.

+}
+
 void AudioManagerBase::NotifyAllOutputDeviceChangeListeners() {
   DCHECK(message_loop_->BelongsToCurrentThread());
   DVLOG(1) << "Firing OnDeviceChange() notifications.";
   FOR_EACH_OBSERVER(AudioDeviceListener, output_listeners_, OnDeviceChange());
 }
 
 }  // namespace media