Moving SwitchOutputDevice out of OutputDevice interface, eliminating OutputDevice

media/base/mock_audio_renderer_sink.h

 // 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.
 
 #ifndef MEDIA_BASE_MOCK_AUDIO_RENDERER_SINK_H_
 #define MEDIA_BASE_MOCK_AUDIO_RENDERER_SINK_H_
 
 #include <string>
 
 #include "base/macros.h"
 #include "media/audio/audio_parameters.h"
 #include "media/base/audio_renderer_sink.h"
 #include "testing/gmock/include/gmock/gmock.h"
 
 namespace media {
 
 class FakeOutputDevice;
 
-class MockAudioRendererSink : public RestartableAudioRendererSink {
+class MockAudioRendererSink : public SwitchableAudioRendererSink {
  public:
+  class OutputDeviceInfoFactory {
+   public:
+    virtual OutputDeviceInfo* NewOutputDeviceInfo(

[Guido Urdaneta, 2016/03/22 15:30:18]

Why should this return pointers?
Why a factory for this?
Is there an implementation of this? I didn't see it.

[o1ka, 2016/03/22 16:13:56]

Ok, removing this for now.

+        const std::string& device_id) = 0;
+
+   protected:
+    virtual ~OutputDeviceInfoFactory() {}
+  };
+
   MockAudioRendererSink();
   explicit MockAudioRendererSink(OutputDeviceStatus device_status);
+  MockAudioRendererSink(OutputDeviceStatus device_status,
+                        const AudioParameters& device_output_params);
+  MockAudioRendererSink(const std::string& device_id,
+                        OutputDeviceStatus device_status);
+  MockAudioRendererSink(const std::string& device_id,
+                        OutputDeviceStatus device_status,
+                        const AudioParameters& device_output_params);
+
+  // Use this constructor if you need a working implementation of
+  // SwitchOutputDevice(). Do not pass nullptr as |device_factory|.
+  MockAudioRendererSink(OutputDeviceInfoFactory* device_info_factory,
+                        const std::string& device_id);
 
   MOCK_METHOD0(Start, void());
   MOCK_METHOD0(Stop, void());
   MOCK_METHOD0(Pause, void());
   MOCK_METHOD0(Play, void());
   MOCK_METHOD1(SetVolume, bool(double volume));
-  OutputDevice* GetOutputDevice();
 
+  OutputDeviceInfo GetOutputDeviceInfo();
+
+  void SwitchOutputDevice(const std::string& device_id,
+                          const url::Origin& security_origin,
+                          const OutputDeviceStatusCB& callback) override;
   void Initialize(const AudioParameters& params,
                   RenderCallback* renderer) override;
   AudioRendererSink::RenderCallback* callback() { return callback_; }
 
  protected:
   ~MockAudioRendererSink() override;
 
  private:
+  OutputDeviceInfoFactory* device_info_factory_;
   RenderCallback* callback_;
-  scoped_ptr<FakeOutputDevice> output_device_;
+  scoped_ptr<OutputDeviceInfo> output_device_info_;

[Guido Urdaneta, 2016/03/22 15:30:18]

I think this shouldn't be a pointer.

[o1ka, 2016/03/22 16:13:56]

Done.

 
   DISALLOW_COPY_AND_ASSIGN(MockAudioRendererSink);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_MOCK_AUDIO_RENDERER_SINK_H_