Fire visibilityChange event on out-of-process iframes.

third_party/WebKit/Source/web/WebFrameWidgetImpl.cpp

 /*
  * Copyright (C) 2014 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 1053 matching lines @@
         m_layerTreeView->attachCompositorAnimationTimeline(compositorTimeline->animationTimeline());
 
 }
 
 void WebFrameWidgetImpl::detachCompositorAnimationTimeline(CompositorAnimationTimeline* compositorTimeline)
 {
     if (m_layerTreeView)
         m_layerTreeView->detachCompositorAnimationTimeline(compositorTimeline->animationTimeline());
 }
 
-void WebFrameWidgetImpl::setVisibilityState(WebPageVisibilityState visibilityState, bool isInitialState)
+void WebFrameWidgetImpl::setVisibilityState(WebPageVisibilityState visibilityState)
 {
-    if (!page())
-        return;
-
-    // FIXME: This is not correct, since Show and Hide messages for a frame's Widget do not necessarily
-    // correspond to Page visibility, but is necessary until we properly sort out OOPIF visibility.
-    page()->setVisibilityState(static_cast<PageVisibilityState>(visibilityState), isInitialState);
-
-    m_localRoot->frame()->frameScheduler()->setPageVisible(visibilityState == WebPageVisibilityStateVisible);
-
-    if (m_layerTreeView) {
-        bool visible = visibilityState == WebPageVisibilityStateVisible;
-        m_layerTreeView->setVisible(visible);
-    }
+    if (m_layerTreeView)

[kenrb, 2016/05/24 20:25:18]

Why is it okay to remove the call to WebFrameScheduler::setPageVisible()?

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

This is already done in WebViewSchedulerImpl::setPageVisible, which iterates
over all LocalFrames in the tree and updates its visibility.
WebViewSchedulerImpl::setPageVisible is called when setting page visibility.

[kenrb, 2016/05/25 20:29:14]

Acknowledged. This is fine, although I notice that there is a
WebFrameScheduler::setFrameVisible() that currently has no callers. I assume
that is a work in progress by the scheduler people.

[lfg, 2016/05/31 19:54:01]

That's right. However, if we do that here, the correct way to do it is to
iterate over all LocalFrames in this local root.

+        m_layerTreeView->setVisible(visibilityState == WebPageVisibilityStateVisible);
 }
 
 HitTestResult WebFrameWidgetImpl::hitTestResultForRootFramePos(const IntPoint& posInRootFrame)
 {
     IntPoint docPoint(m_localRoot->frame()->view()->rootFrameToContents(posInRootFrame));
     HitTestResult result = m_localRoot->frame()->eventHandler().hitTestResultAtPoint(docPoint, HitTestRequest::ReadOnly | HitTestRequest::Active);
     result.setToShadowHostIfInUserAgentShadowRoot();
     return result;
 }
 
 } // namespace blink