Unreviewed, rolling out r105906.
authorleviw@chromium.org <leviw@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 25 Jan 2012 21:06:51 +0000 (21:06 +0000)
committerleviw@chromium.org <leviw@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 25 Jan 2012 21:06:51 +0000 (21:06 +0000)
http://trac.webkit.org/changeset/105906
https://bugs.webkit.org/show_bug.cgi?id=77038

Breaks compositing/visibility/layer-visible-content.html and
compositing/visibility/visibility-image-layers-dynamic.html
(Requested by leviw|gardening on #webkit).

Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-01-25

Source/WebCore:

* rendering/RenderLayerBacking.cpp:
(WebCore::RenderLayerBacking::updateGraphicsLayerGeometry):

LayoutTests:

* compositing/visibility/layer-visible-content-expected.txt:
* compositing/visibility/visibility-image-layers-dynamic-expected.txt:
* platform/chromium/test_expectations.txt:

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

LayoutTests/ChangeLog
LayoutTests/compositing/visibility/layer-visible-content-expected.txt
LayoutTests/compositing/visibility/visibility-image-layers-dynamic-expected.txt
LayoutTests/platform/chromium/test_expectations.txt
Source/WebCore/ChangeLog
Source/WebCore/rendering/RenderLayerBacking.cpp

index 1c902958caf0e46de7a72ae0350db5935660f443..32bc9e35dfd7d188da895452e3a9abf01a05e258 100644 (file)
@@ -1,3 +1,17 @@
+2012-01-25  Sheriff Bot  <webkit.review.bot@gmail.com>
+
+        Unreviewed, rolling out r105906.
+        http://trac.webkit.org/changeset/105906
+        https://bugs.webkit.org/show_bug.cgi?id=77038
+
+        Breaks compositing/visibility/layer-visible-content.html and
+        compositing/visibility/visibility-image-layers-dynamic.html
+        (Requested by leviw|gardening on #webkit).
+
+        * compositing/visibility/layer-visible-content-expected.txt:
+        * compositing/visibility/visibility-image-layers-dynamic-expected.txt:
+        * platform/chromium/test_expectations.txt:
+
 2012-01-25  Hajime Morita  <morrita@google.com>
 
         <content> should create HTMLContentElement object
index b7e3fbda0ac309ec0d13d2107c6739a1230bacef..a5c18c4877079eeb0d238ed07d07d4d8ed7837a0 100644 (file)
@@ -3,10 +3,11 @@
   (children 1
     (GraphicsLayer
       (bounds 800.00 600.00)
-      (children 2
+      (children 1
         (GraphicsLayer
           (bounds 200.00 200.00)
           (drawsContent 1)
+          (contentsVisible 0)
           (children 1
             (GraphicsLayer
               (bounds 200.00 200.00)
             )
           )
         )
-        (GraphicsLayer
-          (bounds 800.00 15.00)
-          (opacity 0.00)
-          (drawsContent 1)
-        )
       )
     )
   )
index e7be785badc2f243bfaacfed94add23ea42929c8..a10ebd47b96ec69210e4484ea886457edd7392f1 100644 (file)
@@ -35,6 +35,7 @@ Initial
         (GraphicsLayer
           (position 14.00 314.00)
           (bounds 757.00 152.00)
+          (contentsVisible 0)
           (children 1
             (GraphicsLayer
               (position 20.00 20.00)
@@ -49,10 +50,10 @@ Initial
 After step 1
 
 (GraphicsLayer
-  (bounds 785.00 1286.00)
+  (bounds 785.00 1301.00)
   (children 1
     (GraphicsLayer
-      (bounds 785.00 1286.00)
+      (bounds 785.00 1301.00)
       (children 3
         (GraphicsLayer
           (position 14.00 6.00)
@@ -79,6 +80,7 @@ After step 1
         (GraphicsLayer
           (position 14.00 314.00)
           (bounds 757.00 152.00)
+          (contentsVisible 0)
           (children 1
             (GraphicsLayer
               (position 20.00 20.00)
@@ -93,10 +95,10 @@ After step 1
 After step 2
 
 (GraphicsLayer
-  (bounds 785.00 1935.00)
+  (bounds 785.00 1965.00)
   (children 1
     (GraphicsLayer
-      (bounds 785.00 1935.00)
+      (bounds 785.00 1965.00)
       (children 3
         (GraphicsLayer
           (position 14.00 6.00)
@@ -122,6 +124,7 @@ After step 2
         (GraphicsLayer
           (position 14.00 314.00)
           (bounds 757.00 152.00)
+          (contentsVisible 0)
           (children 1
             (GraphicsLayer
               (position 20.00 20.00)
index a7e5e450cdcc70ce8a4f08467c29dcf02e3f8b8f..2b4fa429f3f5f3ee74251da1f5459cb69b9022a5 100644 (file)
@@ -4004,6 +4004,8 @@ BUG_CARYCLARK MAC : fast/dom/blur-contenteditable.html = IMAGE
 // Need to implement file name logging.
 BUGWK76572 : http/tests/download = FAIL
 
+BUGWK76716 : compositing/visibility/layer-visible-content.html = IMAGE+TEXT
+
 BUGWK76967 : media/media-fragments/TC0030-TC0039.html = PASS TIMEOUT
 
 BUGABARTH : media/audio-garbage-collect.html = PASS TIMEOUT
index 40b8d29d2d3193501b9e072333f9a56ad19f3f98..aa6abb2779f3d48a1344b96234284d131be94154 100644 (file)
@@ -1,3 +1,16 @@
+2012-01-25  Sheriff Bot  <webkit.review.bot@gmail.com>
+
+        Unreviewed, rolling out r105906.
+        http://trac.webkit.org/changeset/105906
+        https://bugs.webkit.org/show_bug.cgi?id=77038
+
+        Breaks compositing/visibility/layer-visible-content.html and
+        compositing/visibility/visibility-image-layers-dynamic.html
+        (Requested by leviw|gardening on #webkit).
+
+        * rendering/RenderLayerBacking.cpp:
+        (WebCore::RenderLayerBacking::updateGraphicsLayerGeometry):
+
 2012-01-25  Mike Fenton  <mifenton@rim.com>
 
         Page Up and Page Down mappings are backwards on BlackBerry port.
index 7e9ed4ab05d1ba9857edb1100900a9c692e5e734..628185c358b91b04f983487b06276952ed63aef1 100644 (file)
@@ -403,11 +403,7 @@ void RenderLayerBacking::updateGraphicsLayerGeometry()
 #endif
     
     m_owningLayer->updateVisibilityStatus();
-
-    // m_graphicsLayer is the corresponding GraphicsLayer for this RenderLayer and its non-compositing
-    // descendants. So, the visibility flag for m_graphicsLayer should be true if there are any
-    // non-compositing visible layers.
-    m_graphicsLayer->setContentsVisible(m_owningLayer->hasVisibleContent() || m_owningLayer->hasVisibleDescendant());
+    m_graphicsLayer->setContentsVisible(m_owningLayer->hasVisibleContent());
     
     RenderStyle* style = renderer()->style();
     m_graphicsLayer->setPreserves3D(style->transformStyle3D() == TransformStyle3DPreserve3D && !renderer()->hasReflection());