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

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

Issue 1424053002: Adds a flag to support "Virtual Time" to the blink scheduler (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Another try Created 5 years, 2 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/scheduler_helper.cc
diff --git a/components/scheduler/child/scheduler_helper.cc b/components/scheduler/child/scheduler_helper.cc
index e0d683e7222dcec667edfd342e3a5fa5cf83c08a..1e9ce01c636b09c167052460075bfe2dca43ced8 100644
--- a/components/scheduler/child/scheduler_helper.cc
+++ b/components/scheduler/child/scheduler_helper.cc
@@ -8,12 +8,12 @@
#include "base/trace_event/trace_event.h"
#include "base/trace_event/trace_event_argument.h"
#include "components/scheduler/base/task_queue_impl.h"
-#include "components/scheduler/child/scheduler_task_runner_delegate.h"
+#include "components/scheduler/child/scheduler_tqm_delegate.h"
namespace scheduler {
SchedulerHelper::SchedulerHelper(
- scoped_refptr<SchedulerTaskRunnerDelegate> main_task_runner,
+ scoped_refptr<SchedulerTqmDelegate> main_task_runner,
const char* tracing_category,
const char* disabled_by_default_tracing_category,
const char* disabled_by_default_verbose_tracing_category)
@@ -34,7 +34,6 @@ SchedulerHelper::SchedulerHelper(
.SetShouldNotifyObservers(false))),
default_task_runner_(NewTaskQueue(TaskQueue::Spec("default_tq")
.SetShouldMonitorQuiescence(true))),
- time_source_(new base::DefaultTickClock),
observer_(nullptr),
tracing_category_(tracing_category),
disabled_by_default_tracing_category_(
@@ -79,12 +78,6 @@ scoped_refptr<TaskQueue> SchedulerHelper::ControlAfterWakeUpTaskRunner() {
return control_after_wakeup_task_runner_;
}
-void SchedulerHelper::SetTimeSourceForTesting(
- scoped_ptr<base::TickClock> time_source) {
- CheckOnValidThread();
- time_source_ = time_source.Pass();
-}
-
void SchedulerHelper::SetWorkBatchSizeForTesting(size_t work_batch_size) {
CheckOnValidThread();
DCHECK(task_queue_manager_.get());
@@ -96,8 +89,8 @@ TaskQueueManager* SchedulerHelper::GetTaskQueueManagerForTesting() {
return task_queue_manager_.get();
}
-base::TimeTicks SchedulerHelper::Now() const {
- return time_source_->NowTicks();
+base::TickClock* SchedulerHelper::tick_clock() const {
+ return task_queue_manager_->tick_clock();
}
base::TimeTicks SchedulerHelper::NextPendingDelayedTaskRunTime() const {

Powered by Google App Engine
This is Rietveld 408576698