Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(52)

Unified Diff: remoting/client/plugin/pepper_main_thread_task_runner.cc

Issue 2122543002: Replace Closure in TaskRunner::PostTask with OneShotCallback (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@07_oneshot
Patch Set: fix Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: remoting/client/plugin/pepper_main_thread_task_runner.cc
diff --git a/remoting/client/plugin/pepper_main_thread_task_runner.cc b/remoting/client/plugin/pepper_main_thread_task_runner.cc
index 9dade1744e7e22f7d956e3f8adfcff54b155ba11..f789821f0f2a62aafb1b299635cd5b5e0652c244 100644
--- a/remoting/client/plugin/pepper_main_thread_task_runner.cc
+++ b/remoting/client/plugin/pepper_main_thread_task_runner.cc
@@ -14,19 +14,22 @@ PepperMainThreadTaskRunner::PepperMainThreadTaskRunner()
bool PepperMainThreadTaskRunner::PostDelayedTask(
const tracked_objects::Location& from_here,
- const base::Closure& task,
+ base::OnceClosure task,
base::TimeDelta delay) {
- core_->CallOnMainThread(delay.InMillisecondsRoundedUp(),
- callback_factory_.NewCallback(
- &PepperMainThreadTaskRunner::RunTask, task));
+ core_->CallOnMainThread(
+ delay.InMillisecondsRoundedUp(),
+ callback_factory_.NewCallback(
+ &PepperMainThreadTaskRunner::RunTask,
+ base::Bind([](base::OnceClosure cb) { std::move(cb).Run(); },
+ base::Passed(&task))));
return true;
}
bool PepperMainThreadTaskRunner::PostNonNestableDelayedTask(
const tracked_objects::Location& from_here,
- const base::Closure& task,
+ base::OnceClosure task,
base::TimeDelta delay) {
- return PostDelayedTask(from_here, task, delay);
+ return PostDelayedTask(from_here, std::move(task), delay);
}
bool PepperMainThreadTaskRunner::RunsTasksOnCurrentThread() const {

Powered by Google App Engine
This is Rietveld 408576698