Scheduler/child/TimeSource could be replaced with base/time/DefaultTickClock.

components/scheduler/child/scheduler_helper.h

 // Copyright 2015 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.
 
 #ifndef COMPONENTS_SCHEDULER_CHILD_SCHEDULER_HELPER_H_
 #define COMPONENTS_SCHEDULER_CHILD_SCHEDULER_HELPER_H_
 
+#include "base/time/default_tick_clock.h"

[Sami, 2015/06/04 13:51:47]

Could you forward declare this instead?

[Jimmy Jo, 2015/06/05 00:45:09]

Done.

 #include "components/scheduler/child/prioritizing_task_queue_selector.h"
 #include "components/scheduler/child/task_queue_manager.h"
-#include "components/scheduler/child/time_source.h"
 #include "components/scheduler/scheduler_export.h"
 
 namespace scheduler {
 
 class NestableSingleThreadTaskRunner;
 
 // Common scheduler functionality for default tasks.
 class SCHEDULER_EXPORT SchedulerHelper {
  public:
   // NOTE |total_task_queue_count| must be >= TASK_QUEUE_COUNT.
@@ skipping 65 matching lines @@
   bool IsQueueEnabled(size_t queue_index) const;
   void SetPumpPolicy(size_t queue_index,
                      TaskQueueManager::PumpPolicy pump_policy);
   void SetWakeupPolicy(size_t queue_index,
                        TaskQueueManager::WakeupPolicy wakeup_policy);
   void PumpQueue(size_t queue_index);
   uint64 GetQuiescenceMonitoredTaskQueueMask() const;
   uint64 GetAndClearTaskWasRunOnQueueBitmap();
 
   // Test helpers.
-  void SetTimeSourceForTesting(scoped_ptr<TimeSource> time_source);
+  void SetTimeSourceForTesting(scoped_ptr<base::DefaultTickClock> time_source);

[Sami, 2015/06/04 13:51:47]

I think this should be base::TickClock since we don't really care that
DefaultTickClock is involved.

[Jimmy Jo, 2015/06/05 00:45:09]

Done.

   void SetWorkBatchSizeForTesting(size_t work_batch_size);
   TaskQueueManager* GetTaskQueueManagerForTesting();
 
  private:
   friend class SchedulerHelperTest;
 
   base::ThreadChecker thread_checker_;
   scoped_ptr<PrioritizingTaskQueueSelector> task_queue_selector_;
   scoped_ptr<TaskQueueManager> task_queue_manager_;
 
   uint64 quiescence_monitored_task_queue_mask_;
 
   scoped_refptr<base::SingleThreadTaskRunner> control_task_runner_;
   scoped_refptr<base::SingleThreadTaskRunner> control_after_wakeup_task_runner_;
   scoped_refptr<base::SingleThreadTaskRunner> default_task_runner_;
 
-  scoped_ptr<TimeSource> time_source_;
+  scoped_ptr<base::DefaultTickClock> time_source_;

[Sami, 2015/06/04 13:51:47]

base::TickClock here too.

[Jimmy Jo, 2015/06/05 00:45:09]

Done.

 
   const char* tracing_category_;
   const char* disabled_by_default_tracing_category_;
 
   DISALLOW_COPY_AND_ASSIGN(SchedulerHelper);
 };
 
 }  // namespace scheduler
 
 #endif  // COMPONENTS_SCHEDULER_CHILD_SCHEDULER_HELPER_H_