Index: media/audio/win/audio_manager_win.cc |
diff --git a/media/audio/win/audio_manager_win.cc b/media/audio/win/audio_manager_win.cc |
index df7faf452eae1c6b2112968396fc299647bb0fac..a3d5ead202f401e31f25c54713b657e7a02c48fb 100644 |
--- a/media/audio/win/audio_manager_win.cc |
+++ b/media/audio/win/audio_manager_win.cc |
@@ -126,19 +126,14 @@ |
return (base::win::GetVersion() == base::win::VERSION_VISTA) ? 4 : 3; |
} |
-AudioManagerWin::AudioManagerWin( |
- scoped_refptr<base::SingleThreadTaskRunner> task_runner, |
- scoped_refptr<base::SingleThreadTaskRunner> worker_task_runner, |
- AudioLogFactory* audio_log_factory) |
- : AudioManagerBase(std::move(task_runner), |
- std::move(worker_task_runner), |
- audio_log_factory), |
+AudioManagerWin::AudioManagerWin(AudioLogFactory* audio_log_factory) |
+ : AudioManagerBase(audio_log_factory), |
// |CoreAudioUtil::IsSupported()| uses static variables to avoid doing |
// multiple initializations. This is however not thread safe. |
// So, here we call it explicitly before we kick off the audio thread |
// or do any other work. |
- enumeration_type_(CoreAudioUtil::IsSupported() ? kMMDeviceEnumeration |
- : kWaveEnumeration) { |
+ enumeration_type_(CoreAudioUtil::IsSupported() ? |
+ kMMDeviceEnumeration : kWaveEnumeration) { |
SetMaxOutputStreamsAllowed(kMaxOutputStreams); |
// WARNING: This is executed on the UI loop, do not add any code here which |
@@ -152,6 +147,10 @@ |
} |
AudioManagerWin::~AudioManagerWin() { |
+ // It's safe to post a task here since Shutdown() will wait for all tasks to |
+ // complete before returning. |
+ GetTaskRunner()->PostTask(FROM_HERE, base::Bind( |
+ &AudioManagerWin::ShutdownOnAudioThread, base::Unretained(this))); |
Shutdown(); |
} |
@@ -173,6 +172,11 @@ |
base::Bind(&AudioManagerWin::NotifyAllOutputDeviceChangeListeners, |
base::Unretained(this))))); |
} |
+} |
+ |
+void AudioManagerWin::ShutdownOnAudioThread() { |
+ DCHECK(GetTaskRunner()->BelongsToCurrentThread()); |
+ output_device_listener_.reset(); |
} |
base::string16 AudioManagerWin::GetAudioInputDeviceModel() { |
@@ -533,13 +537,8 @@ |
} |
/// static |
-ScopedAudioManagerPtr CreateAudioManager( |
- scoped_refptr<base::SingleThreadTaskRunner> task_runner, |
- scoped_refptr<base::SingleThreadTaskRunner> worker_task_runner, |
- AudioLogFactory* audio_log_factory) { |
- return ScopedAudioManagerPtr( |
- new AudioManagerWin(std::move(task_runner), std::move(worker_task_runner), |
- audio_log_factory)); |
+AudioManager* CreateAudioManager(AudioLogFactory* audio_log_factory) { |
+ return new AudioManagerWin(audio_log_factory); |
} |
} // namespace media |