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

Unified Diff: content/browser/renderer_host/media/audio_input_device_manager.h

Issue 2763383002: Switching AudioInputDeviceManager from using AudioManager interface to AudioSystem one. (Closed)
Patch Set: rebase Created 3 years, 8 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: content/browser/renderer_host/media/audio_input_device_manager.h
diff --git a/content/browser/renderer_host/media/audio_input_device_manager.h b/content/browser/renderer_host/media/audio_input_device_manager.h
index 254051483a74ee45f430eb579d5ddc4421304787..b99b467de5e4209e9a62bf99130d5ab7fa37f812 100644
--- a/content/browser/renderer_host/media/audio_input_device_manager.h
+++ b/content/browser/renderer_host/media/audio_input_device_manager.h
@@ -26,7 +26,7 @@
#include "content/public/common/media_stream_request.h"
namespace media {
-class AudioManager;
+class AudioSystem;
}
namespace content {
@@ -40,7 +40,7 @@ class CONTENT_EXPORT AudioInputDeviceManager : public MediaStreamProvider {
// TODO(xians): Remove it when the webrtc unittest does not need it any more.
static const int kFakeOpenSessionId;
- explicit AudioInputDeviceManager(media::AudioManager* audio_manager);
+ explicit AudioInputDeviceManager(media::AudioSystem* audio_system);
// Gets the opened device info by |session_id|. Returns NULL if the device
// is not opened, otherwise the opened device. Called on IO thread.
@@ -66,19 +66,18 @@ class CONTENT_EXPORT AudioInputDeviceManager : public MediaStreamProvider {
typedef std::vector<StreamDeviceInfo> StreamDeviceList;
~AudioInputDeviceManager() override;
- // Opens the device on media stream device thread.
- void OpenOnDeviceThread(int session_id, const MediaStreamDevice& device);
+ // Callback called on IO thread when device is opened.
+ void OpenedOnIOThread(int session_id,
+ const MediaStreamDevice& device,
+ base::TimeTicks start_time,
+ const media::AudioParameters& input_params,
+ const media::AudioParameters& matched_output_params,
+ const std::string& matched_output_device_id);
- // Callback used by OpenOnDeviceThread(), called with the session_id
- // referencing the opened device on IO thread.
- void OpenedOnIOThread(int session_id, const StreamDeviceInfo& info);
- // Callback used by CloseOnDeviceThread(), called with the session_id
- // referencing the closed device on IO thread.
+ // Callback called on IO thread with the session_id referencing the closed
+ // device.
void ClosedOnIOThread(MediaStreamType type, int session_id);
- // Verifies that the calling thread is media stream device thread.
- bool IsOnDeviceThread() const;
-
// Helper to return iterator to the device referenced by |session_id|. If no
// device is found, it will return devices_.end().
StreamDeviceList::iterator GetDevice(int session_id);
@@ -98,10 +97,7 @@ class CONTENT_EXPORT AudioInputDeviceManager : public MediaStreamProvider {
int keyboard_mic_streams_count_;
#endif
- media::AudioManager* const audio_manager_; // Weak.
-
- // The message loop of media stream device thread that this object runs on.
- scoped_refptr<base::SingleThreadTaskRunner> device_task_runner_;
+ media::AudioSystem* const audio_system_;
DISALLOW_COPY_AND_ASSIGN(AudioInputDeviceManager);
};
« no previous file with comments | « content/browser/browser_main_loop.h ('k') | content/browser/renderer_host/media/audio_input_device_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698