Refactor browser process coordinate transformation methods

content/browser/frame_host/render_widget_host_view_child_frame.h

Index: content/browser/frame_host/render_widget_host_view_child_frame.h
diff --git a/content/browser/frame_host/render_widget_host_view_child_frame.h b/content/browser/frame_host/render_widget_host_view_child_frame.h
index 74d535b84ed5d843170379a5752689c2a05c72d4..6bfc7732ab75357f70f61e46ac42b5a4de373e46 100644
--- a/content/browser/frame_host/render_widget_host_view_child_frame.h
+++ b/content/browser/frame_host/render_widget_host_view_child_frame.h
@@ -138,6 +138,12 @@ class CONTENT_EXPORT RenderWidgetHostViewChildFrame
   void ProcessGestureEvent(const blink::WebGestureEvent& event,
                            const ui::LatencyInfo& latency) override;
   gfx::Point TransformPointToRootCoordSpace(const gfx::Point& point) override;
+  gfx::Point TransformPointToLocalCoordSpace(
+      const gfx::Point& point,
+      const cc::SurfaceId& original_surface) override;
+  gfx::Point TransformPointToCoordSpaceForView(
+      const gfx::Point& point,
+      RenderWidgetHostViewBase* target_view) override;
 
 #if defined(OS_MACOSX)
   // RenderWidgetHostView implementation.
@@ -179,6 +185,11 @@ class CONTENT_EXPORT RenderWidgetHostViewChildFrame
     return frame_connector_;
   }
 
+  // Returns the view into which this view is directly embedded. This can
+  // return nullptr when this view's associated child frame is not connected
+  // to the frame tree.

[nasko, 2016/07/27 22:52:13]

Is this the case only when we have started navigation in a new frame but we
haven't committed it yet?

[kenrb, 2016/07/28 16:24:37]

I think there are a couple of cases, but yes the View and FrameConnector aren't
linked until CommitPending(). I believe there are destruction paths that can
result in this also.

+  RenderWidgetHostViewBase* GetParentView();
+
   void RegisterSurfaceNamespaceId();
   void UnregisterSurfaceNamespaceId();