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

Unified Diff: extensions/browser/api/audio/audio_service.h

Issue 2605983002: Simplify logic behind chrome.audio.setActiveDevices (Closed)
Patch Set: . Created 3 years, 11 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: extensions/browser/api/audio/audio_service.h
diff --git a/extensions/browser/api/audio/audio_service.h b/extensions/browser/api/audio/audio_service.h
index e733f7afd399dc7240e4cb8d373193667205b25a..449c24031a0069e921189473ad75612bcdceefab 100644
--- a/extensions/browser/api/audio/audio_service.h
+++ b/extensions/browser/api/audio/audio_service.h
@@ -5,6 +5,7 @@
#ifndef EXTENSIONS_BROWSER_API_AUDIO_AUDIO_SERVICE_H_
#define EXTENSIONS_BROWSER_API_AUDIO_AUDIO_SERVICE_H_
+#include <memory>
#include <string>
#include <vector>
@@ -55,6 +56,10 @@ class AudioService {
virtual bool GetInfo(OutputInfo* output_info_out,
InputInfo* input_info_out) = 0;
+ virtual bool SetActiveDeviceLists(
+ const std::unique_ptr<DeviceIdList>& input_devices,
+ const std::unique_ptr<DeviceIdList>& output_devives) = 0;
jennyz 2017/01/07 00:33:00 Please add some comments about the new function.
tbarzic 2017/01/07 05:16:31 Done.
+
// Sets the active devices to the devices specified by |device_list|.
// It can pass in the "complete" active device list of either input
// devices, or output devices, or both. If only input devices are passed in,

Powered by Google App Engine
This is Rietveld 408576698