Replace base::Closure with OnceClosure in media/filters

This completes the conversion of media/filters/*.

Bug: 1007805
Change-Id: I86c90287a612b055e780eb175ef7f162d87f420f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2532903Reviewed-by: Chrome Cunningham <chcunningham@chromium.org>
Commit-Queue: David Bienvenu <davidbienvenu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#827275}

media/filters/audio_decoder_unittest.cc

@@ -301,12 +301,12 @@ class AudioDecoderTest
     decoded_audio_.push_back(std::move(buffer));
   }
 
-  void DecodeFinished(const base::Closure& quit_closure, Status status) {
+  void DecodeFinished(base::OnceClosure quit_closure, Status status) {
     EXPECT_TRUE(pending_decode_);
     EXPECT_FALSE(pending_reset_);
     pending_decode_ = false;
     last_decode_status_ = std::move(status);
-    quit_closure.Run();
+    std::move(quit_closure).Run();
   }
 
   void ResetFinished() {

media/filters/chunk_demuxer_unittest.cc

@@ -4388,10 +4388,10 @@ TEST_F(ChunkDemuxerTest,
 }
 
 namespace {
-void QuitLoop(base::Closure quit_closure,
+void QuitLoop(base::OnceClosure quit_closure,
               DemuxerStream::Type type,
               const std::vector<DemuxerStream*>& streams) {
-  quit_closure.Run();
+  std::move(quit_closure).Run();
 }
 
 void DisableAndEnableDemuxerTracks(

media/filters/demuxer_perftest.cc

@@ -45,10 +45,10 @@ class DemuxerHostImpl : public media::DemuxerHost {
   DISALLOW_COPY_AND_ASSIGN(DemuxerHostImpl);
 };
 
-static void QuitLoopWithStatus(base::Closure quit_cb,
+static void QuitLoopWithStatus(base::OnceClosure quit_cb,
                                media::PipelineStatus status) {
   CHECK_EQ(status, media::PIPELINE_OK);
-  quit_cb.Run();
+  std::move(quit_cb).Run();
 }
 
 static void OnEncryptedMediaInitData(EmeInitDataType init_data_type,
@@ -81,7 +81,7 @@ class StreamReader {
 
  private:
   void OnReadDone(scoped_refptr<base::SingleThreadTaskRunner> task_runner,
-                  const base::Closure& quit_when_idle_closure,
+                  base::OnceClosure quit_when_idle_closure,
                   bool* end_of_stream,
                   base::TimeDelta* timestamp,
                   media::DemuxerStream::Status status,
@@ -139,7 +139,7 @@ bool StreamReader::IsDone() {
 
 void StreamReader::OnReadDone(
     scoped_refptr<base::SingleThreadTaskRunner> task_runner,
-    const base::Closure& quit_when_idle_closure,
+    base::OnceClosure quit_when_idle_closure,
     bool* end_of_stream,
     base::TimeDelta* timestamp,
     media::DemuxerStream::Status status,
@@ -148,7 +148,7 @@ void StreamReader::OnReadDone(
   CHECK(buffer);
   *end_of_stream = buffer->end_of_stream();
   *timestamp = *end_of_stream ? media::kNoTimestamp : buffer->timestamp();
-  task_runner->PostTask(FROM_HERE, quit_when_idle_closure);
+  task_runner->PostTask(FROM_HERE, std::move(quit_when_idle_closure));
 }
 
 int StreamReader::GetNextStreamIndexToRead() {

media/filters/ffmpeg_demuxer_unittest.cc

@@ -1653,10 +1653,10 @@ TEST_F(FFmpegDemuxerTest, Seek_FallbackToDisabledAudioStream) {
 }
 
 namespace {
-void QuitLoop(base::Closure quit_closure,
+void QuitLoop(base::OnceClosure quit_closure,
               DemuxerStream::Type type,
               const std::vector<DemuxerStream*>& streams) {
-  quit_closure.Run();
+  std::move(quit_closure).Run();
 }
 
 void DisableAndEnableDemuxerTracks(

media/filters/fuchsia/fuchsia_video_decoder.cc

@@ -134,7 +134,7 @@ class OutputMailbox {
       // The mailbox is referenced by a VideoFrame. It will be deleted  as soon
       // as the frame is destroyed.
       DCHECK(reuse_callback_);
-      reuse_callback_ = base::Closure();
+      reuse_callback_ = base::OnceClosure();
     } else {
       delete this;
     }

media/filters/fuchsia/fuchsia_video_decoder_unittest.cc

@@ -212,8 +212,8 @@ class TestRasterContextProvider
   TestRasterContextProvider(TestRasterContextProvider&) = delete;
   TestRasterContextProvider& operator=(TestRasterContextProvider&) = delete;
 
-  void SetOnDestroyedClosure(base::Closure on_destroyed) {
-    on_destroyed_ = on_destroyed;
+  void SetOnDestroyedClosure(base::OnceClosure on_destroyed) {
+    on_destroyed_ = std::move(on_destroyed);
   }
 
   // viz::RasterContextProvider implementation;
@@ -277,7 +277,7 @@ class TestRasterContextProvider
   TestSharedImageInterface shared_image_interface_;
   viz::TestContextSupport gpu_context_support_;
 
-  base::Closure on_destroyed_;
+  base::OnceClosure on_destroyed_;
 };
 
 }  // namespace

media/filters/vpx_video_decoder_fuzzertest.cc

@@ -30,15 +30,15 @@ struct Env {
   base::test::SingleThreadTaskEnvironment task_environment;
 };
 
-void OnDecodeComplete(const base::Closure& quit_closure, media::Status status) {
-  quit_closure.Run();
+void OnDecodeComplete(base::OnceClosure quit_closure, media::Status status) {
+  std::move(quit_closure).Run();
 }
 
-void OnInitDone(const base::Closure& quit_closure,
+void OnInitDone(base::OnceClosure quit_closure,
                 bool* success_dest,
                 media::Status status) {
   *success_dest = status.is_ok();
-  quit_closure.Run();
+  std::move(quit_closure).Run();
 }
 
 void OnOutputComplete(scoped_refptr<media::VideoFrame> frame) {}