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 f6dd2bcf4ec36063c21d5fa380864d51db59eef8..c4d8884c12897d8d8c70c704175067a494a17a9a 100644
--- a/cc/scheduler/scheduler.h
+++ b/cc/scheduler/scheduler.h
@@ -18,6 +18,7 @@
 #include "cc/scheduler/draw_result.h"
 #include "cc/scheduler/scheduler_settings.h"
 #include "cc/scheduler/scheduler_state_machine.h"
+#include "ui/gfx/frame_time.h"

[Sami, 2014/08/19 18:45:04]

Not sure why you needed to add this include here?

[mithro-old, 2014/08/21 17:39:18]

Removed.

 
 namespace base {
 namespace debug {
@@ -154,7 +155,7 @@ class CC_EXPORT Scheduler {
   class CC_EXPORT SyntheticBeginFrameSource : public TimeSourceClient {
    public:
     SyntheticBeginFrameSource(Scheduler* scheduler,
-                              base::SingleThreadTaskRunner* task_runner);
+                              scoped_refptr<DelayBasedTimeSource> time_source);
     virtual ~SyntheticBeginFrameSource();
 
     // Updates the phase and frequency of the timer.
@@ -186,6 +187,8 @@ class CC_EXPORT Scheduler {
       int layer_tree_host_id,
       const scoped_refptr<base::SingleThreadTaskRunner>& impl_task_runner);
 
+  virtual base::TimeTicks Now() const;

[Sami, 2014/08/19 18:45:04]

bikeshed: I'd prefer a more descriptive name like CurrentFrameTime() or
something to that effect.

[mithro-old, 2014/08/21 17:39:18]

I'm worried it will be to easy to confuse CurrentFrameTime with the
"CurrentFrameTimeToRenderFor".

+
   const SchedulerSettings settings_;
   SchedulerClient* client_;
   int layer_tree_host_id_;