Chromium Code Reviews| 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..148c1729fed168c10885f888ceca5005a3366ed8 100644 |
| --- a/media/audio/audio_debug_file_writer.h |
| +++ b/media/audio/audio_debug_file_writer.h |
| @@ -13,6 +13,7 @@ |
| #include "base/macros.h" |
| #include "base/memory/ref_counted.h" |
| #include "base/sequence_checker.h" |
| +#include "base/sequenced_task_runner.h" |
| #include "base/single_thread_task_runner.h" |
| #include "media/base/audio_parameters.h" |
| #include "media/base/media_export.h" |
| @@ -65,20 +66,21 @@ class MEDIA_EXPORT AudioDebugFileWriter { |
| class AudioFileWriter; |
| // Deleter for AudioFileWriter. |
| - struct OnThreadDeleter { |
| + struct OnSequenceDeleter { |
| public: |
| - OnThreadDeleter(); |
| - OnThreadDeleter(const OnThreadDeleter& other); |
| - OnThreadDeleter(scoped_refptr<base::SingleThreadTaskRunner> task_runner); |
| - ~OnThreadDeleter(); |
| + OnSequenceDeleter(); |
| + OnSequenceDeleter(OnSequenceDeleter&& other); |
| + OnSequenceDeleter& operator=(const OnSequenceDeleter&); |
|
gab
2017/05/18 22:11:21
Actually you want the matching move assignment ope
Sébastien Marchand
2017/05/18 22:47:25
Ha! Time for me to re learn C++11 and the move sem
|
| + OnSequenceDeleter(scoped_refptr<base::SequencedTaskRunner> task_runner); |
| + ~OnSequenceDeleter(); |
| void operator()(AudioFileWriter* ptr) const; |
| private: |
| - scoped_refptr<base::SingleThreadTaskRunner> task_runner_; |
| + scoped_refptr<base::SequencedTaskRunner> task_runner_; |
| }; |
| using AudioFileWriterUniquePtr = |
| - std::unique_ptr<AudioFileWriter, OnThreadDeleter>; |
| + std::unique_ptr<AudioFileWriter, OnSequenceDeleter>; |
| AudioFileWriterUniquePtr file_writer_; |
| base::SequenceChecker client_sequence_checker_; |