Fixing and enhancing OrderedSimpleTaskRunner to allow 100% deterministic tests.

cc/scheduler/scheduler.h

Index: cc/scheduler/scheduler.h
diff --git a/cc/scheduler/scheduler.h b/cc/scheduler/scheduler.h
index ce3fed783bf8de1012ee2aa40795227f5769b63e..b6fc979ff6db708db0e76c19c48651ef0be477ea 100644
--- a/cc/scheduler/scheduler.h
+++ b/cc/scheduler/scheduler.h
@@ -154,7 +154,7 @@ class CC_EXPORT Scheduler {
   class CC_EXPORT SyntheticBeginFrameSource : public TimeSourceClient {
    public:
     SyntheticBeginFrameSource(Scheduler* scheduler,
-                              base::SingleThreadTaskRunner* task_runner);
+                              scoped_refptr<DelayBasedTimeSource> time_source);

[Sami, 2014/08/29 13:13:51]

Could this be a scoped_ptr instead? Looks like this class could take ownership
of the time source.

[mithro-old, 2014/09/01 06:19:46]

DelayBasedTimeSource is ref counted, so have to use a scoped_refptr here. I
don't want to change DelayBasedTimeSource in this CL, but definite candidate for
fixes in the future.

     virtual ~SyntheticBeginFrameSource();
 
     // Updates the phase and frequency of the timer.
@@ -185,6 +185,8 @@ class CC_EXPORT Scheduler {
             int layer_tree_host_id,
             const scoped_refptr<base::SingleThreadTaskRunner>& task_runner);
 
+  virtual base::TimeTicks Now() const;
+
   const SchedulerSettings settings_;
   SchedulerClient* client_;
   int layer_tree_host_id_;