Pass Callback to TaskRunner by value and consume it on invocation (1)

content/browser/dom_storage/dom_storage_task_runner.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/dom_storage/dom_storage_task_runner.h"
 
 #include <utility>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
@@ skipping 13 matching lines @@
 
 DOMStorageWorkerPoolTaskRunner::~DOMStorageWorkerPoolTaskRunner() = default;
 
 bool DOMStorageWorkerPoolTaskRunner::RunsTasksOnCurrentThread() const {
   // It is valid for an implementation to always return true.
   return true;
 }
 
 bool DOMStorageWorkerPoolTaskRunner::PostDelayedTask(
     const tracked_objects::Location& from_here,
-    const base::Closure& task,
+    base::Closure task,
     base::TimeDelta delay) {
-  return primary_sequence_->PostDelayedTask(from_here, task, delay);
+  return primary_sequence_->PostDelayedTask(from_here, std::move(task), delay);
 }
 
 bool DOMStorageWorkerPoolTaskRunner::PostShutdownBlockingTask(
     const tracked_objects::Location& from_here,
     SequenceID sequence_id,
-    const base::Closure& task) {
-  return GetSequencedTaskRunner(sequence_id)->PostTask(from_here, task);
+    base::Closure task) {
+  return GetSequencedTaskRunner(sequence_id)
+      ->PostTask(from_here, std::move(task));
 }
 
 void DOMStorageWorkerPoolTaskRunner::AssertIsRunningOnPrimarySequence() const {
   DCHECK(primary_sequence_->RunsTasksOnCurrentThread());
 }
 
 void DOMStorageWorkerPoolTaskRunner::AssertIsRunningOnCommitSequence() const {
   DCHECK(commit_sequence_->RunsTasksOnCurrentThread());
 }
 
@@ skipping 12 matching lines @@
     : task_runner_(std::move(task_runner)) {}
 
 MockDOMStorageTaskRunner::~MockDOMStorageTaskRunner() = default;
 
 bool MockDOMStorageTaskRunner::RunsTasksOnCurrentThread() const {
   return task_runner_->RunsTasksOnCurrentThread();
 }
 
 bool MockDOMStorageTaskRunner::PostDelayedTask(
     const tracked_objects::Location& from_here,
-    const base::Closure& task,
+    base::Closure task,
     base::TimeDelta delay) {
-  return task_runner_->PostTask(from_here, task);
+  return task_runner_->PostTask(from_here, std::move(task));
 }
 
 bool MockDOMStorageTaskRunner::PostShutdownBlockingTask(
     const tracked_objects::Location& from_here,
     SequenceID sequence_id,
-    const base::Closure& task) {
-  return task_runner_->PostTask(from_here, task);
+    base::Closure task) {
+  return task_runner_->PostTask(from_here, std::move(task));
 }
 
 void MockDOMStorageTaskRunner::AssertIsRunningOnPrimarySequence() const {
   DCHECK(RunsTasksOnCurrentThread());
 }
 
 void MockDOMStorageTaskRunner::AssertIsRunningOnCommitSequence() const {
   DCHECK(RunsTasksOnCurrentThread());
 }
 
 scoped_refptr<base::SequencedTaskRunner>
 MockDOMStorageTaskRunner::GetSequencedTaskRunner(SequenceID sequence_id) {
   return task_runner_;
 }
 
 }  // namespace content