WebCore:
authormitz@apple.com <mitz@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 20 Jun 2008 02:53:12 +0000 (02:53 +0000)
committermitz@apple.com <mitz@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 20 Jun 2008 02:53:12 +0000 (02:53 +0000)
        Reviewed by John Sullivan.

        - fix a bug where anonymous tables were inserted in the wrong place

        Tests: tables/mozilla/bugs/bug2479-2.html
               tables/mozilla/bugs/bug278266.html
               tables/mozilla/bugs/bug8411.xml

        * rendering/RenderContainer.cpp:
        (WebCore::RenderContainer::addChild): Changed to create an anonymous
        table and insert it before beforeChild unless the insertion point is
        right after an existing anonymous table, in which case the existing
        table is used.

LayoutTests:

        Reviewed by John Sullivan.

        - updated tests and results after fixing a bug where anonymous tables were inserted in the wrong place

        * fast/forms/form-hides-table.html: Wrapped every case in a block in
        order to prevent cross-talk.
        * platform/mac/fast/forms/form-hides-table-expected.checksum:
        * platform/mac/fast/forms/form-hides-table-expected.png:
        * platform/mac/fast/forms/form-hides-table-expected.txt:
        * platform/mac/tables/mozilla/bugs/bug2479-2-expected.checksum: Added.
        * platform/mac/tables/mozilla/bugs/bug2479-2-expected.png: Added.
        * platform/mac/tables/mozilla/bugs/bug2479-2-expected.txt: Added.
        * platform/mac/tables/mozilla/bugs/bug2479-4-expected.txt:
        * platform/mac/tables/mozilla/bugs/bug278266-expected.checksum: Added.
        * platform/mac/tables/mozilla/bugs/bug278266-expected.png: Added.
        * platform/mac/tables/mozilla/bugs/bug278266-expected.txt: Added.
        * platform/mac/tables/mozilla/bugs/bug8411-expected.checksum: Added.
        * platform/mac/tables/mozilla/bugs/bug8411-expected.png: Added.
        * platform/mac/tables/mozilla/bugs/bug8411-expected.txt: Added.
        * platform/mac/tables/mozilla_expected_failures/bugs/bug2479-2-expected.checksum: Removed.
        * platform/mac/tables/mozilla_expected_failures/bugs/bug2479-2-expected.png: Removed.
        * platform/mac/tables/mozilla_expected_failures/bugs/bug2479-2-expected.txt: Removed.
        * platform/mac/tables/mozilla_expected_failures/bugs/bug278266-expected.checksum: Removed.
        * platform/mac/tables/mozilla_expected_failures/bugs/bug278266-expected.png: Removed.
        * platform/mac/tables/mozilla_expected_failures/bugs/bug278266-expected.txt: Removed.
        * platform/mac/tables/mozilla_expected_failures/bugs/bug8411-expected.checksum: Removed.
        * platform/mac/tables/mozilla_expected_failures/bugs/bug8411-expected.png: Removed.
        * platform/mac/tables/mozilla_expected_failures/bugs/bug8411-expected.txt: Removed.
        * platform/mac/tables/mozilla_expected_failures/other/test4-expected.txt:
        * tables/mozilla/bugs/bug2479-2.html: Copied from LayoutTests/tables/mozilla_expected_failures/bugs/bug2479-2.html.
        * tables/mozilla/bugs/bug278266.html: Copied from LayoutTests/tables/mozilla_expected_failures/bugs/bug278266.html.
        * tables/mozilla/bugs/bug8411.xml: Copied from LayoutTests/tables/mozilla_expected_failures/bugs/bug8411.xml.
        * tables/mozilla_expected_failures/bugs/bug2479-2.html: Removed.
        * tables/mozilla_expected_failures/bugs/bug278266.html: Removed.
        * tables/mozilla_expected_failures/bugs/bug8411.xml: Removed.

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

28 files changed:
LayoutTests/ChangeLog
LayoutTests/fast/forms/form-hides-table.html
LayoutTests/platform/mac/fast/forms/form-hides-table-expected.checksum
LayoutTests/platform/mac/fast/forms/form-hides-table-expected.png
LayoutTests/platform/mac/fast/forms/form-hides-table-expected.txt
LayoutTests/platform/mac/tables/mozilla/bugs/bug2479-2-expected.checksum [new file with mode: 0644]
LayoutTests/platform/mac/tables/mozilla/bugs/bug2479-2-expected.png [new file with mode: 0644]
LayoutTests/platform/mac/tables/mozilla/bugs/bug2479-2-expected.txt [moved from LayoutTests/platform/mac/tables/mozilla_expected_failures/bugs/bug2479-2-expected.txt with 98% similarity]
LayoutTests/platform/mac/tables/mozilla/bugs/bug2479-4-expected.txt
LayoutTests/platform/mac/tables/mozilla/bugs/bug278266-expected.checksum [new file with mode: 0644]
LayoutTests/platform/mac/tables/mozilla/bugs/bug278266-expected.png [new file with mode: 0644]
LayoutTests/platform/mac/tables/mozilla/bugs/bug278266-expected.txt [moved from LayoutTests/platform/mac/tables/mozilla_expected_failures/bugs/bug278266-expected.txt with 87% similarity]
LayoutTests/platform/mac/tables/mozilla/bugs/bug8411-expected.checksum [new file with mode: 0644]
LayoutTests/platform/mac/tables/mozilla/bugs/bug8411-expected.png [new file with mode: 0644]
LayoutTests/platform/mac/tables/mozilla/bugs/bug8411-expected.txt [new file with mode: 0644]
LayoutTests/platform/mac/tables/mozilla_expected_failures/bugs/bug2479-2-expected.checksum [deleted file]
LayoutTests/platform/mac/tables/mozilla_expected_failures/bugs/bug2479-2-expected.png [deleted file]
LayoutTests/platform/mac/tables/mozilla_expected_failures/bugs/bug278266-expected.checksum [deleted file]
LayoutTests/platform/mac/tables/mozilla_expected_failures/bugs/bug278266-expected.png [deleted file]
LayoutTests/platform/mac/tables/mozilla_expected_failures/bugs/bug8411-expected.checksum [deleted file]
LayoutTests/platform/mac/tables/mozilla_expected_failures/bugs/bug8411-expected.png [deleted file]
LayoutTests/platform/mac/tables/mozilla_expected_failures/bugs/bug8411-expected.txt [deleted file]
LayoutTests/platform/mac/tables/mozilla_expected_failures/other/test4-expected.txt
LayoutTests/tables/mozilla/bugs/bug2479-2.html [moved from LayoutTests/tables/mozilla_expected_failures/bugs/bug2479-2.html with 100% similarity]
LayoutTests/tables/mozilla/bugs/bug278266.html [moved from LayoutTests/tables/mozilla_expected_failures/bugs/bug278266.html with 100% similarity]
LayoutTests/tables/mozilla/bugs/bug8411.xml [moved from LayoutTests/tables/mozilla_expected_failures/bugs/bug8411.xml with 100% similarity]
WebCore/ChangeLog
WebCore/rendering/RenderContainer.cpp

