Refactor EventHandler out of RenderWidgetHostViewAura

content/browser/renderer_host/render_widget_host_view_event_handler.h

Index: content/browser/renderer_host/render_widget_host_view_event_handler.h
diff --git a/content/browser/renderer_host/render_widget_host_view_event_handler.h b/content/browser/renderer_host/render_widget_host_view_event_handler.h
new file mode 100644
index 0000000000000000000000000000000000000000..0f5274a673fff76f0c3e6b2835312686ec48627c
--- /dev/null
+++ b/content/browser/renderer_host/render_widget_host_view_event_handler.h
@@ -0,0 +1,192 @@
+// Copyright 2016 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_RENDERER_HOST_RENDER_WIDGET_HOST_VIEW_EVENT_HANDLER_H_
+#define CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_VIEW_EVENT_HANDLER_H_
+
+#include <memory>
+
+#include "base/macros.h"
+#include "content/common/content_export.h"
+#include "ui/aura/window_tracker.h"
+#include "ui/events/event_handler.h"
+#include "ui/events/gestures/motion_event_aura.h"
+#include "ui/events/latency_info.h"
+
+namespace aura {
+
+namespace client {
+class ScopedTooltipDisabler;
+}  // namespace client
+
+class Window;
+
+}  // namespace aura
+
+namespace blink {
+
+class WebMouseEvent;
+class WebMouseWheelEvent;
+class WebTouchEvent;
+
+}  // namespace blink
+
+namespace content {
+#if defined(OS_WIN)
+struct ContextMenuParams;
+#endif  // defined(OS_WIN)
+class RenderViewHost;
+class RenderViewHostDelegateView;
+class RenderWidgetHostImpl;
+class RenderWidgetHostViewAura;
+
+class CONTENT_EXPORT RenderWidgetHostViewEventHandler
+    : public ui::EventHandler {
+ public:
+  RenderWidgetHostViewEventHandler(RenderWidgetHostImpl* host,
+                                   RenderWidgetHostViewAura* host_view);

[sadrul, 2016/09/13 15:39:46]

Use a delegate interface instead?

[jonross, 2016/09/15 16:39:59]

I've split this into a RWHVBase and a Delegate.

+  ~RenderWidgetHostViewEventHandler() override;
+
+  void SetPopupChildHostView(RenderWidgetHostViewAura* popup_child_host_view);
+  void TrackHost(aura::Window* reference_window);
+#if defined(OS_WIN)
+  void SetContextMenuParams(const ContextMenuParams& params);
+
+  // Updates the cursor clip region. Used for mouse locking.
+  void UpdateMouseLockRegion();
+#endif  // defined(OS_WIN)
+
+  bool accept_return_character() { return accept_return_character_; }
+  void disable_input_event_router_for_testing() {
+    disable_input_event_router_for_testing_ = true;
+  }
+  bool mouse_locked() { return mouse_locked_; }
+  const ui::MotionEventAura& pointer_state() const { return pointer_state_; }
+  void set_focus_on_mouse_down_or_key_event(
+      bool focus_on_mouse_down_or_key_event) {
+    set_focus_on_mouse_down_or_key_event_ = focus_on_mouse_down_or_key_event;
+  }
+  void set_window(aura::Window* window) { window_ = window; }
+
+  // RenderWidgetHostViewBase
+  bool LockMouse();
+  void UnlockMouse();
+
+  // ui::EventHandler:
+  void OnKeyEvent(ui::KeyEvent* event) override;
+  void OnMouseEvent(ui::MouseEvent* event) override;
+  void OnScrollEvent(ui::ScrollEvent* event) override;
+  void OnTouchEvent(ui::TouchEvent* event) override;
+  void OnGestureEvent(ui::GestureEvent* event) override;
+
+ private:
+  FRIEND_TEST_ALL_PREFIXES(InputMethodResultAuraTest,
+                           FinishImeCompositionSession);
+  // Returns true if the |event| passed in can be forwarded to the renderer.
+  bool CanRendererHandleEvent(const ui::MouseEvent* event,
+                              bool mouse_locked,
+                              bool selection_popup) const;
+
+  // Confirm existing composition text in the webpage and ask the input method
+  // to cancel its ongoing composition session.
+  void FinishImeCompositionSession();
+
+  // Forwards a mouse event to this view's parent window delegate.
+  void ForwardMouseEventToParent(ui::MouseEvent* event);
+
+  // Returns the RenderViewHostDelegateView instance for |host_|. Returns
+  // nullptr on failure.
+  RenderViewHostDelegateView* GetRenderViewHostDelegateView();
+
+  // Performs gesture handling needed for touch text selection. Sets event as
+  // handled if it should not be further processed.
+  void HandleGestureForTouchSelection(ui::GestureEvent* event);
+
+  // This method computes movementX/Y and keeps track of mouse location for
+  // mouse lock on all mouse move events.
+  void ModifyEventMovementAndCoords(blink::WebMouseEvent* event);
+
+  // Helper function to set keyboard focus to the main window.
+  void SetKeyboardFocus();
+
+  // Helper method to determine if, in mouse locked mode, the cursor should be
+  // moved to center.
+  bool ShouldMoveToCenter();
+
+  // Returns true when we can do SurfaceHitTesting for the event type.
+  bool ShouldRouteEvent(const ui::Event* event) const;
+
+  // RenderWidgetHostViewBase
+  void ProcessMouseEvent(const blink::WebMouseEvent& event,
+                         const ui::LatencyInfo& latency);
+  void ProcessMouseWheelEvent(const blink::WebMouseWheelEvent& event,
+                              const ui::LatencyInfo& latency);  // override;
+  void ProcessTouchEvent(const blink::WebTouchEvent& event,
+                         const ui::LatencyInfo& latency);
+
+  // Whether return characters should be passed on to the RenderWidgetHostImpl.
+  bool accept_return_character_;
+
+  // Allows tests to send gesture events for testing without first sending a
+  // corresponding touch sequence, as would be required by
+  // RenderWidgetHostInputEventRouter.
+  bool disable_input_event_router_for_testing_;
+
+  // While the mouse is locked, the cursor is hidden from the user. Mouse events
+  // are still generated. However, the position they report is the last known
+  // mouse position just as mouse lock was entered; the movement they report
+  // indicates what the change in position of the mouse would be had it not been
+  // locked.
+  bool mouse_locked_;
+
+  // Whether pinch-to-zoom should be enabled and pinch events forwarded to the
+  // renderer.
+  bool pinch_zoom_enabled_;
+
+  // This flag when set ensures that we send over a notification to blink that
+  // the current view has focus. Defaults to false.
+  bool set_focus_on_mouse_down_or_key_event_;
+
+  // Used to track the state of the window we're created from. Only used when
+  // created fullscreen.
+  std::unique_ptr<aura::WindowTracker> host_tracker_;
+
+  std::unique_ptr<aura::client::ScopedTooltipDisabler> tooltip_disabler_;
+
+  // Used to record the last position of the mouse.
+  // While the mouse is locked, they store the last known position just as mouse
+  // lock was entered.
+  // Relative to the upper-left corner of the view.
+  gfx::Point unlocked_mouse_position_;
+  // Relative to the upper-left corner of the screen.
+  gfx::Point unlocked_global_mouse_position_;
+  // Last cursor position relative to screen. Used to compute movementX/Y.
+  gfx::Point global_mouse_position_;
+  // In mouse locked mode, we synthetically move the mouse cursor to the center
+  // of the window when it reaches the window borders to avoid it going outside.
+  // This flag is used to differentiate between these synthetic mouse move
+  // events vs. normal mouse move events.
+  bool synthetic_move_sent_;
+  // Stores the current state of the active pointers targeting this
+  // object.
+  ui::MotionEventAura pointer_state_;
+
+#if defined(OS_WIN)
+  // Contains a copy of the last context menu request parameters. Only set when
+  // we receive a request to show the context menu on a long press.
+  std::unique_ptr<ContextMenuParams> last_context_menu_params_;
+#endif  // defined(OS_WIN)
+
+  // Not owned/
+  RenderWidgetHostImpl* const host_;
+  RenderWidgetHostViewAura* host_view_;
+  RenderWidgetHostViewAura* popup_child_host_view_;
+  aura::Window* window_;
+
+  DISALLOW_COPY_AND_ASSIGN(RenderWidgetHostViewEventHandler);
+};
+
+}  // namespace content
+
+#endif  // CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_VIEW_EVENT_HANDLER_H_