Move compositor thread input handling logic into content

content/renderer/gpu/input_handler_client_impl_client.h

+// Copyright (c) 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.
+
+#ifndef CONTENT_RENDERER_GPU_INPUT_HANDLER_CLIENT_IMPL_CLIENT_H_
+#define CONTENT_RENDERER_GPU_INPUT_HANDLER_CLIENT_IMPL_CLIENT_H_
+
+namespace WebKit {
+class WebGestureCurve;
+struct WebActiveWheelFlingParameters;
+struct WebFloatPoint;
+struct WebSize;
+}
+
+namespace content {
+
+class InputHandlerClientImplClient {
+public:

[danakj, 2013/05/01 19:20:43]

space before p

+    // Callbacks invoked from the compositor thread.
+    virtual void willShutdown() = 0;
+
+    // Exactly one of the following two callbacks will be invoked after every
+    // call to WebCompositorInputHandler::handleInputEvent():

[danakj, 2013/05/01 19:20:43]

WebCompositorInputHandler::handle* -> InputHandlerClientImpl::Handle*?

few more comments referring to WebCompositor* in here too

+
+    // Called when the WebCompositorInputHandler handled the input event and no
+    // further processing is required.
+    virtual void didHandleInputEvent() = 0;

[danakj, 2013/05/01 19:20:43]

Can all these function names change to chromium style?

+
+    // Called when the WebCompositorInputHandler did not handle the input event.
+    // If sendToWidget is true, the input event should be forwarded to the
+    // WebWidget associated with this compositor for further processing.
+    virtual void didNotHandleInputEvent(bool sendToWidget) = 0;

[danakj, 2013/05/01 19:20:43]

argument is webkit style (in the comment too)

+
+    // Transfers an active wheel fling animation initiated by a previously
+    // handled input event out to the client.
+    // FIXME: Make pure virtual once implementation lands on Chromium side.

[danakj, 2013/05/01 19:20:43]

TODO (or possibly this is done now?)

+    virtual void transferActiveWheelFlingAnimation(
+        const WebKit::WebActiveWheelFlingParameters&) = 0;
+
+    // Creates a new fling animation curve instance for device |deviceSource|
+    // with |velocity| and already scrolled |cumulativeScroll| pixels.
+    virtual WebKit::WebGestureCurve* createFlingAnimationCurve(
+        int deviceSource,

[danakj, 2013/05/01 19:20:43]

webkit style (in the comment too)

+        const WebKit::WebFloatPoint& velocity,
+        const WebKit::WebSize& cumulativeScroll) = 0;

[danakj, 2013/05/01 19:20:43]

ditto

+
+protected:

[danakj, 2013/05/01 19:20:43]

space before p

+    virtual ~InputHandlerClientImplClient() { }

[danakj, 2013/05/01 19:20:43]

no space in {}

+};
+
+} // namespace content

[danakj, 2013/05/01 19:20:43]

2 spaces before //

+
+#endif // CONTENT_RENDERER_GPU_INPUT_HANDLER_CLIENT_IMPL_CLIENT_H_

[danakj, 2013/05/01 19:20:43]

ditto