SendPageMessage() should not send to a WebView guest's embedder.

chrome/browser/apps/guest_view/web_view_browsertest.cc

 // Copyright 2013 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.
 
 #include <queue>
 #include <utility>
 
 #include "base/location.h"
 #include "base/macros.h"
 #include "base/path_service.h"
@@ skipping 2411 matching lines @@
   WebViewCaptureTest() {}
   ~WebViewCaptureTest() override {}
   void SetUp() override {
     EnablePixelOutput();
     WebViewTest::SetUp();
   }
 };
 INSTANTIATE_TEST_CASE_P(WebViewTests, WebViewCaptureTest, testing::Bool());
 
 IN_PROC_BROWSER_TEST_P(WebViewTest, Shim_TestZoomAPI) {
-  // 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/607978
-  if (!GetParam())
     TestHelper("testZoomAPI", "web_view/shim", NO_TEST_SERVER);

[nasko, 2016/05/04 21:58:58]

Need to fix indent after removal of the if statement.

[wjmaclean, 2016/05/05 15:40:56]

Ooops! :-)

Fixed.

 }
 
 IN_PROC_BROWSER_TEST_P(WebViewTest, Shim_TestFindAPI) {
   TestHelper("testFindAPI", "web_view/shim", NO_TEST_SERVER);
 }
 
 IN_PROC_BROWSER_TEST_P(WebViewTest, Shim_TestFindAPI_findupdate) {
   TestHelper("testFindAPI_findupdate", "web_view/shim", NO_TEST_SERVER);
 }
 
@@ skipping 648 matching lines @@
   // 4 tasks expected. The order is arbitrary.
   // Tab: about:blank,
   // Background Page: <webview> task manager test,
   // App: <webview> task manager test,
   // Webview: WebViewed test content.
   EXPECT_EQ(4U, task_manager.tasks().size());
   EXPECT_TRUE(HasExpectedGuestTask(task_manager, guest_contents));
 }
 
 #endif  // defined(ENABLE_TASK_MANAGER)