Enable WebViewTest.NestedGuestContainerBounds for --site-per-process

content/browser/browser_plugin/browser_plugin_guest.cc

Index: content/browser/browser_plugin/browser_plugin_guest.cc
diff --git a/content/browser/browser_plugin/browser_plugin_guest.cc b/content/browser/browser_plugin/browser_plugin_guest.cc
index 4114d6535131c6b5b97f4c115020c1b58ea88c4e..8f26ef0690081c2eabc1add1f64474f37874be78 100644
--- a/content/browser/browser_plugin/browser_plugin_guest.cc
+++ b/content/browser/browser_plugin/browser_plugin_guest.cc
@@ -486,6 +486,14 @@ gfx::Point BrowserPluginGuest::GetCoordinatesInEmbedderWebContents(
   point +=
       owner_rwhv->TransformPointToRootCoordSpace(guest_window_rect_.origin())
           .OffsetFromOrigin();
+  if (embedder_web_contents()->GetBrowserPluginGuest()) {
+    // |point| is currently with respect to the top-most view (outermost
+    // WebContents). We should subtract a displacement to find the point with
+    // resepct to embedder's WebContents.
+    point -= owner_rwhv->TransformPointToRootCoordSpace(gfx::Point())
+                 .OffsetFromOrigin();

[lfg, 2017/05/17 18:03:24]

Is this plus the transformation above the same as transforming the point to the
parent's coord space? There are functions to do that in
CrossProcessFrameConnector, but they aren't exposed in RenderWidgetHostView.

Is that something we should consider? It may not be worth it, since this is only
used for browser plugin.

WDYT?

[EhsanK, 2017/05/17 18:22:49]

Yes exactly.
Well we are inside content/ and they are exposed in RWHVBase so that should not
be a problem. The one I need is TransformPointToCoordSpaceForView.
The problem is the dependency on the implementation of
TransformPointToLocalCoordSpace or TransformPointToCoordSpaceForView for
RenderWidgetHostViewGuest. The RWHVG does not implement either so IMO it might
not be worth the effort (given that it is hopefully going away soon).

I could try implementing those if you think it is worth it.

+  }
+
   return point;
 }