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

Unified Diff: media/audio/fake_audio_manager.cc

Issue 2784433002: Ensures that audio tasks cannot run after AudioManager is deleted. (Closed)
Patch Set: rebase Created 3 years, 7 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 | « media/audio/fake_audio_manager.h ('k') | media/audio/linux/audio_manager_linux.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/audio/fake_audio_manager.cc
diff --git a/media/audio/fake_audio_manager.cc b/media/audio/fake_audio_manager.cc
index 07e88a5b964c2d182451888d09ba2138049a62f8..7460a0121cdc7c863154b17c33565240b800771a 100644
--- a/media/audio/fake_audio_manager.cc
+++ b/media/audio/fake_audio_manager.cc
@@ -16,18 +16,11 @@ const int kDefaultSampleRate = 48000;
} // namespace
-FakeAudioManager::FakeAudioManager(
- 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) {
-}
+FakeAudioManager::FakeAudioManager(std::unique_ptr<AudioThread> audio_thread,
+ AudioLogFactory* audio_log_factory)
+ : AudioManagerBase(std::move(audio_thread), audio_log_factory) {}
-FakeAudioManager::~FakeAudioManager() {
- Shutdown();
-}
+FakeAudioManager::~FakeAudioManager() = default;
// Implementation of AudioManager.
bool FakeAudioManager::HasAudioOutputDevices() { return false; }
« no previous file with comments | « media/audio/fake_audio_manager.h ('k') | media/audio/linux/audio_manager_linux.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698