Fire visibilityChange event on out-of-process iframes.

third_party/WebKit/Source/core/page/Page.cpp

 /*
  * Copyright (C) 2006, 2007, 2008, 2009, 2010, 2011, 2012, 2013 Apple Inc. All Rights Reserved.
  * Copyright (C) 2008 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  * This library is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 340 matching lines @@
 {
     static const double waitingTimeBeforeCompressingString = 10;
 
     if (m_visibilityState == visibilityState)
         return;
     m_visibilityState = visibilityState;
 
     if (!isInitialState)
         notifyPageVisibilityChanged();
 
-    if (!isInitialState && m_mainFrame && m_mainFrame->isLocalFrame())
-        deprecatedLocalMainFrame()->didChangeVisibilityState();
+    if (!isInitialState && m_mainFrame)
+        m_mainFrame->didChangeVisibilityState();

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

How does this not crash in an OOPIF renderer?

[kenrb, 2016/05/24 20:33:20]

Never mind, I just looked at this a bit closer. This ends up walking the entire
frame tree, whether the main frame is local or not. This is probably fine.

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

Right, Daniel moved that block from LocalFrame to Frame already.

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

As an aside, related to a discussion we had on this earlier, it doesn't look
like didChangeVisibilityState() triggers RemoteFrame::visibilityChanged(), which
sends the IPC to the browser process. That is a different condition.

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

Acknowledged.

 
     // Compress CompressibleStrings when 10 seconds have passed since the page
     // went to background.
     if (m_visibilityState == PageVisibilityStateHidden) {
         if (!m_timerForCompressStrings.isActive())
             m_timerForCompressStrings.startOneShot(waitingTimeBeforeCompressingString, BLINK_FROM_HERE);
     } else if (m_timerForCompressStrings.isActive()) {
         m_timerForCompressStrings.stop();
     }
 }
@@ skipping 203 matching lines @@
 {
 }
 
 Page::PageClients::~PageClients()
 {
 }
 
 template class CORE_TEMPLATE_EXPORT Supplement<Page>;
 
 } // namespace blink