index 43057bf..282eaac 100644 (file)
@@ -1,3 +1,41 @@
+2008-06-19  Dan Bernstein  <mitz@apple.com>
+
+        Reviewed by John Sullivan.
+
+        - updated tests and results after fixing a bug where anonymous tables were inserted in the wrong place
+
+        * fast/forms/form-hides-table.html: Wrapped every case in a block in
+        order to prevent cross-talk.
+        * platform/mac/fast/forms/form-hides-table-expected.checksum:
+        * platform/mac/fast/forms/form-hides-table-expected.png:
+        * platform/mac/fast/forms/form-hides-table-expected.txt:
+        * platform/mac/tables/mozilla/bugs/bug2479-2-expected.checksum: Added.
+        * platform/mac/tables/mozilla/bugs/bug2479-2-expected.png: Added.
+        * platform/mac/tables/mozilla/bugs/bug2479-2-expected.txt: Added.
+        * platform/mac/tables/mozilla/bugs/bug2479-4-expected.txt:
+        * platform/mac/tables/mozilla/bugs/bug278266-expected.checksum: Added.
+        * platform/mac/tables/mozilla/bugs/bug278266-expected.png: Added.
+        * platform/mac/tables/mozilla/bugs/bug278266-expected.txt: Added.
+        * platform/mac/tables/mozilla/bugs/bug8411-expected.checksum: Added.
+        * platform/mac/tables/mozilla/bugs/bug8411-expected.png: Added.
+        * platform/mac/tables/mozilla/bugs/bug8411-expected.txt: Added.
+        * platform/mac/tables/mozilla_expected_failures/bugs/bug2479-2-expected.checksum: Removed.
+        * platform/mac/tables/mozilla_expected_failures/bugs/bug2479-2-expected.png: Removed.
+        * platform/mac/tables/mozilla_expected_failures/bugs/bug2479-2-expected.txt: Removed.
+        * platform/mac/tables/mozilla_expected_failures/bugs/bug278266-expected.checksum: Removed.
+        * platform/mac/tables/mozilla_expected_failures/bugs/bug278266-expected.png: Removed.
+        * platform/mac/tables/mozilla_expected_failures/bugs/bug278266-expected.txt: Removed.
+        * platform/mac/tables/mozilla_expected_failures/bugs/bug8411-expected.checksum: Removed.
+        * platform/mac/tables/mozilla_expected_failures/bugs/bug8411-expected.png: Removed.
+        * platform/mac/tables/mozilla_expected_failures/bugs/bug8411-expected.txt: Removed.
+        * platform/mac/tables/mozilla_expected_failures/other/test4-expected.txt:
+        * tables/mozilla/bugs/bug2479-2.html: Copied from LayoutTests/tables/mozilla_expected_failures/bugs/bug2479-2.html.
+        * tables/mozilla/bugs/bug278266.html: Copied from LayoutTests/tables/mozilla_expected_failures/bugs/bug278266.html.
+        * tables/mozilla/bugs/bug8411.xml: Copied from LayoutTests/tables/mozilla_expected_failures/bugs/bug8411.xml.
+        * tables/mozilla_expected_failures/bugs/bug2479-2.html: Removed.
+        * tables/mozilla_expected_failures/bugs/bug278266.html: Removed.
+        * tables/mozilla_expected_failures/bugs/bug8411.xml: Removed.
+
 2008-06-19  Justin Garcia  <justin.garcia@apple.com>
 
         Reviewed by Oliver.
index 5b1be96..b6bed58 100644 (file)
@@ -1,18 +1,18 @@
 <p>This page has a few tables within form elements within divs with various display styles.</p>
 <p>See <a href="http://bugzilla.opendarwin.org/show_bug.cgi?id=4977">Bugzilla Bug 4977</a>.</p>
