[scheduler] Implement time-based cpu throttling.

third_party/WebKit/Source/platform/scheduler/renderer/renderer_scheduler_impl.h

Index: third_party/WebKit/Source/platform/scheduler/renderer/renderer_scheduler_impl.h
diff --git a/third_party/WebKit/Source/platform/scheduler/renderer/renderer_scheduler_impl.h b/third_party/WebKit/Source/platform/scheduler/renderer/renderer_scheduler_impl.h
index bedca1e75f6e8b15ad1ba825dcc6827874296e83..c2dc23060ea5ec78785ef9e0ca1ef7cc421b0fb8 100644
--- a/third_party/WebKit/Source/platform/scheduler/renderer/renderer_scheduler_impl.h
+++ b/third_party/WebKit/Source/platform/scheduler/renderer/renderer_scheduler_impl.h
@@ -17,7 +17,7 @@
 #include "platform/scheduler/renderer/idle_time_estimator.h"
 #include "platform/scheduler/renderer/render_widget_signals.h"
 #include "platform/scheduler/renderer/task_cost_estimator.h"
-#include "platform/scheduler/renderer/throttling_helper.h"
+#include "platform/scheduler/renderer/task_queue_throttler.h"

[caseq, 2016/09/22 00:52:34]

Is this necessary? If so, please remove TaskQueueThrottler forward declaration
below.

[altimin, 2016/09/22 13:45:58]

Done.

 #include "platform/scheduler/renderer/user_model.h"
 #include "platform/scheduler/renderer/web_view_scheduler_impl.h"
 #include "public/platform/scheduler/renderer/renderer_scheduler.h"
@@ -34,7 +34,7 @@ namespace scheduler {
 class AutoAdvancingVirtualTimeDomain;
 class RenderWidgetSchedulingState;
 class WebViewSchedulerImpl;
-class ThrottlingHelper;
+class TaskQueueThrottler;
 
 class BLINK_PLATFORM_EXPORT RendererSchedulerImpl
     : public RendererScheduler,
@@ -131,7 +131,9 @@ class BLINK_PLATFORM_EXPORT RendererSchedulerImpl
                                    const base::PendingTask& task) override;
 
   // TaskTimeObserver implementation:
-  void ReportTaskTime(double start_time, double end_time) override;
+  void ReportTaskTime(TaskQueue* task_queue,
+                      double start_time,
+                      double end_time) override;
 
   // QueueingTimeEstimator::Client implementation:
   void OnQueueingTimeForWindowEstimated(base::TimeDelta queueing_time) override;
@@ -151,6 +153,9 @@ class BLINK_PLATFORM_EXPORT RendererSchedulerImpl
   void AddTaskTimeObserver(TaskTimeObserver* task_time_observer);
   void RemoveTaskTimeObserver(TaskTimeObserver* task_time_observer);
 
+  // Snapshots this RendererScheduler for tracing.
+  void CreateTraceEventObjectSnapshot() const;

[caseq, 2016/09/22 00:52:34]

Why is this public?

[altimin, 2016/09/22 13:45:58]

It's public to scheduler/ because other components in scheduler can request a
trace snapshot.

+
   // Test helpers.
   SchedulerHelper* GetSchedulerHelperForTesting();
   TaskCostEstimator* GetLoadingTaskCostEstimatorForTesting();
@@ -170,7 +175,9 @@ class BLINK_PLATFORM_EXPORT RendererSchedulerImpl
 
   AutoAdvancingVirtualTimeDomain* GetVirtualTimeDomain();
 
-  ThrottlingHelper* throttling_helper() { return throttling_helper_.get(); }
+  TaskQueueThrottler* task_queue_throttler() const {
+    return task_queue_throttler_.get();
+  }
 
  private:
   friend class RendererSchedulerImplTest;
@@ -250,6 +257,7 @@ class BLINK_PLATFORM_EXPORT RendererSchedulerImpl
       base::TimeTicks optional_now) const;
   std::unique_ptr<base::trace_event::ConvertableToTraceFormat> AsValueLocked(
       base::TimeTicks optional_now) const;
+  void CreateTraceEventObjectSnapshotLocked() const;
 
   static bool ShouldPrioritizeInputEvent(const WebInputEvent& web_input_event);
 
@@ -338,7 +346,7 @@ class BLINK_PLATFORM_EXPORT RendererSchedulerImpl
 
   SchedulerHelper helper_;
   IdleHelper idle_helper_;
-  std::unique_ptr<ThrottlingHelper> throttling_helper_;
+  std::unique_ptr<TaskQueueThrottler> task_queue_throttler_;
   RenderWidgetSignals render_widget_scheduler_signals_;
 
   const scoped_refptr<TaskQueue> control_task_runner_;