Fix page zoom to be frame-centric for out-of-process frames.

chrome/browser/apps/guest_view/web_view_browsertest.cc

Index: chrome/browser/apps/guest_view/web_view_browsertest.cc
diff --git a/chrome/browser/apps/guest_view/web_view_browsertest.cc b/chrome/browser/apps/guest_view/web_view_browsertest.cc
index ba5e2f7971283c071d0d8b35ad0530c78dc97631..f49e07638996371a832030996b4d66fb5edd3937 100644
--- a/chrome/browser/apps/guest_view/web_view_browsertest.cc
+++ b/chrome/browser/apps/guest_view/web_view_browsertest.cc
@@ -2438,7 +2438,13 @@ class WebViewCaptureTest : public WebViewTest {
 INSTANTIATE_TEST_CASE_P(WebViewTests, WebViewCaptureTest, testing::Bool());
 
 IN_PROC_BROWSER_TEST_P(WebViewTest, Shim_TestZoomAPI) {
-  TestHelper("testZoomAPI", "web_view/shim", NO_TEST_SERVER);
+  // TODO(wjmaclean): This test is broken, but only when the
+  // --use-cross-process-frames-for-guests flag is specified. This needs to be
+  // fixed. See also comment in
+  // CrossProcessFrameConnector::OnFrameRectChanged().
+  // http://crbug.com/xxxxxx <-- Don't land without filing this bug.

[alexmos, 2016/04/27 23:39:51]

Bug numbers still needed here. :)  (Also the one in FrameView::setFrameRect and
CrossProcessFrameConnector::OnFrameRectChanged)

[wjmaclean, 2016/04/28 13:20:11]

I generally put these in just before landing, as I dislike filing bugs far in
advance for issues that don't exist until the CL lands. I haven't forgotten
these though. :-)

+  if (!GetParam())
+    TestHelper("testZoomAPI", "web_view/shim", NO_TEST_SERVER);
 }
 
 IN_PROC_BROWSER_TEST_P(WebViewTest, Shim_TestFindAPI) {