Fire visibilityChange event on out-of-process iframes.

content/browser/web_contents/web_contents_impl.cc

Index: content/browser/web_contents/web_contents_impl.cc
diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index 5621f827b46d1f009ff3cb81d369fc9adbb12d02..f26f792778079393b891c7023c32b90d02d1e60a 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -1313,6 +1313,8 @@ void WebContentsImpl::WasShown() {
     }
   }
 
+  SendPageMessage(new PageMsg_WasShown(MSG_ROUTING_NONE));
+
   last_active_time_ = base::TimeTicks::Now();
 
   // The resize rect might have changed while this was inactive -- send the new
@@ -1342,6 +1344,8 @@ void WebContentsImpl::WasHidden() {
       if (view)
         view->Hide();
     }
+
+    SendPageMessage(new PageMsg_WasHidden(MSG_ROUTING_NONE));

[dcheng, 2016/05/24 20:38:23]

MSG_ROUTING_NONE? This looks weird. Either it should be a control message or we
should attach a real routing ID.

[lfg, 2016/05/24 23:33:50]

SendPageMessage() will attach the appropriate routing IDs (and we can't create
routed message classes without providing a routing ID).

   }
 
   FOR_EACH_OBSERVER(WebContentsObserver, observers_, WasHidden());