WebCore:
authorbdakin <bdakin@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 23 Jan 2007 23:38:17 +0000 (23:38 +0000)
committerbdakin <bdakin@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 23 Jan 2007 23:38:17 +0000 (23:38 +0000)
        Reviewed by Hyatt.

        Follow-up to the fix below for 4843226.

        * rendering/RenderLayer.cpp:
        (WebCore::transparencyClipBox): Has to be recursive or we will miss
        out on grandchildren!

LayoutTests:
        Reviewed by Hyatt.

        Updated test for 4843226 to include a grandchild.

        * fast/clip/nestedTransparencyClip-expected.checksum:
        * fast/clip/nestedTransparencyClip-expected.png:
        * fast/clip/nestedTransparencyClip-expected.txt:
        * fast/clip/nestedTransparencyClip.html:

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

LayoutTests/ChangeLog
LayoutTests/fast/clip/nestedTransparencyClip-expected.checksum
LayoutTests/fast/clip/nestedTransparencyClip-expected.png
LayoutTests/fast/clip/nestedTransparencyClip-expected.txt
LayoutTests/fast/clip/nestedTransparencyClip.html
WebCore/ChangeLog
WebCore/rendering/RenderLayer.cpp

index 33649a1..fec75df 100644 (file)
@@ -1,3 +1,14 @@
+2007-01-23  Beth Dakin  <bdakin@apple.com>
+
+        Reviewed by Hyatt.
+
+        Updated test for 4843226 to include a grandchild.
+
+        * fast/clip/nestedTransparencyClip-expected.checksum:
+        * fast/clip/nestedTransparencyClip-expected.png:
+        * fast/clip/nestedTransparencyClip-expected.txt:
+        * fast/clip/nestedTransparencyClip.html:
+
 2007-01-23  Justin Garcia  <justin.garcia@apple.com>
 
         Reviewed by john
index 72d4d0a..5f954ec 100644 (file)
@@ -1 +1 @@
-718759ff972235af76d7fc1c9e4e488b
\ No newline at end of file
+e8a51dc8066bb3677c3150325863d49a
\ No newline at end of file
index 4446c30..9e49f1e 100644 (file)
Binary files a/LayoutTests/fast/clip/nestedTransparencyClip-expected.png and b/LayoutTests/fast/clip/nestedTransparencyClip-expected.png differ
index 139234e..ec22225 100644 (file)
@@ -6,15 +6,20 @@ layer at (0,0) size 800x600
       RenderBlock (anonymous) at (0,0) size 784x54
         RenderText {#text} at (0,0) size 741x36
           text run at (0,0) width 741: "This test is for a bug we had with clipping transparency layers. The bug is visual, so the pixel test is more worthwhile."
-          text run at (0,18) width 681: "Below, both \"Applications\" and \"Productivity\" should appear. Before, we were clipping out \"Productivity.\""
-        RenderBR {BR} at (681,32) size 0x0
+          text run at (0,18) width 646: "Below, \"Peach,\" \"Plum,\" and \"Pear\" should appear. Before, we were clipping out \"Plum\" and \"Pear.\""
+        RenderBR {BR} at (646,32) size 0x0
         RenderBR {BR} at (0,36) size 0x18
 layer at (8,62) size 784x18
   RenderBlock {DIV} at (0,54) size 784x18
-    RenderText {#text} at (0,0) size 84x18
-      text run at (0,0) width 84: "Applications "
+    RenderText {#text} at (0,0) size 42x18
+      text run at (0,0) width 42: "Peach "
     RenderText {#text} at (0,0) size 0x0
-layer at (8,80) size 77x18
-  RenderBlock (positioned) {DIV} at (8,80) size 77x18
-    RenderText {#text} at (0,0) size 77x18
-      text run at (0,0) width 77: "Productivity"
+layer at (8,80) size 33x18
+  RenderBlock (positioned) {DIV} at (8,80) size 33x18
+    RenderText {#text} at (0,0) size 33x18
+      text run at (0,0) width 33: "Plum"
+    RenderText {#text} at (0,0) size 0x0
+layer at (8,98) size 28x18
+  RenderBlock (positioned) {DIV} at (0,18) size 28x18
+    RenderText {#text} at (0,0) size 28x18
+      text run at (0,0) width 28: "Pear"
index aee48fe..f74b36a 100644 (file)
@@ -1,5 +1,7 @@
-This test is for a bug we had with clipping transparency layers. The bug is visual, so the pixel test is more worthwhile. Below, both "Applications" and "Productivity" should appear. Before, we were clipping out "Productivity."<br><br>
+This test is for a bug we had with clipping transparency layers. The bug is visual, so the pixel test is more worthwhile. Below, "Peach," "Plum," and "Pear" should appear. Before, we were clipping out "Plum" and "Pear."<br><br>
 
-<div style="opacity:.95;">Applications
-    <div style="position:absolute; opacity:.95;">Productivity</div>
+<div style="opacity:.95;">Peach
+    <div style="position:absolute; opacity:.95;">Plum
+        <div style="position:absolute; opacity:.95;">Pear</div>
+    </div>
 </div>
index a5ade3f..e09ffd4 100644 (file)
@@ -1,3 +1,13 @@
+2007-01-23  Beth Dakin  <bdakin@apple.com>
+
+        Reviewed by Hyatt.
+
+        Follow-up to the fix below for 4843226.
+
+        * rendering/RenderLayer.cpp:
+        (WebCore::transparencyClipBox): Has to be recursive or we will miss 
+        out on grandchildren!
+
 2007-01-23  Oliver Hunt  <oliver@apple.com>
 
         Reviewed by Adam.
index 608b9ef..f7a6145 100644 (file)
@@ -459,7 +459,7 @@ static IntRect transparencyClipBox(RenderLayer* l)
     // Note: we don't have to walk z-order lists since transparent elements always establish
     // a stacking context.  This means we can just walk the layer tree directly. 
     for (RenderLayer* curr = l->firstChild(); curr; curr = curr->nextSibling())
-        clipRect.unite(curr->absoluteBoundingBox());
+        clipRect.unite(transparencyClipBox(curr));
     
     return clipRect;
 }