-<div style="display:inline;"><form><table><tr><td>display: inline</td></tr></table></form></div>
-<div style="display:block;"><form><table><tr><td>display: block</td></tr></table></form></div>
-<div style="display:list-item;"><form><table><tr><td>display: list-item</td></tr></table></form></div>
-<div style="display:run-in;"><form><table><tr><td>display: run-in</td></tr></table></form></div>
-<div style="display:compact;"><form><table><tr><td>display: compact</td></tr></table></form></div>
-<div style="display:inline-block;"><form><table><tr><td>display: inline-block</td></tr></table></form></div>
-<div style="display:table;"><form><table><tr><td>display: table</td></tr></table></form></div>
-<div style="display:inline-table;"><form><table><tr><td>display: inline-table</td></tr></table></form></div>
-<div style="display:table-row-group;"><form><table><tr><td>display: table-row-group</td></tr></table></form></div>
-<div style="display:table-header-group;"><form><table><tr><td>display: table-header-group</td></tr></table></form></div>
-<div style="display:table-footer-group;"><form><table><tr><td>display: table-footer-group</td></tr></table></form></div>
-<div style="display:table-row;"><form><table><tr><td>display: table-row</td></tr></table></form></div>
-<div style="display:table-column-group;"><form><table><tr><td>display: table-column-group</td></tr></table></form></div>
-<div style="display:table-column;"><form><table><tr><td>display: table-column</td></tr></table></form></div>
-<div style="display:table-cell;"><form><table><tr><td>display: table-cell</td></tr></table></form></div>
-<div style="display:table-caption;"><form><table><tr><td>display: table-caption</td></tr></table></form></div>
+<div><div style="display:inline;"><form><table><tr><td>display: inline</td></tr></table></form></div></div>
+<div><div style="display:block;"><form><table><tr><td>display: block</td></tr></table></form></div></div>
+<div><div style="display:list-item;"><form><table><tr><td>display: list-item</td></tr></table></form></div></div>
+<div><div style="display:run-in;"><form><table><tr><td>display: run-in</td></tr></table></form></div></div>
+<div><div style="display:compact;"><form><table><tr><td>display: compact</td></tr></table></form></div></div>
+<div><div style="display:inline-block;"><form><table><tr><td>display: inline-block</td></tr></table></form></div></div>
+<div><div style="display:table;"><form><table><tr><td>display: table</td></tr></table></form></div></div>
+<div><div style="display:inline-table;"><form><table><tr><td>display: inline-table</td></tr></table></form></div></div>
+<div><div style="display:table-row-group;"><form><table><tr><td>display: table-row-group</td></tr></table></form></div></div>
+<div><div style="display:table-header-group;"><form><table><tr><td>display: table-header-group</td></tr></table></form></div></div>
+<div><div style="display:table-footer-group;"><form><table><tr><td>display: table-footer-group</td></tr></table></form></div></div>
+<div><div style="display:table-row;"><form><table><tr><td>display: table-row</td></tr></table></form></div></div>
+<div><div style="display:table-column-group;"><form><table><tr><td>display: table-column-group</td></tr></table></form></div></div>
+<div><div style="display:table-column;"><form><table><tr><td>display: table-column</td></tr></table></form></div></div>
+<div><div style="display:table-cell;"><form><table><tr><td>display: table-cell</td></tr></table></form></div></div>
+<div><div style="display:table-caption;"><form><table><tr><td>display: table-caption</td></tr></table></form></div></div>
index e34a4f8..33da59d 100644 (file)
Binary files a/LayoutTests/platform/mac/fast/forms/form-hides-table-expected.png and b/LayoutTests/platform/mac/fast/forms/form-hides-table-expected.png differ
index 7864088..b930447 100644 (file)
@@ -14,54 +14,58 @@ layer at (0,0) size 785x682
             text run at (27,0) width 120: "Bugzilla Bug 4977"
         RenderText {#text} at (147,0) size 4x18
           text run at (147,0) width 4: "."
-      RenderBlock (anonymous) at (0,68) size 769x0
-        RenderInline {DIV} at (0,0) size 0x0
-      RenderBlock (anonymous) at (0,68) size 769x24
-        RenderBlock {FORM} at (0,0) size 769x24
-          RenderTable {TABLE} at (0,0) size 94x24
-            RenderTableSection {TBODY} at (0,0) size 94x24
-              RenderTableRow {TR} at (0,2) size 94x20
-                RenderTableCell {TD} at (2,2) size 90x20 [r=0 c=0 rs=1 cs=1]
-                  RenderText {#text} at (1,1) size 88x18
-                    text run at (1,1) width 88: "display: inline"
-      RenderBlock (anonymous) at (0,108) size 769x0
-        RenderInline {DIV} at (0,0) size 0x0
-        RenderText {#text} at (0,0) size 0x0
+      RenderBlock {DIV} at (0,68) size 769x24
+        RenderBlock (anonymous) at (0,0) size 769x0
+          RenderInline {DIV} at (0,0) size 0x0
+        RenderBlock (anonymous) at (0,0) size 769x24
+          RenderBlock {FORM} at (0,0) size 769x24
+            RenderTable {TABLE} at (0,0) size 94x24
+              RenderTableSection {TBODY} at (0,0) size 94x24
+                RenderTableRow {TR} at (0,2) size 94x20
+                  RenderTableCell {TD} at (2,2) size 90x20 [r=0 c=0 rs=1 cs=1]
+                    RenderText {#text} at (1,1) size 88x18
+                      text run at (1,1) width 88: "display: inline"
+        RenderBlock (anonymous) at (0,40) size 769x0
+          RenderInline {DIV} at (0,0) size 0x0
       RenderBlock {DIV} at (0,108) size 769x24
-        RenderBlock {FORM} at (0,0) size 769x24
-          RenderTable {TABLE} at (0,0) size 94x24
-            RenderTableSection {TBODY} at (0,0) size 94x24
-              RenderTableRow {TR} at (0,2) size 94x20
-                RenderTableCell {TD} at (2,2) size 90x20 [r=0 c=0 rs=1 cs=1]
-                  RenderText {#text} at (1,1) size 88x18
-                    text run at (1,1) width 88: "display: block"
-      RenderListItem {DIV} at (0,148) size 769x42
-        RenderBlock (anonymous) at (0,0) size 769x18
-          RenderListMarker at (-17,0) size 7x18: bullet
-        RenderBlock {FORM} at (0,18) size 769x24
-          RenderTable {TABLE} at (0,0) size 109x24
-            RenderTableSection {TBODY} at (0,0) size 109x24
-              RenderTableRow {TR} at (0,2) size 109x20
-                RenderTableCell {TD} at (2,2) size 105x20 [r=0 c=0 rs=1 cs=1]
-                  RenderText {#text} at (1,1) size 103x18
-                    text run at (1,1) width 103: "display: list-item"
-      RenderBlock (run-in) {DIV} at (0,206) size 769x24
-        RenderBlock {FORM} at (0,0) size 769x24
-          RenderTable {TABLE} at (0,0) size 97x24
-            RenderTableSection {TBODY} at (0,0) size 97x24
-              RenderTableRow {TR} at (0,2) size 97x20
-                RenderTableCell {TD} at (2,2) size 93x20 [r=0 c=0 rs=1 cs=1]
-                  RenderText {#text} at (1,1) size 91x18
-                    text run at (1,1) width 91: "display: run-in"
-      RenderBlock (compact) {DIV} at (0,246) size 769x24
-        RenderBlock {FORM} at (0,0) size 769x24
-          RenderTable {TABLE} at (0,0) size 112x24
-            RenderTableSection {TBODY} at (0,0) size 112x24
-              RenderTableRow {TR} at (0,2) size 112x20
-                RenderTableCell {TD} at (2,2) size 108x20 [r=0 c=0 rs=1 cs=1]
-                  RenderText {#text} at (1,1) size 106x18
-                    text run at (1,1) width 106: "display: compact"
-      RenderBlock (anonymous) at (0,286) size 769x40
+        RenderBlock {DIV} at (0,0) size 769x24
+          RenderBlock {FORM} at (0,0) size 769x24
+            RenderTable {TABLE} at (0,0) size 94x24
+              RenderTableSection {TBODY} at (0,0) size 94x24
+                RenderTableRow {TR} at (0,2) size 94x20
+                  RenderTableCell {TD} at (2,2) size 90x20 [r=0 c=0 rs=1 cs=1]
+                    RenderText {#text} at (1,1) size 88x18
+                      text run at (1,1) width 88: "display: block"
+      RenderBlock {DIV} at (0,148) size 769x42
+        RenderListItem {DIV} at (0,0) size 769x42
+          RenderBlock (anonymous) at (0,0) size 769x18
+            RenderListMarker at (-17,0) size 7x18: bullet
+          RenderBlock {FORM} at (0,18) size 769x24
+            RenderTable {TABLE} at (0,0) size 109x24
+              RenderTableSection {TBODY} at (0,0) size 109x24
+                RenderTableRow {TR} at (0,2) size 109x20
+                  RenderTableCell {TD} at (2,2) size 105x20 [r=0 c=0 rs=1 cs=1]
+                    RenderText {#text} at (1,1) size 103x18
+                      text run at (1,1) width 103: "display: list-item"
+      RenderBlock {DIV} at (0,206) size 769x24
+        RenderBlock (run-in) {DIV} at (0,0) size 769x24
+          RenderBlock {FORM} at (0,0) size 769x24
+            RenderTable {TABLE} at (0,0) size 97x24
+              RenderTableSection {TBODY} at (0,0) size 97x24
+                RenderTableRow {TR} at (0,2) size 97x20
+                  RenderTableCell {TD} at (2,2) size 93x20 [r=0 c=0 rs=1 cs=1]
+                    RenderText {#text} at (1,1) size 91x18
+                      text run at (1,1) width 91: "display: run-in"
+      RenderBlock {DIV} at (0,246) size 769x24
+        RenderBlock (compact) {DIV} at (0,0) size 769x24
+          RenderBlock {FORM} at (0,0) size 769x24
+            RenderTable {TABLE} at (0,0) size 112x24
+              RenderTableSection {TBODY} at (0,0) size 112x24
+                RenderTableRow {TR} at (0,2) size 112x20
+                  RenderTableCell {TD} at (2,2) size 108x20 [r=0 c=0 rs=1 cs=1]
+                    RenderText {#text} at (1,1) size 106x18
+                      text run at (1,1) width 106: "display: compact"
+      RenderBlock {DIV} at (0,286) size 769x40
         RenderBlock {DIV} at (0,0) size 134x40
           RenderBlock {FORM} at (0,0) size 134x24
             RenderTable {TABLE} at (0,0) size 134x24
@@ -70,65 +74,19 @@ layer at (0,0) size 785x682
                   RenderTableCell {TD} at (2,2) size 130x20 [r=0 c=0 rs=1 cs=1]
                     RenderText {#text} at (1,1) size 128x18
                       text run at (1,1) width 128: "display: inline-block"
-        RenderText {#text} at (0,0) size 0x0
-      RenderTable {DIV} at (0,326) size 89x40
-        RenderTableSection (anonymous) at (0,0) size 89x40
-          RenderTableRow (anonymous) at (0,0) size 89x40
-            RenderTableCell (anonymous) at (0,0) size 89x40 [r=0 c=0 rs=1 cs=1]
-              RenderBlock {FORM} at (0,0) size 89x24
-                RenderTable {TABLE} at (0,0) size 89x24
-                  RenderTableSection {TBODY} at (0,0) size 89x24
-                    RenderTableRow {TR} at (0,2) size 89x20
-                      RenderTableCell {TD} at (2,2) size 85x20 [r=0 c=0 rs=1 cs=1]
-                        RenderText {#text} at (1,1) size 83x18
-                          text run at (1,1) width 83: "display: table"
-      RenderTable at (0,366) size 161x40
-        RenderTableSection {DIV} at (0,0) size 161x40
-          RenderTableRow (anonymous) at (0,0) size 161x40
-            RenderTableCell (anonymous) at (0,0) size 161x40 [r=0 c=0 rs=1 cs=1]
-              RenderBlock {FORM} at (0,0) size 161x24
-                RenderTable {TABLE} at (0,0) size 161x24
-                  RenderTableSection {TBODY} at (0,0) size 161x24
-                    RenderTableRow {TR} at (0,2) size 161x20
-                      RenderTableCell {TD} at (2,2) size 157x20 [r=0 c=0 rs=1 cs=1]
-                        RenderText {#text} at (1,1) size 155x18
-                          text run at (1,1) width 155: "display: table-row-group"
-      RenderTable at (0,406) size 178x40
-        RenderTableSection {DIV} at (0,0) size 178x40
-          RenderTableRow (anonymous) at (0,0) size 178x40
-            RenderTableCell (anonymous) at (0,0) size 178x40 [r=0 c=0 rs=1 cs=1]
-              RenderBlock {FORM} at (0,0) size 178x24
-                RenderTable {TABLE} at (0,0) size 178x24
-                  RenderTableSection {TBODY} at (0,0) size 178x24
-                    RenderTableRow {TR} at (0,2) size 178x20
-                      RenderTableCell {TD} at (2,2) size 174x20 [r=0 c=0 rs=1 cs=1]
-                        RenderText {#text} at (1,1) size 172x18
-                          text run at (1,1) width 172: "display: table-header-group"
-      RenderTable at (0,446) size 173x40
-        RenderTableSection {DIV} at (0,0) size 173x40
-          RenderTableRow (anonymous) at (0,0) size 173x40
-            RenderTableCell (anonymous) at (0,0) size 173x40 [r=0 c=0 rs=1 cs=1]
-              RenderBlock {FORM} at (0,0) size 173x24
-                RenderTable {TABLE} at (0,0) size 173x24
-                  RenderTableSection {TBODY} at (0,0) size 173x24
-                    RenderTableRow {TR} at (0,2) size 173x20
-                      RenderTableCell {TD} at (2,2) size 169x20 [r=0 c=0 rs=1 cs=1]
-                        RenderText {#text} at (1,1) size 167x18
-                          text run at (1,1) width 167: "display: table-footer-group"
-      RenderTable at (0,486) size 119x40
-        RenderTableSection (anonymous) at (0,0) size 119x40
-          RenderTableRow {DIV} at (0,0) size 119x40
-            RenderTableCell (anonymous) at (0,0) size 119x40 [r=0 c=0 rs=1 cs=1]
-              RenderBlock {FORM} at (0,0) size 119x24
-                RenderTable {TABLE} at (0,0) size 119x24
-                  RenderTableSection {TBODY} at (0,0) size 119x24
-                    RenderTableRow {TR} at (0,2) size 119x20
-                      RenderTableCell {TD} at (2,2) size 115x20 [r=0 c=0 rs=1 cs=1]
-                        RenderText {#text} at (1,1) size 113x18
-                          text run at (1,1) width 113: "display: table-row"
-      RenderTable at (0,526) size 0x0
-        RenderTableCol {DIV} at (0,0) size 0x0
-      RenderBlock (anonymous) at (0,526) size 769x40
+      RenderBlock {DIV} at (0,326) size 769x40
+        RenderTable {DIV} at (0,0) size 89x40
+          RenderTableSection (anonymous) at (0,0) size 89x40
+            RenderTableRow (anonymous) at (0,0) size 89x40
+              RenderTableCell (anonymous) at (0,0) size 89x40 [r=0 c=0 rs=1 cs=1]
+                RenderBlock {FORM} at (0,0) size 89x24
+                  RenderTable {TABLE} at (0,0) size 89x24
+                    RenderTableSection {TBODY} at (0,0) size 89x24
+                      RenderTableRow {TR} at (0,2) size 89x20
+                        RenderTableCell {TD} at (2,2) size 85x20 [r=0 c=0 rs=1 cs=1]
+                          RenderText {#text} at (1,1) size 83x18
+                            text run at (1,1) width 83: "display: table"
+      RenderBlock {DIV} at (0,366) size 769x40
         RenderTable {DIV} at (0,0) size 129x40
           RenderTableSection (anonymous) at (0,0) size 129x40
             RenderTableRow (anonymous) at (0,0) size 129x40
@@ -140,27 +98,80 @@ layer at (0,0) size 785x682
                         RenderTableCell {TD} at (2,2) size 125x20 [r=0 c=0 rs=1 cs=1]
                           RenderText {#text} at (1,1) size 123x18
                             text run at (1,1) width 123: "display: inline-table"
-        RenderText {#text} at (0,0) size 0x0
-      RenderTable at (0,566) size 116x40
-        RenderTableSection (anonymous) at (0,0) size 116x40
-          RenderTableRow (anonymous) at (0,0) size 116x40
-            RenderTableCell {DIV} at (0,0) size 116x40 [r=0 c=0 rs=1 cs=1]
-              RenderBlock {FORM} at (0,0) size 116x24
-                RenderTable {TABLE} at (0,0) size 116x24
-                  RenderTableSection {TBODY} at (0,0) size 116x24
-                    RenderTableRow {TR} at (0,2) size 116x20
-                      RenderTableCell {TD} at (2,2) size 112x20 [r=0 c=0 rs=1 cs=1]
-                        RenderText {#text} at (1,1) size 110x18
-                          text run at (1,1) width 110: "display: table-cell"
-      RenderTable at (0,606) size 55x60
-        RenderBlock {DIV} at (0,0) size 55x60
-          RenderBlock {FORM} at (0,0) size 55x60
-            RenderTable {TABLE} at (0,0) size 55x60
-              RenderTableSection {TBODY} at (0,0) size 55x60
-                RenderTableRow {TR} at (0,2) size 55x56
-                  RenderTableCell {TD} at (2,2) size 51x56 [r=0 c=0 rs=1 cs=1]
-                    RenderText {#text} at (1,1) size 49x54
-                      text run at (1,1) width 49: "display:"
-                      text run at (1,19) width 35: "table-"
-                      text run at (1,37) width 46: "caption"
-      RenderTableCol {DIV} at (0,666) size 0x0
+      RenderBlock {DIV} at (0,406) size 769x40
+        RenderTable at (0,0) size 161x40
+          RenderTableSection {DIV} at (0,0) size 161x40
+            RenderTableRow (anonymous) at (0,0) size 161x40
+              RenderTableCell (anonymous) at (0,0) size 161x40 [r=0 c=0 rs=1 cs=1]
+                RenderBlock {FORM} at (0,0) size 161x24
+                  RenderTable {TABLE} at (0,0) size 161x24
+                    RenderTableSection {TBODY} at (0,0) size 161x24
+                      RenderTableRow {TR} at (0,2) size 161x20
+                        RenderTableCell {TD} at (2,2) size 157x20 [r=0 c=0 rs=1 cs=1]
+                          RenderText {#text} at (1,1) size 155x18
+                            text run at (1,1) width 155: "display: table-row-group"
+      RenderBlock {DIV} at (0,446) size 769x40
+        RenderTable at (0,0) size 178x40
+          RenderTableSection {DIV} at (0,0) size 178x40
+            RenderTableRow (anonymous) at (0,0) size 178x40
+              RenderTableCell (anonymous) at (0,0) size 178x40 [r=0 c=0 rs=1 cs=1]
+                RenderBlock {FORM} at (0,0) size 178x24
+                  RenderTable {TABLE} at (0,0) size 178x24
+                    RenderTableSection {TBODY} at (0,0) size 178x24
+                      RenderTableRow {TR} at (0,2) size 178x20
+                        RenderTableCell {TD} at (2,2) size 174x20 [r=0 c=0 rs=1 cs=1]
+                          RenderText {#text} at (1,1) size 172x18
+                            text run at (1,1) width 172: "display: table-header-group"
+      RenderBlock {DIV} at (0,486) size 769x40
+        RenderTable at (0,0) size 173x40
+          RenderTableSection {DIV} at (0,0) size 173x40
+            RenderTableRow (anonymous) at (0,0) size 173x40
+              RenderTableCell (anonymous) at (0,0) size 173x40 [r=0 c=0 rs=1 cs=1]
+                RenderBlock {FORM} at (0,0) size 173x24
+                  RenderTable {TABLE} at (0,0) size 173x24
+                    RenderTableSection {TBODY} at (0,0) size 173x24
+                      RenderTableRow {TR} at (0,2) size 173x20
+                        RenderTableCell {TD} at (2,2) size 169x20 [r=0 c=0 rs=1 cs=1]
+                          RenderText {#text} at (1,1) size 167x18
+                            text run at (1,1) width 167: "display: table-footer-group"
+      RenderBlock {DIV} at (0,526) size 769x40
+        RenderTable at (0,0) size 119x40
+          RenderTableSection (anonymous) at (0,0) size 119x40
+            RenderTableRow {DIV} at (0,0) size 119x40
+              RenderTableCell (anonymous) at (0,0) size 119x40 [r=0 c=0 rs=1 cs=1]
+                RenderBlock {FORM} at (0,0) size 119x24
+                  RenderTable {TABLE} at (0,0) size 119x24
+                    RenderTableSection {TBODY} at (0,0) size 119x24
+                      RenderTableRow {TR} at (0,2) size 119x20
+                        RenderTableCell {TD} at (2,2) size 115x20 [r=0 c=0 rs=1 cs=1]
+                          RenderText {#text} at (1,1) size 113x18
+                            text run at (1,1) width 113: "display: table-row"
+      RenderBlock {DIV} at (0,566) size 769x0
+        RenderTable at (0,0) size 0x0
+          RenderTableCol {DIV} at (0,0) size 0x0
+      RenderBlock {DIV} at (0,566) size 769x0
+        RenderTableCol {DIV} at (0,0) size 0x0
+      RenderBlock {DIV} at (0,566) size 769x40
+        RenderTable at (0,0) size 116x40
+          RenderTableSection (anonymous) at (0,0) size 116x40
+            RenderTableRow (anonymous) at (0,0) size 116x40
+              RenderTableCell {DIV} at (0,0) size 116x40 [r=0 c=0 rs=1 cs=1]
+                RenderBlock {FORM} at (0,0) size 116x24
+                  RenderTable {TABLE} at (0,0) size 116x24
+                    RenderTableSection {TBODY} at (0,0) size 116x24
+                      RenderTableRow {TR} at (0,2) size 116x20
+                        RenderTableCell {TD} at (2,2) size 112x20 [r=0 c=0 rs=1 cs=1]
+                          RenderText {#text} at (1,1) size 110x18
+                            text run at (1,1) width 110: "display: table-cell"
+      RenderBlock {DIV} at (0,606) size 769x60
+        RenderTable at (0,0) size 55x60
+          RenderBlock {DIV} at (0,0) size 55x60
+            RenderBlock {FORM} at (0,0) size 55x60
+              RenderTable {TABLE} at (0,0) size 55x60
+                RenderTableSection {TBODY} at (0,0) size 55x60
+                  RenderTableRow {TR} at (0,2) size 55x56
+                    RenderTableCell {TD} at (2,2) size 51x56 [r=0 c=0 rs=1 cs=1]
+                      RenderText {#text} at (1,1) size 49x54
+                        text run at (1,1) width 49: "display:"
+                        text run at (1,19) width 35: "table-"
+                        text run at (1,37) width 46: "caption"
diff --git a/LayoutTests/platform/mac/tables/mozilla/bugs/bug2479-2-expected.checksum b/LayoutTests/platform/mac/tables/mozilla/bugs/bug2479-2-expected.checksum
new file mode 100644 (file)
index 0000000..4672c07
--- /dev/null
@@ -0,0 +1 @@
+da535356547f5106ccd17544c5df3d28
\ No newline at end of file
diff --git a/LayoutTests/platform/mac/tables/mozilla/bugs/bug2479-2-expected.png b/LayoutTests/platform/mac/tables/mozilla/bugs/bug2479-2-expected.png
new file mode 100644 (file)
index 0000000..9eb6054
Binary files /dev/null and b/LayoutTests/platform/mac/tables/mozilla/bugs/bug2479-2-expected.png differ
@@ -3,7 +3,13 @@ layer at (0,0) size 800x600
 layer at (0,0) size 800x574
   RenderBlock {HTML} at (0,0) size 800x574 [bgcolor=#00FF99]
     RenderBody {BODY} at (57,64) size 686x446 [bgcolor=#99FFCC] [border: (3px solid #000000)]
-      RenderTable at (3,3) size 308x19
+      RenderBlock {P} at (3,3) size 680x64
+        RenderText {#text} at (0,6) size 636x51
+          text run at (0,6) width 597: "These links are in an element with display table-row, and the li elements "
+          text run at (597,6) width 39: "have"
+          text run at (0,38) width 153: "display: table-cell. "
+          text run at (153,38) width 131: "They disappear:"
+      RenderTable at (3,67) size 308x19
         RenderTableSection (anonymous) at (0,0) size 308x19
           RenderTableRow {UL} at (0,0) size 308x19 [bgcolor=#00CC99]
             RenderTableCell {LI} at (0,0) size 77x19 [r=0 c=0 rs=1 cs=1]
@@ -22,12 +28,6 @@ layer at (0,0) size 800x574
               RenderInline {A} at (0,0) size 35x19 [color=#0000EE]
                 RenderText {#text} at (21,0) size 35x19
                   text run at (21,0) width 35: "links"
-      RenderBlock {P} at (3,22) size 680x64
-        RenderText {#text} at (0,6) size 636x51
-          text run at (0,6) width 597: "These links are in an element with display table-row, and the li elements "
-          text run at (597,6) width 39: "have"
-          text run at (0,38) width 153: "display: table-cell. "
-          text run at (153,38) width 131: "They disappear:"
       RenderBlock {DIV} at (3,86) size 680x357
         RenderBlock (anonymous) at (16,16) size 648x19
           RenderText {#text} at (0,0) size 514x19
index e601494..4ae5a94 100644 (file)
@@ -1,8 +1,8 @@
-layer at (0,0) size 785x2526
+layer at (0,0) size 785x2562
   RenderView at (0,0) size 785x600
-layer at (0,0) size 785x2526
-  RenderBlock {HTML} at (0,0) size 785x2526
-    RenderBody {BODY} at (8,21) size 769x2489 [bgcolor=#FFFFFF]
+layer at (0,0) size 785x2562
+  RenderBlock {HTML} at (0,0) size 785x2562
+    RenderBody {BODY} at (8,21) size 769x2525 [bgcolor=#FFFFFF]
       RenderBlock {H1} at (0,0) size 769x37
         RenderText {#text} at (0,0) size 365x37
           text run at (0,0) width 365: "Table Inheritance Tests - 1"
@@ -143,23 +143,23 @@ layer at (0,0) size 785x2526
                 RenderText {#text} at (0,0) size 716x74
                   text run at (0,0) width 716: "This text should be white on black, bold, twice"
                   text run at (0,37) width 587: "the size of normal text, and sans serif."
-      RenderTable at (0,1782) size 769x74
+      RenderBlock {H3} at (0,1800) size 769x22
+        RenderText {#text} at (0,0) size 248x22
+          text run at (0,0) width 248: "4.2 Styles set directly on ROW"
+      RenderTable at (0,1840) size 769x74
         RenderTableSection (anonymous) at (0,0) size 769x74
           RenderTableRow {DIV} at (0,0) size 769x74 [color=#FFFFFF] [bgcolor=#000000]
             RenderTableCell (anonymous) at (0,0) size 769x74 [r=0 c=0 rs=1 cs=1]
               RenderText {#text} at (0,0) size 716x74
                 text run at (0,0) width 716: "This text should be white on black, bold, twice"
                 text run at (0,37) width 587: "the size of normal text, and sans serif."
-      RenderBlock {H3} at (0,1874) size 769x22
-        RenderText {#text} at (0,0) size 248x22
-          text run at (0,0) width 248: "4.2 Styles set directly on ROW"
-      RenderBlock {H2} at (0,1915) size 769x28
+      RenderBlock {H2} at (0,1933) size 769x28
         RenderText {#text} at (0,0) size 369x28
           text run at (0,0) width 369: "5. Implicit CSS Tables (cell explicit)"
-      RenderBlock {H3} at (0,1962) size 769x22
+      RenderBlock {H3} at (0,1980) size 769x22
         RenderText {#text} at (0,0) size 250x22
           text run at (0,0) width 250: "5.1 Inheritance through a DIV"
-      RenderBlock {DIV} at (0,2002) size 769x74 [color=#FFFFFF] [bgcolor=#000000]
+      RenderBlock {DIV} at (0,2020) size 769x74 [color=#FFFFFF] [bgcolor=#000000]
         RenderTable at (0,0) size 769x74
           RenderTableSection (anonymous) at (0,0) size 769x74
             RenderTableRow (anonymous) at (0,0) size 769x74
@@ -167,20 +167,20 @@ layer at (0,0) size 785x2526
                 RenderText {#text} at (0,0) size 716x74
                   text run at (0,0) width 716: "This text should be white on black, bold, twice"
                   text run at (0,37) width 587: "the size of normal text, and sans serif."
-      RenderTable at (0,2076) size 769x74
+      RenderBlock {H3} at (0,2112) size 769x22
+        RenderText {#text} at (0,0) size 253x22
+          text run at (0,0) width 253: "5.2 Styles set directly on CELL"
+      RenderTable at (0,2152) size 769x74
         RenderTableSection (anonymous) at (0,0) size 769x74
           RenderTableRow (anonymous) at (0,0) size 769x74
             RenderTableCell {DIV} at (0,0) size 769x74 [color=#FFFFFF] [bgcolor=#000000] [r=0 c=0 rs=1 cs=1]
               RenderText {#text} at (0,0) size 716x74
                 text run at (0,0) width 716: "This text should be white on black, bold, twice"
                 text run at (0,37) width 587: "the size of normal text, and sans serif."
-      RenderBlock {H3} at (0,2168) size 769x22
-        RenderText {#text} at (0,0) size 253x22
-          text run at (0,0) width 253: "5.2 Styles set directly on CELL"
-      RenderBlock {H2} at (0,2209) size 769x28
+      RenderBlock {H2} at (0,2245) size 769x28
         RenderText {#text} at (0,0) size 154x28
           text run at (0,0) width 154: "Submit Results"
-      RenderBlock {FORM} at (0,2256) size 769x45
+      RenderBlock {FORM} at (0,2292) size 769x45
         RenderBlock {P} at (0,0) size 769x45
           RenderText {#text} at (0,1) size 263x18
             text run at (0,1) width 263: "How does your browser fare on this test? "
@@ -200,8 +200,8 @@ layer at (0,0) size 785x2526
               RenderText at (0,0) size 38x13
                 text run at (0,0) width 38: "Submit"
           RenderText {#text} at (0,0) size 0x0
-      RenderBlock {HR} at (0,2317) size 769x2 [border: (1px inset #000000)]
-      RenderBlock {P} at (0,2335) size 769x18
+      RenderBlock {HR} at (0,2353) size 769x2 [border: (1px inset #000000)]
+      RenderBlock {P} at (0,2371) size 769x18
         RenderText {#text} at (0,0) size 63x18
           text run at (0,0) width 63: "Up to the "
         RenderInline {A} at (0,0) size 73x18 [color=#0000EE]
@@ -209,7 +209,7 @@ layer at (0,0) size 785x2526
             text run at (63,0) width 73: "Table Tests"
         RenderText {#text} at (136,0) size 4x18
           text run at (136,0) width 4: "."
-      RenderBlock {P} at (0,2369) size 769x18
+      RenderBlock {P} at (0,2405) size 769x18
         RenderText {#text} at (0,0) size 63x18
           text run at (0,0) width 63: "Up to the "
         RenderInline {A} at (0,0) size 98x18 [color=#0000EE]
@@ -217,13 +217,13 @@ layer at (0,0) size 785x2526
             text run at (63,0) width 98: "Evil Tests Page"
         RenderText {#text} at (161,0) size 4x18
           text run at (161,0) width 4: "."
-      RenderBlock {P} at (0,2403) size 769x18
+      RenderBlock {P} at (0,2439) size 769x18
         RenderText {#text} at (0,0) size 61x18
           text run at (0,0) width 61: "Bugzilla: "
         RenderInline {A} at (0,0) size 63x18 [color=#0000EE]
           RenderText {#text} at (61,0) size 63x18
             text run at (61,0) width 63: "Bug 1044"
-      RenderBlock {P} at (0,2437) size 769x18
+      RenderBlock {P} at (0,2473) size 769x18
         RenderText {#text} at (0,0) size 173x18
           text run at (0,0) width 173: "This page is maintained by "
         RenderInline {A} at (0,0) size 77x18 [color=#0000EE]
@@ -236,8 +236,8 @@ layer at (0,0) size 785x2526
             text run at (259,0) width 123: "py8ieh@bath.ac.uk"
         RenderText {#text} at (382,0) size 9x18
           text run at (382,0) width 9: ")."
-      RenderBlock {P} at (0,2471) size 769x18
+      RenderBlock {P} at (0,2507) size 769x18
         RenderText {#text} at (0,0) size 182x18
           text run at (0,0) width 182: "Last updated in March 1999."
-layer at (83,2304) size 142x13
+layer at (83,2340) size 142x13
   RenderBlock {DIV} at (3,3) size 142x13
diff --git a/LayoutTests/platform/mac/tables/mozilla/bugs/bug278266-expected.checksum b/LayoutTests/platform/mac/tables/mozilla/bugs/bug278266-expected.checksum
new file mode 100644 (file)
index 0000000..686341e
--- /dev/null
@@ -0,0 +1 @@
+19c73b73ababb1131e02065a782399ea
\ No newline at end of file
diff --git a/LayoutTests/platform/mac/tables/mozilla/bugs/bug278266-expected.png b/LayoutTests/platform/mac/tables/mozilla/bugs/bug278266-expected.png
new file mode 100644 (file)
index 0000000..ac510fb
Binary files /dev/null and b/LayoutTests/platform/mac/tables/mozilla/bugs/bug278266-expected.png differ
@@ -3,12 +3,12 @@ layer at (0,0) size 800x600
 layer at (0,0) size 800x600
   RenderBlock {HTML} at (0,0) size 800x600
     RenderBody {BODY} at (8,8) size 784x584
-      RenderTable at (0,0) size 118x18
+      RenderBlock (anonymous) at (0,0) size 784x18
+        RenderText {#text} at (0,0) size 229x18
+          text run at (0,0) width 229: "There should be text below this line:"
+      RenderTable at (0,18) size 118x18
         RenderTableSection (anonymous) at (0,0) size 118x18
           RenderTableRow {DIV} at (0,0) size 118x18
             RenderTableCell (anonymous) at (0,0) size 118x18 [r=0 c=0 rs=1 cs=1]
               RenderText {#text} at (0,0) size 118x18
                 text run at (0,0) width 118: "Second line of text"
-      RenderBlock (anonymous) at (0,18) size 784x18
-        RenderText {#text} at (0,0) size 229x18
-          text run at (0,0) width 229: "There should be text below this line:"
diff --git a/LayoutTests/platform/mac/tables/mozilla/bugs/bug8411-expected.checksum b/LayoutTests/platform/mac/tables/mozilla/bugs/bug8411-expected.checksum
new file mode 100644 (file)
index 0000000..4f1370a
--- /dev/null
@@ -0,0 +1 @@
+454de5505aec0302ecc49372b1ba9d28
\ No newline at end of file
diff --git a/LayoutTests/platform/mac/tables/mozilla/bugs/bug8411-expected.png b/LayoutTests/platform/mac/tables/mozilla/bugs/bug8411-expected.png
new file mode 100644 (file)
index 0000000..5bb7d04
Binary files /dev/null and b/LayoutTests/platform/mac/tables/mozilla/bugs/bug8411-expected.png differ
diff --git a/LayoutTests/platform/mac/tables/mozilla/bugs/bug8411-expected.txt b/LayoutTests/platform/mac/tables/mozilla/bugs/bug8411-expected.txt
new file mode 100644 (file)
index 0000000..d4d71ab
--- /dev/null
@@ -0,0 +1,19 @@
+layer at (0,0) size 800x600
+  RenderView at (0,0) size 800x600
+layer at (0,0) size 800x36
+  RenderBlock {doc} at (0,0) size 800x36
+    RenderInline {p} at (0,0) size 738x18
+      RenderText {#text} at (0,0) size 738x18
+        text run at (0,0) width 738: "The following text is in a caption element whose style is set to display:table-caption. The element is not inside a table."
+    RenderText {#text} at (738,0) size 4x18
+      text run at (738,0) width 4: " "
+    RenderInline {captionelement} at (0,0) size 795x36
+      RenderText {#text} at (742,0) size 795x36
+        text run at (742,0) width 53: "This is a"
+        text run at (0,18) width 103: "caption element."
+    RenderText {#text} at (103,18) size 4x18
+      text run at (103,18) width 4: " "
+    RenderInline {p} at (0,0) size 549x18
+      RenderText {#text} at (107,18) size 549x18
+        text run at (107,18) width 549: "If there's no red text between this text and the previous text the caption is not displayed."
+    RenderText {#text} at (0,0) size 0x0
diff --git a/LayoutTests/platform/mac/tables/mozilla_expected_failures/bugs/bug2479-2-expected.checksum b/LayoutTests/platform/mac/tables/mozilla_expected_failures/bugs/bug2479-2-expected.checksum
deleted file mode 100644 (file)
index 382e979..0000000
+++ /dev/null
@@ -1 +0,0 @@
-aa62c38a5e21bd2763a485890f4e001a
\ No newline at end of file
diff --git a/LayoutTests/platform/mac/tables/mozilla_expected_failures/bugs/bug2479-2-expected.png b/LayoutTests/platform/mac/tables/mozilla_expected_failures/bugs/bug2479-2-expected.png
deleted file mode 100644 (file)
index 32e741e..0000000
Binary files a/LayoutTests/platform/mac/tables/mozilla_expected_failures/bugs/bug2479-2-expected.png and /dev/null differ
diff --git a/LayoutTests/platform/mac/tables/mozilla_expected_failures/bugs/bug278266-expected.checksum b/LayoutTests/platform/mac/tables/mozilla_expected_failures/bugs/bug278266-expected.checksum
deleted file mode 100644 (file)
index cd0cf3a..0000000
+++ /dev/null
@@ -1 +0,0 @@
-896b5f70044f7711deedee3072300ad1
\ No newline at end of file
diff --git a/LayoutTests/platform/mac/tables/mozilla_expected_failures/bugs/bug278266-expected.png b/LayoutTests/platform/mac/tables/mozilla_expected_failures/bugs/bug278266-expected.png
deleted file mode 100644 (file)
index 3734bfa..0000000
Binary files a/LayoutTests/platform/mac/tables/mozilla_expected_failures/bugs/bug278266-expected.png and /dev/null differ
diff --git a/LayoutTests/platform/mac/tables/mozilla_expected_failures/bugs/bug8411-expected.checksum b/LayoutTests/platform/mac/tables/mozilla_expected_failures/bugs/bug8411-expected.checksum
deleted file mode 100644 (file)
index 0587be8..0000000
+++ /dev/null
@@ -1 +0,0 @@
-6c08177009cda25160419d3e15ca3a0a
\ No newline at end of file
diff --git a/LayoutTests/platform/mac/tables/mozilla_expected_failures/bugs/bug8411-expected.png b/LayoutTests/platform/mac/tables/mozilla_expected_failures/bugs/bug8411-expected.png
deleted file mode 100644 (file)
index 3fe4747..0000000
Binary files a/LayoutTests/platform/mac/tables/mozilla_expected_failures/bugs/bug8411-expected.png and /dev/null differ
diff --git a/LayoutTests/platform/mac/tables/mozilla_expected_failures/bugs/bug8411-expected.txt b/LayoutTests/platform/mac/tables/mozilla_expected_failures/bugs/bug8411-expected.txt
deleted file mode 100644 (file)
index e57e8d9..0000000
+++ /dev/null
@@ -1,16 +0,0 @@
-layer at (0,0) size 800x600
-  RenderView at (0,0) size 800x600
-layer at (5,5) size 790x90
-  RenderBlock {doc} at (5,5) size 790x90
-    RenderTable at (0,0) size 53x54
-      RenderBlock {captionelement} at (0,0) size 53x54 [color=#FF0000]
-        RenderText {#text} at (0,0) size 53x54
-          text run at (0,0) width 53: "This is a"
-          text run at (3,18) width 46: "caption"
-          text run at (0,36) width 53: "element."
-    RenderBlock {p} at (0,54) size 790x18
-      RenderText {#text} at (0,0) size 738x18
-        text run at (0,0) width 738: "The following text is in a caption element whose style is set to display:table-caption. The element is not inside a table."
-    RenderBlock {p} at (0,72) size 790x18
-      RenderText {#text} at (0,0) size 549x18
-        text run at (0,0) width 549: "If there's no red text between this text and the previous text the caption is not displayed."
index 5acd874..08ba10a 100644 (file)
@@ -1,8 +1,8 @@
-layer at (0,0) size 785x2416
+layer at (0,0) size 785x2452
   RenderView at (0,0) size 785x600
-layer at (0,0) size 785x2416
-  RenderBlock {HTML} at (0,0) size 785x2416
-    RenderBody {BODY} at (8,8) size 769x2392
+layer at (0,0) size 785x2452
+  RenderBlock {HTML} at (0,0) size 785x2452
+    RenderBody {BODY} at (8,8) size 769x2428
       RenderBlock {H1} at (0,0) size 769x37
         RenderText {#text} at (0,0) size 432x37
           text run at (0,0) width 432: "Example 4: Some simple tables."
@@ -420,7 +420,11 @@ layer at (0,0) size 785x2416
         RenderBlock {CAPTION} at (0,0) size 190x18
           RenderText {#text} at (2,0) size 186x18
             text run at (2,0) width 186: "Table 5 has a scrolling tbody."
-      RenderTable at (0,1204) size 204x18
+      RenderBlock (anonymous) at (0,1204) size 769x36
+        RenderBR {BR} at (0,0) size 0x18
+        RenderText {#text} at (0,18) size 614x18
+          text run at (0,18) width 614: "This is a table formed from a list with display of table-row and list items with display of table-cell."
+      RenderTable at (0,1240) size 204x18
         RenderTableSection (anonymous) at (0,0) size 204x18
           RenderTableRow {UL} at (0,0) size 204x18 [bgcolor=#FFA500]
             RenderTableCell {LI} at (0,0) size 34x18 [r=0 c=0 rs=1 cs=1]
@@ -438,7 +442,12 @@ layer at (0,0) size 785x2416
             RenderTableCell {LI} at (168,0) size 36x18 [r=0 c=4 rs=1 cs=1]
               RenderText {#text} at (0,0) size 36x18
                 text run at (0,0) width 36: "FIVE"
-      RenderTable at (0,1222) size 230x18
+      RenderBlock (anonymous) at (0,1258) size 769x36
+        RenderBR {BR} at (0,0) size 0x18
+        RenderText {#text} at (0,18) size 660x18
+          text run at (0,18) width 660: "This is a table formed from a list with display of table-row-group and list items with display of table-cell. "
+        RenderBR {BR} at (0,0) size 0x0
+      RenderTable at (0,1294) size 230x18
         RenderTableSection {UL} at (0,0) size 230x18
           RenderTableRow (anonymous) at (0,0) size 230x18
             RenderTableCell {LI} at (0,0) size 34x18 [bgcolor=#FFA500] [r=0 c=0 rs=1 cs=1]
@@ -459,7 +468,11 @@ layer at (0,0) size 785x2416
             RenderTableCell {LI} at (204,0) size 26x18 [bgcolor=#FFA500] [r=0 c=5 rs=1 cs=1]
               RenderText {#text} at (0,0) size 26x18
                 text run at (0,0) width 26: "SIX"
-      RenderTable at (0,1240) size 230x18
+      RenderBlock (anonymous) at (0,1312) size 769x36
+        RenderBR {BR} at (0,0) size 0x18
+        RenderText {#text} at (0,18) size 407x18
+          text run at (0,18) width 407: "This is like the previous table plus the list's overflow property set"
+      RenderTable at (0,1348) size 230x18
         RenderTableSection {UL} at (0,0) size 230x18
           RenderTableRow (anonymous) at (0,0) size 230x18
             RenderTableCell {LI} at (0,0) size 34x18 [bgcolor=#FFA500] [r=0 c=0 rs=1 cs=1]
@@ -480,19 +493,9 @@ layer at (0,0) size 785x2416
             RenderTableCell {LI} at (204,0) size 26x18 [bgcolor=#FFA500] [r=0 c=5 rs=1 cs=1]
               RenderText {#text} at (0,0) size 26x18
                 text run at (0,0) width 26: "SIX"
-      RenderBlock (anonymous) at (0,1258) size 769x90
+      RenderBlock (anonymous) at (0,1366) size 769x18
         RenderBR {BR} at (0,0) size 0x18
-        RenderText {#text} at (0,18) size 618x18
-          text run at (0,18) width 618: "This is a table formed from a list with display of table-row and list items with display of table-cell. "
-        RenderBR {BR} at (618,32) size 0x0
-        RenderText {#text} at (0,36) size 660x18
-          text run at (0,36) width 660: "This is a table formed from a list with display of table-row-group and list items with display of table-cell. "
-        RenderBR {BR} at (0,0) size 0x0
-        RenderBR {BR} at (0,54) size 0x18
-        RenderText {#text} at (0,72) size 411x18
-          text run at (0,72) width 411: "This is like the previous table plus the list's overflow property set "
-        RenderBR {BR} at (411,86) size 0x0
-      RenderBlock {P} at (0,1364) size 769x224
+      RenderBlock {P} at (0,1400) size 769x224
         RenderBlock (anonymous) at (0,0) size 769x18
           RenderText {#text} at (0,0) size 430x18
             text run at (0,0) width 430: "The following table will have its rows and columns in red collapsed "
@@ -579,7 +582,7 @@ layer at (0,0) size 785x2416
               RenderTableCell {TD} at (68,50) size 66x22 [border: (1px inset #808080)] [r=2 c=2 rs=1 cs=1]
                 RenderText {#text} at (2,2) size 27x18
                   text run at (2,2) width 27: "C33"
-      RenderBlock {P} at (0,1604) size 769x234
+      RenderBlock {P} at (0,1640) size 769x234
         RenderBlock (anonymous) at (0,0) size 769x18
           RenderText {#text} at (0,0) size 750x18
             text run at (0,0) width 750: "The following table will have its 2nd row and 2nd col collapsed. A window resize may be necessary to see it properly. "
@@ -692,7 +695,7 @@ layer at (0,0) size 785x2416
               RenderTableCell {TD} at (155,66) size 31x22 [border: (1px inset #808080)] [r=3 c=3 rs=1 cs=1]
                 RenderText {#text} at (2,2) size 27x18
                   text run at (2,2) width 27: "C44"
-      RenderBlock {P} at (0,1854) size 769x270
+      RenderBlock {P} at (0,1890) size 769x270
         RenderBlock (anonymous) at (0,0) size 769x18
           RenderText {#text} at (0,0) size 459x18
             text run at (0,0) width 459: "The following table will have its 1st row group collapsed (rows 1 and 2) "
@@ -819,7 +822,7 @@ layer at (0,0) size 785x2416
                   text run at (2,2) width 27: "C44"
         RenderBlock (anonymous) at (0,252) size 769x18
           RenderBR {BR} at (0,0) size 0x18
-      RenderBlock {P} at (0,2140) size 769x252
+      RenderBlock {P} at (0,2176) size 769x252
         RenderBlock (anonymous) at (0,0) size 769x36
           RenderText {#text} at (0,0) size 755x36
             text run at (0,0) width 554: "The following table is similar to a previous table except that the direction is right-to-left. "
index db40bc7..a62d28e 100644 (file)
@@ -1,3 +1,19 @@
+2008-06-19  Dan Bernstein  <mitz@apple.com>
+
+        Reviewed by John Sullivan.
+
+        - fix a bug where anonymous tables were inserted in the wrong place
+
+        Tests: tables/mozilla/bugs/bug2479-2.html
+               tables/mozilla/bugs/bug278266.html
+               tables/mozilla/bugs/bug8411.xml
+
+        * rendering/RenderContainer.cpp:
+        (WebCore::RenderContainer::addChild): Changed to create an anonymous
+        table and insert it before beforeChild unless the insertion point is
+        right after an existing anonymous table, in which case the existing
+        table is used.
+
 2008-06-19  Justin Garcia  <justin.garcia@apple.com>
 
         Reviewed by Oliver.
index 6e8bb15..79e0268 100644 (file)
@@ -107,11 +107,10 @@ void RenderContainer::addChild(RenderObject* newChild, RenderObject* beforeChild
     }
 
     if (needsTable) {
-        RenderTable *table;
-        if(!beforeChild)
-            beforeChild = m_lastChild;
-        if(beforeChild && beforeChild->isAnonymous() && beforeChild->isTable())
-            table = static_cast<RenderTable*>(beforeChild);
+        RenderTable* table;
+        RenderObject* afterChild = beforeChild ? beforeChild->previousSibling() : m_lastChild;
+        if (afterChild && afterChild->isAnonymous() && afterChild->isTable())
+            table = static_cast<RenderTable*>(afterChild);
         else {
             table = new (renderArena()) RenderTable(document() /* is anonymous */);
             RenderStyle *newStyle = new (renderArena()) RenderStyle;