Move compositor thread input handling logic into content

cc/trees/layer_tree_host_impl.cc

Index: cc/trees/layer_tree_host_impl.cc
diff --git a/cc/trees/layer_tree_host_impl.cc b/cc/trees/layer_tree_host_impl.cc
index f8230e79175d46bfa92ffaab7b8143c01889e6dd..718095d46ba65f14c81c1b3ad14eb75a505416b9 100644
--- a/cc/trees/layer_tree_host_impl.cc
+++ b/cc/trees/layer_tree_host_impl.cc
@@ -279,6 +279,8 @@ bool LayerTreeHostImpl::CanDraw() {
 
 void LayerTreeHostImpl::Animate(base::TimeTicks monotonic_time,
                                 base::Time wall_clock_time) {
+  if (input_handler_client_)
+    input_handler_client_->Animate(monotonic_time);
   AnimatePageScale(monotonic_time);
   AnimateLayers(monotonic_time, wall_clock_time);
   AnimateScrollbars(monotonic_time);
@@ -722,6 +724,11 @@ bool LayerTreeHostImpl::CalculateRenderPasses(FrameData* frame) {
   return draw_frame;
 }
 
+void LayerTreeHostImpl::MainThreadHasStoppedFlinging() {
+  if (input_handler_client_)
+    input_handler_client_->MainThreadHasStoppedFlinging();
+}
+
 void LayerTreeHostImpl::UpdateBackgroundAnimateTicking(
     bool should_background_tick) {
   bool enabled = should_background_tick &&
@@ -1468,6 +1475,11 @@ bool LayerTreeHostImpl::EnsureRenderSurfaceLayerList() {
   return !active_tree_->RenderSurfaceLayerList().empty();
 }
 
+void LayerTreeHostImpl::BindToClient(InputHandlerClient* client) {

[danakj, 2013/05/06 16:33:39]

If this is going to take ownership of the client*, why not pass in a scoped_ptr?

But the InputHandlerWrapper owns the InputHandlerProxy that is passed in here.
This looks like a double-free?

Is the input_handler_client_ supposed to be a WeakPtr instead of a scoped_ptr?

+  DCHECK(input_handler_client_ == NULL);
+  input_handler_client_.reset(client);
+}
+
 InputHandler::ScrollStatus LayerTreeHostImpl::ScrollBegin(
     gfx::Point viewport_point, InputHandler::ScrollInputType type) {
   TRACE_EVENT0("cc", "LayerTreeHostImpl::ScrollBegin");