Moving SwitchOutputDevice out of OutputDevice interface, eliminating OutputDevice

content/public/renderer/media_stream_audio_renderer.h

Index: content/public/renderer/media_stream_audio_renderer.h
diff --git a/content/public/renderer/media_stream_audio_renderer.h b/content/public/renderer/media_stream_audio_renderer.h
index 5a5d68d3d6234171c33df2e2462e8d430e9c0402..efa6b8e4fa30f79020e13c1591b07e980dc61b87 100644
--- a/content/public/renderer/media_stream_audio_renderer.h
+++ b/content/public/renderer/media_stream_audio_renderer.h
@@ -41,6 +41,22 @@ class MediaStreamAudioRenderer
   // supported.
   virtual media::OutputDevice* GetOutputDevice() = 0;
 
+  // Attempts to switch the audio output device.
+  // Once the attempt is finished, |callback| is invoked with the
+  // result of the operation passed as a parameter. The result is a value from
+  // the  media::SwitchOutputDeviceResult enum.

[Guido Urdaneta, 2016/03/17 17:38:23]

s/media::SwitchOutputDeviceResult/media::OutputDeviceStatus

[o1ka, 2016/03/18 10:45:40]

Done.

+  // There is no guarantee about the thread where |callback| will
+  // be invoked, so users are advised to use media::BindToCurrentLoop() to
+  // ensure that |callback| runs on the correct thread.
+  // Note also that copy constructors and destructors for arguments bound to
+  // |callback| may run on arbitrary threads as |callback| is moved across
+  // threads. It is advisable to bind arguments such that they are released by
+  // |callback| when it runs in order to avoid surprises.
+  virtual void SwitchOutputDevice(
+      const std::string& device_id,
+      const url::Origin& security_origin,
+      const media::SwitchOutputDeviceCB& callback) = 0;
+
   // Time stamp that reflects the current render time. Should not be updated
   // when paused.
   virtual base::TimeDelta GetCurrentRenderTime() const = 0;