Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(551)

Unified Diff: media/base/output_device.h

Issue 1809093003: Moving SwitchOutputDevice out of OutputDevice interface, eliminating OutputDevice (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Replace RestartableAudioRendererSink with SwitchableAudioRendererSink in webmediaplayer_impl unit t… Created 4 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: media/base/output_device.h
diff --git a/media/base/output_device.h b/media/base/output_device.h
index 76f88c78ed927ff1f9f27dc594fe5ae4e4e12fbf..ab9feaac0ee2e91d20f45f669d862ddb2066a1c1 100644
--- a/media/base/output_device.h
+++ b/media/base/output_device.h
@@ -30,21 +30,6 @@ typedef base::Callback<void(OutputDeviceStatus)> SwitchOutputDeviceCB;
class OutputDevice {
public:
- // 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.
- // 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 SwitchOutputDeviceCB& callback) = 0;
-
// Returns the device's audio output parameters.
// The return value is undefined if the device status (as returned by
// GetDeviceStatus()) is different from OUTPUT_DEVICE_STATUS_OK.
@@ -58,6 +43,9 @@ class OutputDevice {
// available. Must never be called on the IO thread.
virtual OutputDeviceStatus GetDeviceStatus() = 0;
+ // Returns the device ID.
+ virtual std::string GetDeviceId() = 0;
+
protected:
virtual ~OutputDevice() {}
};

Powered by Google App Engine
This is Rietveld 408576698