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

Unified Diff: remoting/host/audio_capturer_win.h

Issue 2809293006: [Chromoting] Move DefaultAudioDeviceChangeDetector out of AudioCapturerWin (Closed)
Patch Set: Remove override of destructor 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
« no previous file with comments | « no previous file | remoting/host/audio_capturer_win.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/host/audio_capturer_win.h
diff --git a/remoting/host/audio_capturer_win.h b/remoting/host/audio_capturer_win.h
index 0019242fc48f2f9646f6260ab617a9a43c0fbf91..f4571ad229f115e8b01e3fe101442764137e3141 100644
--- a/remoting/host/audio_capturer_win.h
+++ b/remoting/host/audio_capturer_win.h
@@ -22,6 +22,8 @@
namespace remoting {
+class DefaultAudioDeviceChangeDetector;
+
class AudioCapturerWin : public AudioCapturer {
public:
AudioCapturerWin();
@@ -31,11 +33,6 @@ class AudioCapturerWin : public AudioCapturer {
bool Start(const PacketCapturedCallback& callback) override;
private:
- // An IMMNotificationClient implementation to detect the event of default
- // audio device recently changed. If it indicates a changed happend recently,
- // we need to recreate all audio components.
- class MMNotificationClient;
-
// Executes Deinitialize() and Initialize(). If Initialize() function call
// returns false, Deinitialize() will be called again to ensure we will
// initialize COM components again.
@@ -79,13 +76,12 @@ class AudioCapturerWin : public AudioCapturer {
AudioSilenceDetector silence_detector_;
base::win::ScopedCoMem<WAVEFORMATEX> wave_format_ex_;
- base::win::ScopedComPtr<IMMDeviceEnumerator> mm_device_enumerator_;
base::win::ScopedComPtr<IAudioCaptureClient> audio_capture_client_;
base::win::ScopedComPtr<IAudioClient> audio_client_;
base::win::ScopedComPtr<IMMDevice> mm_device_;
base::win::ScopedComPtr<IAudioEndpointVolume> audio_volume_;
- const std::unique_ptr<MMNotificationClient> mm_notification_client_;
+ std::unique_ptr<DefaultAudioDeviceChangeDetector> default_device_detector_;
HRESULT last_capture_error_;
« no previous file with comments | « no previous file | remoting/host/audio_capturer_win.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698