Implement Mac Pop-up Dictionary for OOPIF.

content/browser/renderer_host/render_widget_host_input_event_router.h

Index: content/browser/renderer_host/render_widget_host_input_event_router.h
diff --git a/content/browser/renderer_host/render_widget_host_input_event_router.h b/content/browser/renderer_host/render_widget_host_input_event_router.h
index 83736c4973dbd2b435f5d09a9041ae9db12e900e..d281e0aa2258d1c6c0415e68ab1661295944436d 100644
--- a/content/browser/renderer_host/render_widget_host_input_event_router.h
+++ b/content/browser/renderer_host/render_widget_host_input_event_router.h
@@ -38,6 +38,7 @@ class LatencyInfo;
 
 namespace content {
 
+class RenderWidgetHostImpl;

[Charlie Reis, 2016/09/02 21:57:47]

Stale?

[kenrb, 2016/09/06 18:14:48]

This declares the Impl, so I don't think it's stale. (?)

[Charlie Reis, 2016/09/06 21:58:26]

Ah yes, ignore this one.  (I must have been overeager when retyping my
comments.)  :)

 class RenderWidgetHostViewBase;
 
 // Class owned by WebContentsImpl for the purpose of directing input events
@@ -79,6 +80,17 @@ class CONTENT_EXPORT RenderWidgetHostInputEventRouter
 
   void OnHittestData(const FrameHostMsg_HittestData_Params& params);
 
+#if defined(OS_MACOSX)

[kenrb, 2016/09/06 18:14:48]

Is the #ifdef necessary here? I don't see anything wrong with exposing this as a
method for all platforms, even if it is only currently used on one.

[EhsanK, 2016/09/08 17:10:39]

Acknowledged.

+  // Returns the RenderWidgetHostImpl inside the |root_view| at |point| where
+  // |point| is with respect to |root_view|'s coordinates. If a RWHI is found,
+  // the value of |transformed_point| is the coordinate of the point with
+  // respect to the RWHI's coordinates.
+  RenderWidgetHostImpl* GetRenderWidgetHostAtPoint(
+      RenderWidgetHostViewBase* root_view,
+      const gfx::Point& point,
+      gfx::Point* transformed_point);
+#endif
+
  private:
   struct HittestData {
     bool ignored_for_hittest;