Delay Input.dispatchKeyEvent response until after key event ack.

content/browser/devtools/protocol/input_handler.h

 // Copyright 2014 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_BROWSER_DEVTOOLS_PROTOCOL_INPUT_HANDLER_H_
 #define CONTENT_BROWSER_DEVTOOLS_PROTOCOL_INPUT_HANDLER_H_
 
 #include "base/macros.h"
 #include "base/memory/weak_ptr.h"
 #include "content/browser/devtools/protocol/devtools_protocol_dispatcher.h"
 #include "content/browser/renderer_host/input/synthetic_gesture.h"
 #include "content/common/input/synthetic_smooth_scroll_gesture_params.h"
+#include "content/public/browser/render_widget_host.h"
 #include "ui/gfx/geometry/size_f.h"
 
 namespace cc {
 class CompositorFrameMetadata;
 }
 
 namespace content {
 
 class RenderWidgetHostImpl;
 
 namespace devtools {
 namespace input {
 
-class InputHandler {
+class InputHandler : public RenderWidgetHost::InputEventObserver {
  public:
   typedef DevToolsProtocolClient::Response Response;
 
   InputHandler();
-  virtual ~InputHandler();
+  ~InputHandler() override;
+
+  // InputEventObserver
+  void OnInputEvent(const blink::WebInputEvent& event) override;

[dgozman, 2016/11/21 19:14:39]

We can probably move these to private.

[samuong, 2016/11/21 22:28:43]

Done.

+  void OnInputEventAck(const blink::WebInputEvent& event) override;
 
   void SetRenderWidgetHost(RenderWidgetHostImpl* host);
   void SetClient(std::unique_ptr<Client> client);
   void OnSwapCompositorFrame(const cc::CompositorFrameMetadata& frame_metadata);
 
-  Response DispatchKeyEvent(const std::string& type,
+  Response DispatchKeyEvent(DevToolsCommandId command_id,
+                            const std::string& type,
                             const int* modifiers,
                             const double* timestamp,
                             const std::string* text,
                             const std::string* unmodified_text,
                             const std::string* key_identifier,
                             const std::string* code,
                             const std::string* key,
                             const int* windows_virtual_key_code,
                             const int* native_virtual_key_code,
                             const bool* auto_repeat,
                             const bool* is_keypad,
                             const bool* is_system_key);
 
-  Response DispatchMouseEvent(const std::string& type,
+  Response DispatchMouseEvent(DevToolsCommandId command_id,
+                              const std::string& type,
                               int x,
                               int y,
                               const int* modifiers,
                               const double* timestamp,
                               const std::string* button,
                               const int* click_count);
 
   Response EmulateTouchFromMouseEvent(const std::string& type,
                                       int x,
                                       int y,
@@ skipping 32 matching lines @@
                                 const int* tap_count,
                                 const std::string* gesture_source_type);
 
   Response DispatchTouchEvent(
       const std::string& type,
       const std::vector<std::unique_ptr<base::DictionaryValue>>& touch_points,
       const int* modifiers,
       const double* timestamp);
 
  private:
+  void SendDispatchKeyEventResponse(DevToolsCommandId command_id);
+  void SendDispatchMouseEventResponse(DevToolsCommandId command_id);
+
   void SendSynthesizePinchGestureResponse(DevToolsCommandId command_id,
                                           SyntheticGesture::Result result);
 
   void SendSynthesizeScrollGestureResponse(DevToolsCommandId command_id,
                                            SyntheticGesture::Result result);
 
   void SendSynthesizeTapGestureResponse(DevToolsCommandId command_id,
                                         bool send_success,
                                         SyntheticGesture::Result result);
 
   void SynthesizeRepeatingScroll(
       SyntheticSmoothScrollGestureParams gesture_params,
       int repeat_count,
       base::TimeDelta repeat_delay,
       std::string interaction_marker_name,
       DevToolsCommandId command_id);
 
   void OnScrollFinished(SyntheticSmoothScrollGestureParams gesture_params,
                         int repeat_count,
                         base::TimeDelta repeat_delay,
                         std::string interaction_marker_name,
                         DevToolsCommandId command_id,
                         SyntheticGesture::Result result);
 
   RenderWidgetHostImpl* host_;
   std::unique_ptr<Client> client_;
+  // DevToolsCommandIds for calls to Input.dispatchKey/MouseEvent that have been
+  // sent to the renderer, but that we haven't yet received an ack for.
+  bool input_queued_;
+  std::deque<DevToolsCommandId> pending_key_command_ids_;
+  std::deque<DevToolsCommandId> pending_mouse_command_ids_;
   float page_scale_factor_;
   gfx::SizeF scrollable_viewport_size_;
   base::WeakPtrFactory<InputHandler> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(InputHandler);
 };
 
 }  // namespace input
 }  // namespace devtools
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_DEVTOOLS_PROTOCOL_INPUT_HANDLER_H_