Hook up DidReceiveInputEvent to the blink scheduler

content/renderer/input/input_handler_wrapper.cc

 // Copyright 2013 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.
 
 #include "content/renderer/input/input_handler_wrapper.h"
 
 #include "base/message_loop/message_loop_proxy.h"
 #include "content/renderer/input/input_event_filter.h"
 #include "content/renderer/input/input_handler_manager.h"
+#include "content/renderer/scheduler/renderer_scheduler.h"
 #include "third_party/WebKit/public/platform/Platform.h"
 
 namespace content {
 
 InputHandlerWrapper::InputHandlerWrapper(
     InputHandlerManager* input_handler_manager,
     int routing_id,
     const scoped_refptr<base::MessageLoopProxy>& main_loop,
     const base::WeakPtr<cc::InputHandler>& input_handler,
-    const base::WeakPtr<RenderViewImpl>& render_view_impl)
+    const base::WeakPtr<RenderViewImpl>& render_view_impl,
+    RendererScheduler* renderer_scheduler)
     : input_handler_manager_(input_handler_manager),
       routing_id_(routing_id),
       input_handler_proxy_(input_handler.get(), this),
       main_loop_(main_loop),
-      render_view_impl_(render_view_impl) {
+      render_view_impl_(render_view_impl),
+      renderer_scheduler_(renderer_scheduler) {
   DCHECK(input_handler);
 }
 
 InputHandlerWrapper::~InputHandlerWrapper() {
 }
 
 void InputHandlerWrapper::TransferActiveWheelFlingAnimation(
     const blink::WebActiveWheelFlingParameters& params) {
   main_loop_->PostTask(
       FROM_HERE,
@@ skipping 16 matching lines @@
 
 void InputHandlerWrapper::DidOverscroll(const DidOverscrollParams& params) {
   input_handler_manager_->DidOverscroll(routing_id_, params);
 }
 
 void InputHandlerWrapper::DidStopFlinging() {
   input_handler_manager_->DidStopFlinging(routing_id_);
 }
 
 void InputHandlerWrapper::DidReceiveInputEvent() {
-  // TODO(skyostil): Hook this up into the renderer scheduler.
+  if (renderer_scheduler_) {

[Sami, 2014/11/06 00:31:20]

Instead of having renderer_scheduler_ as a member of this class, could we put it
into InputHandlerManager and just call
input_handler_manager_->DidReceiveInputEvent() here? Seems more consistent with
the other functions here.

[alex clarke (OOO till 29th), 2014/11/06 19:21:45]

Done.

+    renderer_scheduler_->DidReceiveInputEvent();
+  }
+}
+
+void InputHandlerWrapper::DidAnimate() {
+  if (renderer_scheduler_) {

[Sami, 2014/11/06 00:31:20]

The scheduler should never be null, right?

[alex clarke (OOO till 29th), 2014/11/06 19:21:45]

Acknowledged.

+    renderer_scheduler_->DidAnimate();
+  }
 }
 
 }  // namespace content