Switching audio clients to using RestartableAudioRendererSink interface as a sink.

content/renderer/media/audio_renderer_mixer_manager.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/renderer/media/audio_renderer_mixer_manager.h"
 
 #include <string>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "build/build_config.h"
 #include "content/renderer/media/audio_device_factory.h"
 #include "media/audio/audio_output_device.h"
 #include "media/base/audio_hardware_config.h"
 #include "media/base/audio_renderer_mixer.h"
 #include "media/base/audio_renderer_mixer_input.h"
 
 namespace content {
 
+// static
+media::AudioParameters AudioRendererMixerManager::GetOutputHWParams(

[DaleCurtis, 2016/02/10 23:25:33]

Put this in function order relative to the header file. Only internal static
functions go at the top of the file.

[o1ka, 2016/02/11 17:18:23]

Done.

+    int render_frame_id,
+    int session_id,
+    const std::string& device_id,
+    const url::Origin& security_origin) {
+  media::AudioParameters params;  // Invalid parameters to return by default.
+
+  // TODO(olka): first try to lookup an existing device (cached or belonging
+  // to some mixer) and reuse it.

[DaleCurtis, 2016/02/10 23:25:33]

Can you file a bug and list it here too?

[o1ka, 2016/02/11 17:18:23]

Done.

+
+  // AudioOutputDevice is the only interface we have to communicate with output
+  // device via IPC. So, that's how we get the parameters when there is no
+  // AudioOutputDevice:
+  scoped_refptr<media::AudioOutputDevice> device =
+      AudioDeviceFactory::NewOutputDevice(render_frame_id, session_id,
+                                          device_id, security_origin);
+
+  if (device->GetDeviceStatus() == media::OUTPUT_DEVICE_STATUS_OK)
+    params = device->GetOutputParameters();
+
+  device->Stop();  // TODO(olka): temporary cash for future reuse.
+  return params;
+}
+
 AudioRendererMixerManager::AudioRendererMixerManager()
     : sink_for_testing_(nullptr) {}
 
 AudioRendererMixerManager::~AudioRendererMixerManager() {
   // References to AudioRendererMixers may be owned by garbage collected
   // objects.  During process shutdown they may be leaked, so, transitively,
   // |mixers_| may leak (i.e., may be non-empty at this time) as well.
 }
 
 media::AudioRendererMixerInput* AudioRendererMixerManager::CreateInput(
     int source_render_frame_id,
     const std::string& device_id,
     const url::Origin& security_origin) {
   return new media::AudioRendererMixerInput(
       base::Bind(&AudioRendererMixerManager::GetMixer, base::Unretained(this),
                  source_render_frame_id),
       base::Bind(&AudioRendererMixerManager::RemoveMixer,
                  base::Unretained(this), source_render_frame_id),
-      device_id, security_origin);
+      base::Bind(&AudioRendererMixerManager::GetOutputHWParams,
+                 source_render_frame_id, 0),  // Session id is 0.
+      device_id,
+      security_origin);
 }
 
 void AudioRendererMixerManager::SetAudioRendererSinkForTesting(
     media::AudioRendererSink* sink) {
   sink_for_testing_ = sink;
 }
 
 media::AudioRendererMixer* AudioRendererMixerManager::GetMixer(
     int source_render_frame_id,
     const media::AudioParameters& params,
@@ skipping 92 matching lines @@
     int source_render_frame_id,
     const media::AudioParameters& params,
     const std::string& device_id,
     const url::Origin& security_origin)
     : source_render_frame_id(source_render_frame_id),
       params(params),
       device_id(device_id),
       security_origin(security_origin) {}
 
 }  // namespace content