Replace base::Callback with base::OnceCallback in base::PendingTask

third_party/WebKit/Source/platform/scheduler/base/task_queue_manager.cc

Index: third_party/WebKit/Source/platform/scheduler/base/task_queue_manager.cc
diff --git a/third_party/WebKit/Source/platform/scheduler/base/task_queue_manager.cc b/third_party/WebKit/Source/platform/scheduler/base/task_queue_manager.cc
index 7def64ac93093b72ead7c15b75b697fc0de71f4e..5085d5de0b8a89bbcc9414053347647857598974 100644
--- a/third_party/WebKit/Source/platform/scheduler/base/task_queue_manager.cc
+++ b/third_party/WebKit/Source/platform/scheduler/base/task_queue_manager.cc
@@ -319,8 +319,11 @@ TaskQueueManager::ProcessTaskResult TaskQueueManager::ProcessTaskFromWorkQueue(
     // arbitrarily delayed so the additional delay should not be a problem.
     // TODO(skyostil): Figure out a way to not forget which task queue the
     // task is associated with. See http://crbug.com/522843.
+    // TODO(tzik): Remove base::UnsafeConvertOnceClosureToRepeating once
+    // TaskRunners have migrated to OnceClosure.
     delegate_->PostNonNestableTask(pending_task.posted_from,
-                                   std::move(pending_task.task));
+                                   base::UnsafeConvertOnceClosureToRepeating(

[dcheng, 2016/10/14 02:35:02]

Can we just define UnsafeConvert directly here instead of defining it in base? I
think it's just used here, unless I'm missing something.

[tzik, 2016/10/14 05:07:43]

Done.

+                                       std::move(pending_task.task)));
     return ProcessTaskResult::DEFERRED;
   }
 
@@ -341,7 +344,7 @@ TaskQueueManager::ProcessTaskResult TaskQueueManager::ProcessTaskFromWorkQueue(
   internal::TaskQueueImpl* prev_executing_task_queue =
       currently_executing_task_queue_;
   currently_executing_task_queue_ = queue;
-  task_annotator_.RunTask("TaskQueueManager::PostTask", pending_task);
+  task_annotator_.RunTask("TaskQueueManager::PostTask", &pending_task);
   // Detect if the TaskQueueManager just got deleted.  If this happens we must
   // not access any member variables after this point.
   if (protect->HasOneRef())