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

Unified Diff: components/scheduler/child/web_scheduler_impl.cc

Issue 1223163006: Implement PostDelayedTaskAt for guaranteed timer ordering (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebased. Created 5 years, 5 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: components/scheduler/child/web_scheduler_impl.cc
diff --git a/components/scheduler/child/web_scheduler_impl.cc b/components/scheduler/child/web_scheduler_impl.cc
index 0a918759dfb2f718cbd6144d79fee9cf20da39fd..88e56ce1cc96cbf9666fe95dc8cd1c479f811a50 100644
--- a/components/scheduler/child/web_scheduler_impl.cc
+++ b/components/scheduler/child/web_scheduler_impl.cc
@@ -15,7 +15,7 @@ WebSchedulerImpl::WebSchedulerImpl(
ChildScheduler* child_scheduler,
scoped_refptr<SingleThreadIdleTaskRunner> idle_task_runner,
scoped_refptr<base::SingleThreadTaskRunner> loading_task_runner,
- scoped_refptr<base::SingleThreadTaskRunner> timer_task_runner)
+ scoped_refptr<TaskQueue> timer_task_runner)
: child_scheduler_(child_scheduler),
idle_task_runner_(idle_task_runner),
loading_task_runner_(loading_task_runner),
@@ -107,4 +107,18 @@ void WebSchedulerImpl::postTimerTask(
base::TimeDelta::FromMilliseconds(delayMs));
}
+void WebSchedulerImpl::postTimerTaskAt(
+ const blink::WebTraceLocation& web_location,
+ blink::WebThread::Task* task,
+ double monotonicTime) {
+ DCHECK(timer_task_runner_);
+ scoped_ptr<blink::WebThread::Task> scoped_task(task);
+ tracked_objects::Location location(web_location.functionName(),
+ web_location.fileName(), -1, nullptr);
+ timer_task_runner_->PostDelayedTaskAt(
+ location,
+ base::Bind(&WebSchedulerImpl::runTask, base::Passed(&scoped_task)),
+ base::TimeTicks() + base::TimeDelta::FromSecondsD(monotonicTime));
+}
+
} // namespace scheduler

Powered by Google App Engine
This is Rietveld 408576698