Moving SwitchOutputDevice out of OutputDevice interface, eliminating OutputDevice

media/base/fake_audio_renderer_sink.cc

Index: media/base/fake_audio_renderer_sink.cc
diff --git a/media/base/fake_audio_renderer_sink.cc b/media/base/fake_audio_renderer_sink.cc
index 5ce60740b05db9416cde068aac1a693a417d6525..40c3cdf9c270e8de6c0796700342da302d2f0069 100644
--- a/media/base/fake_audio_renderer_sink.cc
+++ b/media/base/fake_audio_renderer_sink.cc
@@ -7,15 +7,20 @@
 #include "base/bind.h"
 #include "base/location.h"
 #include "base/logging.h"
-#include "base/single_thread_task_runner.h"
-#include "media/base/fake_output_device.h"
 
 namespace media {
 
 FakeAudioRendererSink::FakeAudioRendererSink()
     : state_(kUninitialized),
       callback_(NULL),
-      output_device_(new FakeOutputDevice) {}
+      output_device_info_(
+          std::string(),
+          OUTPUT_DEVICE_STATUS_OK,
+          media::AudioParameters(media::AudioParameters::AUDIO_FAKE,
+                                 media::CHANNEL_LAYOUT_STEREO,
+                                 media::AudioParameters::kTelephoneSampleRate,
+                                 16,

[miu, 2016/03/24 21:32:02]

Side Note: At some point, we should evaluate whether the bits_per_sample
property in AudioParameters can be eliminated.  IMO, we should try to get rid of
it since audio data is passed around in planar float format everywhere.

[o1ka, 2016/03/29 13:39:54]

Yes, we should consider this.

+                                 1)) {}
 
 FakeAudioRendererSink::~FakeAudioRendererSink() {
   DCHECK(!callback_);
@@ -56,8 +61,8 @@ bool FakeAudioRendererSink::SetVolume(double volume) {
   return true;
 }
 
-OutputDevice* FakeAudioRendererSink::GetOutputDevice() {
-  return output_device_.get();
+OutputDeviceInfo FakeAudioRendererSink::GetOutputDeviceInfo() {
+  return output_device_info_;
 }
 
 bool FakeAudioRendererSink::Render(AudioBus* dest,