Fix WebView compositor input handling

content/renderer/gpu/input_handler_manager_client.h

+// Copyright (c) 2012 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_MANAGER_CLIENT_H_
+#define CONTENT_RENDERER_GPU_INPUT_HANDLER_MANAGER_CLIENT_H_
+
+#include "base/basictypes.h"
+#include "base/callback_forward.h"
+
+namespace WebKit {
+class WebInputEvent;
+}
+
+namespace content {
+
+class InputHandlerManagerClient {
+ public:
+  virtual ~InputHandlerManagerClient() {}
+
+  // The Handler allows the client to forward input events in such a way that
+  // callbacks to DidHandleInputEvent or DidNotHandlerInputEvent are made with
+  // the appropriate response.
+  // The Manager will supply a valid |handler| when bound to the client,
+  // valid until the manager shuts down, when it supplies an empty |handler|.
+  // The client should only makes calls to |handler| on the compositing thread.
+  typedef base::Callback<void(int /*routing_id*/,

[jamesr, 2013/06/04 06:51:11]

i think using a return value here instead of Did(Not)HandleInputEvent would make
things a lot clearer

[jdduke (slow), 2013/06/04 15:46:23]

Yeah, just rebased with the new patch that does just that :)  

It largely renders this class redundant, but I believe mkosiba still needs the
"DidAddInputHandler" callbacks for compositor wiring.

+                              const WebKit::WebInputEvent*)> Handler;
+
+  // Called from the main thread.
+  virtual void SetHandler(const Handler& handler) = 0;
+  virtual void DidAddInputHandler(int routing_id) = 0;
+  virtual void DidRemoveInputHandler(int routing_id) = 0;
+
+  // Called from the compositing thread.
+  virtual void DidHandleInputEvent() = 0;
+  virtual void DidNotHandleInputEvent(bool send_to_widget) = 0;
+
+ protected:
+  InputHandlerManagerClient() {}
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(InputHandlerManagerClient);
+};
+
+}  // namespace content
+
+#endif  // CONTENT_COMMON_GPU_INPUT_HANDLER_MANAGER_CLIENT_H_