Make AudioDebugFileWriter create its own file worker thread.

media/audio/audio_debug_file_writer.h

Index: media/audio/audio_debug_file_writer.h
diff --git a/media/audio/audio_debug_file_writer.h b/media/audio/audio_debug_file_writer.h
index 9d8a4c0f9b940e5076520c1a0d1ebf43a7916924..26a21a5fe44d2e3a28e0db817e225629a1439ba2 100644
--- a/media/audio/audio_debug_file_writer.h
+++ b/media/audio/audio_debug_file_writer.h
@@ -14,6 +14,7 @@
 #include "base/memory/ref_counted.h"
 #include "base/sequence_checker.h"
 #include "base/single_thread_task_runner.h"
+#include "base/task_scheduler/post_task.h"
 #include "media/base/audio_parameters.h"
 #include "media/base/media_export.h"
 
@@ -29,9 +30,7 @@ class MEDIA_EXPORT AudioDebugFileWriter {
   // Number of channels and sample rate are used from |params|, the other
   // parameters are ignored. The number of channels in the data passed to
   // Write() must match |params|.
-  AudioDebugFileWriter(
-      const AudioParameters& params,
-      scoped_refptr<base::SingleThreadTaskRunner> file_task_runner);
+  AudioDebugFileWriter(const AudioParameters& params);
 
   virtual ~AudioDebugFileWriter();
 
@@ -84,7 +83,9 @@ class MEDIA_EXPORT AudioDebugFileWriter {
   base::SequenceChecker client_sequence_checker_;
 
   // The task runner to do file output operations on.
-  scoped_refptr<base::SingleThreadTaskRunner> file_task_runner_;
+  const scoped_refptr<base::SingleThreadTaskRunner> file_task_runner_ =
+      base::CreateSingleThreadTaskRunnerWithTraits(

[gab, 2017/05/18 16:10:10]

In general we don't want to create SingleThreadTaskRunners (except with a TODO
to remove it when another top-level bug is fixed or a real explanation of why).

In this case it's just because AudioFileWriter::Create() takes a
SingleThreadTaskRunner, but looking at it it doesn't need to.

AudioFileWriter::AudioFileWriter() merely takes the task runner to assert it's
running on it, that can easily be turned into a SequencedTaskRunner but even
better than that would be to have it use a SequenceChecker to check that all of
its methods run in sequence (will need to DetachFromSequence in
AudioFileWriter::AudioFileWriter() as it's not constructed on same sequence it's
used -- see sequence_checker.h for details).

SequenceChecker + implicit AssertIOAllowed() calls from using base file methods
guarantees this class is being used in sequence with MayBlock() trait. That's
preferred and cleaner than taking a task_runner just to assume running on it.

Then there's OnThreadDeleter which could easily be made OnSequenceDeleter +
SequenceTaskRunner.

And that's it.

So I'd say, please make those changes ahead of this CL and have this one use
base::CreateSequenceTaskRunnerWithTraits() :)

[Sébastien Marchand, 2017/05/18 17:43:06]

Thanks for the detailed comment! I'm addressing this in
https://codereview.chromium.org/2891983002/

+          {base::MayBlock(), base::TaskPriority::BACKGROUND});
 
   DISALLOW_COPY_AND_ASSIGN(AudioDebugFileWriter);
 };