sync compositor: Merge input path with chrome

content/renderer/input/input_handler_manager_client.h

 // 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.
 
 #ifndef CONTENT_RENDERER_INPUT_INPUT_HANDLER_MANAGER_CLIENT_H_
 #define CONTENT_RENDERER_INPUT_INPUT_HANDLER_MANAGER_CLIENT_H_
 
 #include "base/callback.h"
 #include "base/callback_forward.h"
 #include "base/macros.h"
@@ skipping 26 matching lines @@
   // The client should only makes calls to |handler| on the compositor thread.
   typedef base::Callback<
       InputEventAckState(int /*routing_id*/,
                          const blink::WebInputEvent*,
                          ui::LatencyInfo* latency_info)> Handler;
 
   // Called from the main thread.
   virtual void SetBoundHandler(const Handler& handler) = 0;
 
   // Called from the compositor thread.
-  virtual void DidAddInputHandler(
-      int routing_id,
-      ui::SynchronousInputHandlerProxy* synchronous_handler) = 0;
+  virtual void DidAddInputHandler(int routing_id) = 0;
   virtual void DidRemoveInputHandler(int routing_id) = 0;
   virtual void DidOverscroll(int routing_id,
                              const DidOverscrollParams& params) = 0;
   virtual void DidStopFlinging(int routing_id) = 0;
   virtual void NonBlockingInputEventHandled(
       int routing_id,
       blink::WebInputEvent::Type type) = 0;
 
  protected:
   InputHandlerManagerClient() {}
 
  private:
   DISALLOW_COPY_AND_ASSIGN(InputHandlerManagerClient);
 };
 
+class CONTENT_EXPORT InputHandlerManagerSyncHandlerProxyClient {

[tdresser, 2016/02/29 16:42:49]

This name is hard to parse.

Isn't the second "Handler" redundant?
Would something like:

SyncInputHandlerManagerProxyClient 

mean the same thing?

[boliu, 2016/02/29 18:58:53]

Yeah I'm terribe with names. How about just SynchronousInputHandlerProxyClient?
To match SynchronousInputHandlerProxy

[tdresser, 2016/02/29 19:10:25]

SGTM.

[boliu, 2016/02/29 21:04:40]

Done.

+ public:
+  virtual ~InputHandlerManagerSyncHandlerProxyClient() {}
+
+  virtual void DidAddSynchronousHandlerProxy(
+      int routing_id,
+      ui::SynchronousInputHandlerProxy* synchronous_handler) = 0;
+  virtual void DidRemoveSynchronousHandlerProxy(int routing_id) = 0;
+
+ protected:
+  InputHandlerManagerSyncHandlerProxyClient() {}
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(InputHandlerManagerSyncHandlerProxyClient);
+};
+
 }  // namespace content
 
 #endif  // CONTENT_RENDERER_INPUT_INPUT_HANDLER_MANAGER_CLIENT_H_