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

Unified Diff: media/audio/audio_debug_recording_manager_unittest.cc

Issue 2885173002: Make AudioDebugFileWriter create its own file worker thread. (Closed)
Patch Set: Remove useless include 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/audio_debug_recording_manager.cc ('k') | media/audio/audio_input_controller.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/audio/audio_debug_recording_manager_unittest.cc
diff --git a/media/audio/audio_debug_recording_manager_unittest.cc b/media/audio/audio_debug_recording_manager_unittest.cc
index 95d08968070cabdff2f232ce9dbaccbce75f4616..9e3d0887a9a41e24189f31f0329e3513a611659c 100644
--- a/media/audio/audio_debug_recording_manager_unittest.cc
+++ b/media/audio/audio_debug_recording_manager_unittest.cc
@@ -58,11 +58,9 @@ class MockAudioDebugRecordingHelper : public AudioDebugRecordingHelper {
MockAudioDebugRecordingHelper(
const AudioParameters& params,
scoped_refptr<base::SingleThreadTaskRunner> task_runner,
- scoped_refptr<base::SingleThreadTaskRunner> file_task_runner,
base::OnceClosure on_destruction_closure)
: AudioDebugRecordingHelper(params,
std::move(task_runner),
- std::move(file_task_runner),
base::OnceClosure()),
on_destruction_closure_in_mock_(std::move(on_destruction_closure)) {
if (g_expect_enable_after_create_helper)
@@ -90,21 +88,17 @@ class MockAudioDebugRecordingHelper : public AudioDebugRecordingHelper {
class AudioDebugRecordingManagerUnderTest : public AudioDebugRecordingManager {
public:
AudioDebugRecordingManagerUnderTest(
- scoped_refptr<base::SingleThreadTaskRunner> task_runner,
- scoped_refptr<base::SingleThreadTaskRunner> file_task_runner)
- : AudioDebugRecordingManager(std::move(task_runner),
- std::move(file_task_runner)) {}
+ scoped_refptr<base::SingleThreadTaskRunner> task_runner)
+ : AudioDebugRecordingManager(std::move(task_runner)) {}
~AudioDebugRecordingManagerUnderTest() override {}
private:
std::unique_ptr<AudioDebugRecordingHelper> CreateAudioDebugRecordingHelper(
const AudioParameters& params,
scoped_refptr<base::SingleThreadTaskRunner> task_runner,
- scoped_refptr<base::SingleThreadTaskRunner> file_task_runner,
base::OnceClosure on_destruction_closure) override {
return base::MakeUnique<MockAudioDebugRecordingHelper>(
- params, std::move(task_runner), std::move(file_task_runner),
- std::move(on_destruction_closure));
+ params, std::move(task_runner), std::move(on_destruction_closure));
}
DISALLOW_COPY_AND_ASSIGN(AudioDebugRecordingManagerUnderTest);
@@ -114,7 +108,7 @@ class AudioDebugRecordingManagerUnderTest : public AudioDebugRecordingManager {
class AudioDebugRecordingManagerTest : public ::testing::Test {
public:
AudioDebugRecordingManagerTest()
- : manager_(message_loop_.task_runner(), message_loop_.task_runner()),
+ : manager_(message_loop_.task_runner()),
base_file_path_(base::FilePath::FromUTF8Unsafe("base_path")) {}
~AudioDebugRecordingManagerTest() override {}
« no previous file with comments | « media/audio/audio_debug_recording_manager.cc ('k') | media/audio/audio_input_controller.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698