Fixing WebViewVisibilityTests for OOPIF-<webview>.

content/browser/frame_host/cross_process_frame_connector.cc

 // Copyright 2014 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 "content/browser/frame_host/cross_process_frame_connector.h"
 
 #include "cc/surfaces/surface.h"
 #include "cc/surfaces/surface_manager.h"
 #include "content/browser/compositor/surface_utils.h"
 #include "content/browser/frame_host/frame_tree.h"
@@ skipping 215 matching lines @@
 void CrossProcessFrameConnector::OnFrameRectChanged(
     const gfx::Rect& frame_rect) {
   if (!frame_rect.size().IsEmpty())
     SetSize(frame_rect);
 }
 
 void CrossProcessFrameConnector::OnVisibilityChanged(bool visible) {
   if (!view_)
     return;
 
+  // For <webview>, the WebContents should be notified of the change. The

[lazyboy, 2016/01/16 00:46:48]

Avoid using "<webview>". Use "inner WebContents" instead.

[EhsanK, 2016/01/20 00:00:03]

Done.

+  // WebContents will inform the RWH itself.
+  if (frame_proxy_in_parent_renderer_->frame_tree_node()
+          ->render_manager()
+          ->ForInnerDelegate()) {
+    RenderWidgetHostImpl::From(view_->GetRenderWidgetHost())
+        ->delegate()
+        ->ForwardVisibilityChange(visible);
+    return;
+  }
+
   if (visible)
     view_->Show();
   else
     view_->Hide();
 }
 
 void CrossProcessFrameConnector::SetDeviceScaleFactor(float scale_factor) {
   device_scale_factor_ = scale_factor;
   // The RenderWidgetHost is null in unit tests.
   if (view_ && view_->GetRenderWidgetHost()) {
@@ skipping 18 matching lines @@
   // in the case of nested WebContents.
   while (top_host->frame_tree_node()->render_manager()->ForInnerDelegate()) {
     top_host = top_host->frame_tree_node()->render_manager()->
         GetOuterDelegateNode()->frame_tree()->root()->current_frame_host();
   }
 
   return static_cast<RenderWidgetHostViewBase*>(top_host->GetView());
 }
 
 }  // namespace content