Track pages preventing suppression in WebProcessProxy using RefCounter
[WebKit-https.git] / Source / WebKit2 / UIProcess / WebPageProxy.cpp
index 0f525c8..d09e47c 100644 (file)
@@ -388,6 +388,7 @@ WebPageProxy::WebPageProxy(PageClient& pageClient, WebProcessProxy& process, uin
 
     updateViewState();
     updateActivityToken();
+    updateProccessSuppressionState();
     
 #if HAVE(OUT_OF_PROCESS_LAYER_HOSTING)
     m_layerHostingMode = m_viewState & ViewState::IsInWindow ? m_pageClient.viewLayerHostingMode() : LayerHostingMode::OutOfProcess;
@@ -576,6 +577,7 @@ void WebPageProxy::reattachToWebProcess()
 
     updateViewState();
     updateActivityToken();
+    updateProccessSuppressionState();
 
 #if ENABLE(INSPECTOR)
     m_inspector = WebInspectorProxy::create(this);
@@ -659,11 +661,6 @@ void WebPageProxy::initializeWebPage()
 #endif
 }
 
-bool WebPageProxy::isProcessSuppressible() const
-{
-    return (m_viewState & ViewState::IsVisuallyIdle) && m_preferences->pageVisibilityBasedProcessSuppressionEnabled();
-}
-
 void WebPageProxy::close()
 {
     if (m_isClosed)
@@ -1245,9 +1242,7 @@ void WebPageProxy::dispatchViewStateChange()
 
     // This must happen after the SetViewState message is sent, to ensure the page visibility event can fire.
     updateActivityToken();
-
-    if (changed & ViewState::IsVisuallyIdle)
-        m_process->pageSuppressibilityChanged(this);
+    updateProccessSuppressionState();
 
     // If we've started the responsiveness timer as part of telling the web process to update the backing store
     // state, it might not send back a reply (since it won't paint anything if the web page is hidden) so we
@@ -1281,6 +1276,16 @@ void WebPageProxy::updateActivityToken()
 #endif
 }
 
+void WebPageProxy::updateProccessSuppressionState()
+{
+#if PLATFORM(COCOA)
+    if ((m_viewState & ViewState::IsVisuallyIdle) && m_preferences->pageVisibilityBasedProcessSuppressionEnabled())
+        m_preventProcessSuppression = nullptr;
+    else if (!m_preventProcessSuppression)
+        m_preventProcessSuppression = m_process->processSuppressionCounter();
+#endif
+}
+
 void WebPageProxy::layerHostingModeDidChange()
 {
     if (!isValid())
@@ -2489,7 +2494,7 @@ void WebPageProxy::preferencesDidChange()
         inspector()->enableRemoteInspection();
 #endif
 
-    m_process->pagePreferencesChanged(this);
+    updateProccessSuppressionState();
 
     m_pageClient.preferencesDidChange();
 
@@ -4558,6 +4563,9 @@ void WebPageProxy::resetStateAfterProcessExited()
 #if PLATFORM(IOS)
     m_activityToken = nullptr;
 #endif
+#if PLATFORM(COCOA)
+    m_preventProcessSuppression = nullptr;
+#endif
 
     m_isValid = false;
     m_isPageSuspended = false;