Fire visibilityChange event on out-of-process iframes.

content/browser/frame_host/render_frame_host_manager.cc

Index: content/browser/frame_host/render_frame_host_manager.cc
diff --git a/content/browser/frame_host/render_frame_host_manager.cc b/content/browser/frame_host/render_frame_host_manager.cc
index 3e7cbe11ca6bf9114ed2f3e894e93a830f9c56ec..183b4fab5ed38156fe28d65ced4b9b6a36096bb3 100644
--- a/content/browser/frame_host/render_frame_host_manager.cc
+++ b/content/browser/frame_host/render_frame_host_manager.cc
@@ -1768,7 +1768,8 @@ int RenderFrameHostManager::CreateRenderFrameProxy(SiteInstance* instance) {
   if (!render_view_host) {
     CHECK(frame_tree_node_->IsMainFrame());
     render_view_host = frame_tree_node_->frame_tree()->CreateRenderViewHost(
-        instance, MSG_ROUTING_NONE, MSG_ROUTING_NONE, true, true);
+        instance, MSG_ROUTING_NONE, MSG_ROUTING_NONE, true,
+        delegate_->IsHidden());

[Charlie Reis, 2016/06/06 23:15:09]

This scares me a bit, but maybe it's ok.

1) We're doing this in CreateRenderFrameProxy.  Is it ok to tell the RVH it's
visible even if it has no visible frames in it?

2) RFHMDelegate::IsHidden() could be a bit of a broken API, and seems to date
back before we moved RVHM to RFHM.  It returns WebContentsImpl::IsHidden.  Is
that the right value to use in all frames?

[lfg, 2016/06/08 20:00:22]

That's right, the page visibility is determined by the WebContentsImpl, and is
passed to the RenderViewImpl on creation. This change just passes it through
during the creation of the RVH, so it's forwarded to the renderer and the
correct visibility is set when creating the RenderViewImpl (on the subframe
renderer).

[Charlie Reis, 2016/06/08 21:39:07]

Acknowledged.

   }
 
   RenderFrameProxyHost* proxy = GetRenderFrameProxyHost(instance);