[chromium] Compositor visibility setting must be updated even if not actively compositing
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 28 Mar 2012 18:24:01 +0000 (18:24 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 28 Mar 2012 18:24:01 +0000 (18:24 +0000)
https://bugs.webkit.org/show_bug.cgi?id=82406

Patch by James Robinson <jamesr@chromium.org> on 2012-03-28
Reviewed by Adrienne Walker.

Propagate the visibility bit to the WebLayerTreeView even when compositing is inactive.

* src/WebViewImpl.cpp:
(WebKit::WebViewImpl::setVisibilityState):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@112417 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit/chromium/ChangeLog
Source/WebKit/chromium/src/WebViewImpl.cpp

index e211c9b..d18de65 100644 (file)
@@ -1,3 +1,15 @@
+2012-03-28  James Robinson  <jamesr@chromium.org>
+
+        [chromium] Compositor visibility setting must be updated even if not actively compositing
+        https://bugs.webkit.org/show_bug.cgi?id=82406
+
+        Reviewed by Adrienne Walker.
+
+        Propagate the visibility bit to the WebLayerTreeView even when compositing is inactive.
+
+        * src/WebViewImpl.cpp:
+        (WebKit::WebViewImpl::setVisibilityState):
+
 2012-03-28  Fady Samuel  <fsamuel@chromium.org>
 
         [Chromium] Enable Viewport define by default
index c0c4396..1b3ab58 100644 (file)
@@ -3480,9 +3480,9 @@ void WebViewImpl::setVisibilityState(WebPageVisibilityState visibilityState,
 #endif
 
 #if USE(ACCELERATED_COMPOSITING)
-    if (isAcceleratedCompositingActive()) {
+    if (!m_layerTreeView.isNull()) {
         bool visible = visibilityState == WebPageVisibilityStateVisible;
-        if (!visible)
+        if (!visible && isAcceleratedCompositingActive())
             m_nonCompositedContentHost->protectVisibleTileTextures();
         m_layerTreeView.setVisible(visible);
     }