A better idle time estimator

components/scheduler/renderer/renderer_scheduler_impl.h

Index: components/scheduler/renderer/renderer_scheduler_impl.h
diff --git a/components/scheduler/renderer/renderer_scheduler_impl.h b/components/scheduler/renderer/renderer_scheduler_impl.h
index dbe1c003cd671a0d892564032a765833d50019a0..d2e11a6226b7fb246cd89b54c0d571758c432224 100644
--- a/components/scheduler/renderer/renderer_scheduler_impl.h
+++ b/components/scheduler/renderer/renderer_scheduler_impl.h
@@ -11,6 +11,7 @@
 #include "components/scheduler/child/idle_helper.h"
 #include "components/scheduler/child/scheduler_helper.h"
 #include "components/scheduler/renderer/deadline_task_runner.h"
+#include "components/scheduler/renderer/idle_time_estimator.h"
 #include "components/scheduler/renderer/renderer_scheduler.h"
 #include "components/scheduler/renderer/task_cost_estimator.h"
 #include "components/scheduler/renderer/user_model.h"
@@ -75,6 +76,7 @@ class SCHEDULER_EXPORT RendererSchedulerImpl
   SchedulerHelper* GetSchedulerHelperForTesting();
   TaskCostEstimator* GetLoadingTaskCostEstimatorForTesting();
   TaskCostEstimator* GetTimerTaskCostEstimatorForTesting();
+  IdleTimeEstimator* GetIdleTimeEstimatorForTesting();
   base::TimeTicks CurrentIdleTaskDeadlineForTesting() const;
 
  private:
@@ -142,6 +144,10 @@ class SCHEDULER_EXPORT RendererSchedulerImpl
   // other tasks during the initial page load.
   static const int kRailsInitialLoadingPrioritizationMillis = 1000;
 
+  // The ammount of time in milliseconds we have to respond to user input as

[Sami, 2015/10/09 02:32:19]

typo: amount

[alex clarke (OOO till 29th), 2015/10/15 13:24:01]

Done.

+  // defined by RAILS.
+  static const int kRailsResponseTimeMillis = 50;
+
   // For the purposes of deciding whether or not it's safe to turn timers and
   // loading tasks on only in idle periods, we regard the system as being as
   // being "idle period" starved if there hasn't been an idle period in the last
@@ -234,12 +240,13 @@ class SCHEDULER_EXPORT RendererSchedulerImpl
 
     TaskCostEstimator loading_task_cost_estimator;
     TaskCostEstimator timer_task_cost_estimator;
-    cc::RollingTimeDeltaHistory short_idle_period_duration;
+    IdleTimeEstimator idle_time_estimator;
     UseCase current_use_case;
     Policy current_policy;
     base::TimeTicks current_policy_expiration_time;
     base::TimeTicks estimated_next_frame_begin;
-    base::TimeDelta expected_short_idle_period_duration;
+    base::TimeDelta compositor_frame_interval;
+    base::TimeDelta expected_idle_duration;
     int timer_queue_suspend_count;  // TIMER_TASK_QUEUE suspended if non-zero.
     int navigation_task_expected_count;
     bool renderer_hidden;