2007-01-15 Mitz Pettel <mitz@webkit.org>
authorap <ap@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 15 Jan 2007 18:40:08 +0000 (18:40 +0000)
committerap <ap@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 15 Jan 2007 18:40:08 +0000 (18:40 +0000)
        Reviewed by Darin.

        - fix http://bugs.webkit.org/show_bug.cgi?id=8604
          rowspan does not work when set through javascript

WebCore:
        * rendering/RenderTableCell.cpp:
        (WebCore::RenderTableCell::updateFromElement): Mark the section for grid recalc
        if row span or column span change.

LayoutTests:
        * fast/dom/HTMLTableElement/colSpan-expected.checksum: Updated.
        * fast/dom/HTMLTableElement/colSpan-expected.png: Updated.
        * fast/dom/HTMLTableElement/colSpan-expected.txt: Updated.
        * fast/dom/HTMLTableElement/colSpan.html: Removed comment about expected
        failure.
        * tables/mozilla/bugs/bug22246-2-expected.checksum: Added.
        * tables/mozilla/bugs/bug22246-2-expected.png: Added.
        * tables/mozilla/bugs/bug22246-2-expected.txt: Added.
        * tables/mozilla/bugs/bug22246-2a-expected.checksum: Added.
        * tables/mozilla/bugs/bug22246-2a-expected.png: Added.
        * tables/mozilla/bugs/bug22246-2a-expected.txt: Added.
        * tables/mozilla/bugs/bug22246-3-expected.checksum: Added.
        * tables/mozilla/bugs/bug22246-3-expected.png: Added.
        * tables/mozilla/bugs/bug22246-3-expected.txt: Added.
        * tables/mozilla/bugs/bug22246-3a-expected.checksum: Added.
        * tables/mozilla/bugs/bug22246-3a-expected.png: Added.
        * tables/mozilla/bugs/bug22246-3a-expected.txt: Added.
        * tables/mozilla_expected_failures/bugs/bug22246-2-expected.checksum: Removed.
        * tables/mozilla_expected_failures/bugs/bug22246-2-expected.png: Removed.
        * tables/mozilla_expected_failures/bugs/bug22246-2-expected.txt: Removed.
        * tables/mozilla_expected_failures/bugs/bug22246-2.html: Moved to mozilla/ since it
        no longer fails.
        * tables/mozilla_expected_failures/bugs/bug22246-2a-expected.checksum: Removed.
        * tables/mozilla_expected_failures/bugs/bug22246-2a-expected.png: Removed.
        * tables/mozilla_expected_failures/bugs/bug22246-2a-expected.txt: Removed.
        * tables/mozilla_expected_failures/bugs/bug22246-2a.html: Moved to mozilla/ since it
        no longer fails.
        * tables/mozilla_expected_failures/bugs/bug22246-3-expected.checksum: Removed.
        * tables/mozilla_expected_failures/bugs/bug22246-3-expected.png: Removed.
        * tables/mozilla_expected_failures/bugs/bug22246-3-expected.txt: Removed.
        * tables/mozilla_expected_failures/bugs/bug22246-3.html: Moved to mozilla/ since it
        no longer fails.
        * tables/mozilla_expected_failures/bugs/bug22246-3a-expected.checksum: Removed.
        * tables/mozilla_expected_failures/bugs/bug22246-3a-expected.png: Removed.
        * tables/mozilla_expected_failures/bugs/bug22246-3a-expected.txt: Removed.
        * tables/mozilla_expected_failures/bugs/bug22246-3a.html: Moved to mozilla/ since it
        no longer fails.

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

31 files changed:
LayoutTests/ChangeLog
LayoutTests/fast/dom/HTMLTableElement/colSpan-expected.checksum
LayoutTests/fast/dom/HTMLTableElement/colSpan-expected.png
LayoutTests/fast/dom/HTMLTableElement/colSpan-expected.txt
LayoutTests/fast/dom/HTMLTableElement/colSpan.html
LayoutTests/tables/mozilla/bugs/bug22246-2-expected.checksum [new file with mode: 0644]
LayoutTests/tables/mozilla/bugs/bug22246-2-expected.png [new file with mode: 0644]
LayoutTests/tables/mozilla/bugs/bug22246-2-expected.txt [moved from LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-2-expected.txt with 82% similarity]
LayoutTests/tables/mozilla/bugs/bug22246-2.html [moved from LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-2.html with 100% similarity]
LayoutTests/tables/mozilla/bugs/bug22246-2a-expected.checksum [new file with mode: 0644]
LayoutTests/tables/mozilla/bugs/bug22246-2a-expected.png [new file with mode: 0644]
LayoutTests/tables/mozilla/bugs/bug22246-2a-expected.txt [moved from LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-2a-expected.txt with 82% similarity]
LayoutTests/tables/mozilla/bugs/bug22246-2a.html [moved from LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-2a.html with 100% similarity]
LayoutTests/tables/mozilla/bugs/bug22246-3-expected.checksum [new file with mode: 0644]
LayoutTests/tables/mozilla/bugs/bug22246-3-expected.png [new file with mode: 0644]
LayoutTests/tables/mozilla/bugs/bug22246-3-expected.txt [moved from LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-3-expected.txt with 75% similarity]
LayoutTests/tables/mozilla/bugs/bug22246-3.html [moved from LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-3.html with 100% similarity]
LayoutTests/tables/mozilla/bugs/bug22246-3a-expected.checksum [new file with mode: 0644]
LayoutTests/tables/mozilla/bugs/bug22246-3a-expected.png [new file with mode: 0644]
LayoutTests/tables/mozilla/bugs/bug22246-3a-expected.txt [moved from LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-3a-expected.txt with 75% similarity]
LayoutTests/tables/mozilla/bugs/bug22246-3a.html [moved from LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-3a.html with 100% similarity]
LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-2-expected.checksum [deleted file]
LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-2-expected.png [deleted file]
LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-2a-expected.checksum [deleted file]
LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-2a-expected.png [deleted file]
LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-3-expected.checksum [deleted file]
LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-3-expected.png [deleted file]
LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-3a-expected.checksum [deleted file]
LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-3a-expected.png [deleted file]
WebCore/ChangeLog
WebCore/rendering/RenderTableCell.cpp

index 86117c1..f7bf352 100644 (file)
@@ -1,3 +1,48 @@
+2007-01-15  Mitz Pettel  <mitz@webkit.org>
+
+        Reviewed by Darin.
+
+        - updates for http://bugs.webkit.org/show_bug.cgi?id=8604
+          rowspan does not work when set through javascript
+
+        * fast/dom/HTMLTableElement/colSpan-expected.checksum: Updated.
+        * fast/dom/HTMLTableElement/colSpan-expected.png: Updated.
+        * fast/dom/HTMLTableElement/colSpan-expected.txt: Updated.
+        * fast/dom/HTMLTableElement/colSpan.html: Removed comment about expected
+        failure.
+        * tables/mozilla/bugs/bug22246-2-expected.checksum: Added.
+        * tables/mozilla/bugs/bug22246-2-expected.png: Added.
+        * tables/mozilla/bugs/bug22246-2-expected.txt: Added.
+        * tables/mozilla/bugs/bug22246-2a-expected.checksum: Added.
+        * tables/mozilla/bugs/bug22246-2a-expected.png: Added.
+        * tables/mozilla/bugs/bug22246-2a-expected.txt: Added.
+        * tables/mozilla/bugs/bug22246-3-expected.checksum: Added.
+        * tables/mozilla/bugs/bug22246-3-expected.png: Added.
+        * tables/mozilla/bugs/bug22246-3-expected.txt: Added.
+        * tables/mozilla/bugs/bug22246-3a-expected.checksum: Added.
+        * tables/mozilla/bugs/bug22246-3a-expected.png: Added.
+        * tables/mozilla/bugs/bug22246-3a-expected.txt: Added.
+        * tables/mozilla_expected_failures/bugs/bug22246-2-expected.checksum: Removed.
+        * tables/mozilla_expected_failures/bugs/bug22246-2-expected.png: Removed.
+        * tables/mozilla_expected_failures/bugs/bug22246-2-expected.txt: Removed.
+        * tables/mozilla_expected_failures/bugs/bug22246-2.html: Moved to mozilla/ since it
+        no longer fails.
+        * tables/mozilla_expected_failures/bugs/bug22246-2a-expected.checksum: Removed.
+        * tables/mozilla_expected_failures/bugs/bug22246-2a-expected.png: Removed.
+        * tables/mozilla_expected_failures/bugs/bug22246-2a-expected.txt: Removed.
+        * tables/mozilla_expected_failures/bugs/bug22246-2a.html: Moved to mozilla/ since it
+        no longer fails.
+        * tables/mozilla_expected_failures/bugs/bug22246-3-expected.checksum: Removed.
+        * tables/mozilla_expected_failures/bugs/bug22246-3-expected.png: Removed.
+        * tables/mozilla_expected_failures/bugs/bug22246-3-expected.txt: Removed.
+        * tables/mozilla_expected_failures/bugs/bug22246-3.html: Moved to mozilla/ since it
+        no longer fails.
+        * tables/mozilla_expected_failures/bugs/bug22246-3a-expected.checksum: Removed.
+        * tables/mozilla_expected_failures/bugs/bug22246-3a-expected.png: Removed.
+        * tables/mozilla_expected_failures/bugs/bug22246-3a-expected.txt: Removed.
+        * tables/mozilla_expected_failures/bugs/bug22246-3a.html: Moved to mozilla/ since it
+        no longer fails.
+
 2007-01-15  Alexey Proskuryakov  <ap@webkit.org>
 
         Reviewed by Darin.
index b78f163..00ab7d3 100644 (file)
@@ -1 +1 @@
-053262cc3676d13671da1339703760af
\ No newline at end of file
+d708f0feec1cb7eae98408a3280ccb83
\ No newline at end of file
index 58ea1fb..0b31992 100644 (file)
Binary files a/LayoutTests/fast/dom/HTMLTableElement/colSpan-expected.png and b/LayoutTests/fast/dom/HTMLTableElement/colSpan-expected.png differ
index 8b0ec91..ef0f811 100644 (file)
@@ -7,13 +7,7 @@ layer at (0,0) size 800x600
         RenderText {#text} at (0,0) size 167x18
           text run at (0,0) width 167: "Tests: the colspan attribute"
         RenderBR {BR} at (167,14) size 0x0
-      RenderBlock (anonymous) at (0,34) size 784x18
-        RenderInline {B} at (0,0) size 180x18
-          RenderText {#text} at (0,0) size 180x18
-            text run at (0,0) width 180: "This test is expected to fail."
-        RenderText {#text} at (0,0) size 0x0
-      RenderBlock {P} at (0,68) size 784x0
-      RenderBlock {P} at (0,68) size 784x72
+      RenderBlock {P} at (0,34) size 784x72
         RenderText {#text} at (0,0) size 73x18
           text run at (0,0) width 73: "Conditions:"
         RenderBR {BR} at (73,14) size 0x0
@@ -23,16 +17,16 @@ layer at (0,0) size 800x600
           text run at (0,36) width 514: "(\"1\"). The value zero (\"0\") means that the cell spans all columns from the current "
           text run at (514,36) width 253: "column to the last column of the column"
           text run at (0,54) width 317: "group (COLGROUP) in which the cell is defined."
-      RenderBlock {P} at (0,156) size 784x54
+      RenderBlock {P} at (0,122) size 784x54
         RenderText {#text} at (0,0) size 782x54
           text run at (0,0) width 759: "The first table tests the default value of one, and an input of 7, which is within the normal range of expect input. The first"
           text run at (0,18) width 709: "column in the first row should span only column \"one\" and the second cell should span all of the other columns. "
           text run at (709,18) width 73: "The second"
           text run at (0,36) width 520: "table tests the zero value. \"Just A\" should span \"A\" and \"Just B\" should span \"B.\""
-      RenderBlock {HR} at (0,226) size 784x2 [border: (1px inset #000000)]
-      RenderBlock (anonymous) at (0,236) size 784x18
+      RenderBlock {HR} at (0,192) size 784x2 [border: (1px inset #000000)]
+      RenderBlock (anonymous) at (0,202) size 784x18
         RenderBR {BR} at (0,0) size 0x18
-      RenderTable {TABLE} at (0,254) size 270x70 [border: (1px outset #808080)]
+      RenderTable {TABLE} at (0,220) size 270x70 [border: (1px outset #808080)]
         RenderBlock {CAPTION} at (0,0) size 268x18
           RenderText {#text} at (26,0) size 216x18
             text run at (26,0) width 216: "Testing Default and Regular Input"
@@ -41,7 +35,7 @@ layer at (0,0) size 800x600
             RenderTableCell {TD} at (2,2) size 32x22 [border: (1px inset #808080)] [r=0 c=0 rs=1 cs=1]
               RenderText {#text} at (2,2) size 28x18
                 text run at (2,2) width 28: "First"
-            RenderTableCell {TD} at (135,2) size 131x22 [border: (1px inset #808080)] [r=0 c=4 rs=1 cs=7]
+            RenderTableCell {TD} at (36,2) size 230x22 [border: (1px inset #808080)] [r=0 c=1 rs=1 cs=7]
               RenderText {#text} at (2,2) size 75x18
                 text run at (2,2) width 75: "All the Rest"
           RenderTableRow {TR} at (0,26) size 268x22
@@ -69,42 +63,42 @@ layer at (0,0) size 800x600
             RenderTableCell {TD} at (231,26) size 35x22 [border: (1px inset #808080)] [r=1 c=7 rs=1 cs=1]
               RenderText {#text} at (2,2) size 31x18
                 text run at (2,2) width 31: "eight"
-      RenderBlock (anonymous) at (0,324) size 784x18
+      RenderBlock (anonymous) at (0,290) size 784x18
         RenderBR {BR} at (0,0) size 0x18
-      RenderTable {TABLE} at (0,342) size 198x70 [border: (1px outset #808080)]
-        RenderBlock {CAPTION} at (0,0) size 196x18
-          RenderText {#text} at (15,0) size 166x18
-            text run at (15,0) width 166: "Testing Zero Special Case"
-        RenderTableSection {TBODY} at (1,19) size 196x50
-          RenderTableRow {TR} at (0,2) size 196x22
+      RenderTable {TABLE} at (0,308) size 197x70 [border: (1px outset #808080)]
+        RenderBlock {CAPTION} at (0,0) size 195x18
+          RenderText {#text} at (14,0) size 166x18
+            text run at (14,0) width 166: "Testing Zero Special Case"
+        RenderTableSection {TBODY} at (1,19) size 195x50
+          RenderTableRow {TR} at (0,2) size 195x22
             RenderTableCell {TD} at (2,2) size 44x22 [border: (1px inset #808080)] [r=0 c=0 rs=1 cs=1]
               RenderText {#text} at (2,2) size 40x18
                 text run at (2,2) width 40: "Just A"
-            RenderTableCell {TD} at (100,2) size 43x22 [border: (1px inset #808080)] [r=0 c=4 rs=1 cs=1]
+            RenderTableCell {TD} at (48,2) size 43x22 [border: (1px inset #808080)] [r=0 c=1 rs=1 cs=1]
               RenderText {#text} at (2,2) size 39x18
                 text run at (2,2) width 39: "Just B"
-          RenderTableRow {TR} at (0,26) size 196x22
+          RenderTableRow {TR} at (0,26) size 195x22
             RenderTableCell {TD} at (2,26) size 44x22 [border: (1px inset #808080)] [r=1 c=0 rs=1 cs=1]
               RenderText {#text} at (2,2) size 12x18
                 text run at (2,2) width 12: "A"
-            RenderTableCell {TD} at (48,26) size 15x22 [border: (1px inset #808080)] [r=1 c=1 rs=1 cs=1]
+            RenderTableCell {TD} at (48,26) size 43x22 [border: (1px inset #808080)] [r=1 c=1 rs=1 cs=1]
               RenderText {#text} at (2,2) size 11x18
                 text run at (2,2) width 11: "B"
-            RenderTableCell {TD} at (65,26) size 15x22 [border: (1px inset #808080)] [r=1 c=2 rs=1 cs=1]
+            RenderTableCell {TD} at (93,26) size 15x22 [border: (1px inset #808080)] [r=1 c=2 rs=1 cs=1]
               RenderText {#text} at (2,2) size 11x18
                 text run at (2,2) width 11: "C"
-            RenderTableCell {TD} at (82,26) size 16x22 [border: (1px inset #808080)] [r=1 c=3 rs=1 cs=1]
+            RenderTableCell {TD} at (110,26) size 16x22 [border: (1px inset #808080)] [r=1 c=3 rs=1 cs=1]
               RenderText {#text} at (2,2) size 12x18
                 text run at (2,2) width 12: "D"
-            RenderTableCell {TD} at (100,26) size 43x22 [border: (1px inset #808080)] [r=1 c=4 rs=1 cs=1]
+            RenderTableCell {TD} at (128,26) size 14x22 [border: (1px inset #808080)] [r=1 c=4 rs=1 cs=1]
               RenderText {#text} at (2,2) size 10x18
                 text run at (2,2) width 10: "E"
-            RenderTableCell {TD} at (145,26) size 13x22 [border: (1px inset #808080)] [r=1 c=5 rs=1 cs=1]
+            RenderTableCell {TD} at (144,26) size 13x22 [border: (1px inset #808080)] [r=1 c=5 rs=1 cs=1]
               RenderText {#text} at (2,2) size 9x18
                 text run at (2,2) width 9: "F"
-            RenderTableCell {TD} at (160,26) size 16x22 [border: (1px inset #808080)] [r=1 c=6 rs=1 cs=1]
+            RenderTableCell {TD} at (159,26) size 16x22 [border: (1px inset #808080)] [r=1 c=6 rs=1 cs=1]
               RenderText {#text} at (2,2) size 12x18
                 text run at (2,2) width 12: "G"
-            RenderTableCell {TD} at (178,26) size 16x22 [border: (1px inset #808080)] [r=1 c=7 rs=1 cs=1]
+            RenderTableCell {TD} at (177,26) size 16x22 [border: (1px inset #808080)] [r=1 c=7 rs=1 cs=1]
               RenderText {#text} at (2,2) size 12x18
                 text run at (2,2) width 12: "H"
index 9f876d2..01ff94e 100644 (file)
@@ -22,8 +22,6 @@ x[1].colSpan="0"
 <body onload = "colSpanOne(); colSpanTwo()">
 <p>Tests: the colspan attribute<br>
 </p>
-<b>This test is expected to fail.</b>
-</p>
 <p>Conditions:<br>
 The colSpan attribute specifies the number of columns spanned by the current cell. The default value of 
 this attribute is one ("1"). The value zero ("0") means that the cell spans all columns from the current 
diff --git a/LayoutTests/tables/mozilla/bugs/bug22246-2-expected.checksum b/LayoutTests/tables/mozilla/bugs/bug22246-2-expected.checksum
new file mode 100644 (file)
index 0000000..afd8bcd
--- /dev/null
@@ -0,0 +1 @@
+a0b7c34ca68bbaf5754d5cb2670213b7
\ No newline at end of file
diff --git a/LayoutTests/tables/mozilla/bugs/bug22246-2-expected.png b/LayoutTests/tables/mozilla/bugs/bug22246-2-expected.png
new file mode 100644 (file)
index 0000000..c3b2801
Binary files /dev/null and b/LayoutTests/tables/mozilla/bugs/bug22246-2-expected.png differ
@@ -6,35 +6,35 @@ layer at (0,0) size 800x600
       RenderBlock (anonymous) at (0,0) size 784x18
         RenderText {#text} at (0,0) size 4x18
           text run at (0,0) width 4: " "
-      RenderTable {TABLE} at (0,18) size 206x82 [border: (1px outset #808080)]
-        RenderTableSection {TBODY} at (1,1) size 204x80
-          RenderTableRow {TR} at (0,2) size 204x24
+      RenderTable {TABLE} at (0,18) size 238x82 [border: (1px outset #808080)]
+        RenderTableSection {TBODY} at (1,1) size 236x80
+          RenderTableRow {TR} at (0,2) size 236x24
             RenderTableCell {TD} at (2,2) size 14x24 [border: (1px inset #808080)] [r=0 c=0 rs=1 cs=1]
               RenderText {#text} at (3,3) size 8x18
                 text run at (3,3) width 8: "1"
-            RenderTableCell {TD} at (18,2) size 136x24 [border: (1px inset #808080)] [r=0 c=1 rs=2 cs=2]
+            RenderTableCell {TD} at (18,15) size 136x24 [border: (1px inset #808080)] [r=0 c=1 rs=2 cs=2]
               RenderText {#text} at (3,3) size 130x18
                 text run at (3,3) width 130: "colspan 2 rowspan 2"
             RenderTableCell {TD} at (156,2) size 22x24 [border: (1px inset #808080)] [r=0 c=3 rs=1 cs=1]
               RenderText {#text} at (3,3) size 8x18
                 text run at (3,3) width 8: "3"
-          RenderTableRow {TR} at (0,28) size 204x24
+          RenderTableRow {TR} at (0,28) size 236x24
             RenderTableCell {TD} at (2,28) size 14x24 [border: (1px inset #808080)] [r=1 c=0 rs=1 cs=1]
               RenderText {#text} at (3,3) size 8x18
                 text run at (3,3) width 8: "4"
-            RenderTableCell {TD} at (18,28) size 67x24 [border: (1px inset #808080)] [r=1 c=1 rs=1 cs=1]
+            RenderTableCell {TD} at (156,28) size 22x24 [border: (1px inset #808080)] [r=1 c=3 rs=1 cs=1]
               RenderText {#text} at (3,3) size 8x18
                 text run at (3,3) width 8: "5"
-            RenderTableCell {TD} at (87,28) size 67x24 [border: (1px inset #808080)] [r=1 c=2 rs=1 cs=1]
+            RenderTableCell {TD} at (180,28) size 22x24 [border: (1px inset #808080)] [r=1 c=4 rs=1 cs=1]
               RenderText {#text} at (3,3) size 8x18
                 text run at (3,3) width 8: "6"
-            RenderTableCell {TD} at (156,28) size 22x24 [border: (1px inset #808080)] [r=1 c=3 rs=1 cs=1]
+            RenderTableCell {TD} at (204,28) size 14x24 [border: (1px inset #808080)] [r=1 c=5 rs=1 cs=1]
               RenderText {#text} at (3,3) size 8x18
                 text run at (3,3) width 8: "7"
-            RenderTableCell {TD} at (180,28) size 22x24 [border: (1px inset #808080)] [r=1 c=4 rs=1 cs=1]
+            RenderTableCell {TD} at (220,28) size 14x24 [border: (1px inset #808080)] [r=1 c=6 rs=1 cs=1]
               RenderText {#text} at (3,3) size 8x18
                 text run at (3,3) width 8: "8"
-          RenderTableRow {TR} at (0,54) size 204x24
+          RenderTableRow {TR} at (0,54) size 236x24
             RenderTableCell {TD} at (2,54) size 14x24 [border: (1px inset #808080)] [r=2 c=0 rs=1 cs=1]
               RenderText {#text} at (3,3) size 8x18
                 text run at (3,3) width 8: "9"
diff --git a/LayoutTests/tables/mozilla/bugs/bug22246-2a-expected.checksum b/LayoutTests/tables/mozilla/bugs/bug22246-2a-expected.checksum
new file mode 100644 (file)
index 0000000..afd8bcd
--- /dev/null
@@ -0,0 +1 @@
+a0b7c34ca68bbaf5754d5cb2670213b7
\ No newline at end of file
diff --git a/LayoutTests/tables/mozilla/bugs/bug22246-2a-expected.png b/LayoutTests/tables/mozilla/bugs/bug22246-2a-expected.png
new file mode 100644 (file)
index 0000000..c3b2801
Binary files /dev/null and b/LayoutTests/tables/mozilla/bugs/bug22246-2a-expected.png differ
@@ -6,35 +6,35 @@ layer at (0,0) size 800x116
       RenderBlock (anonymous) at (0,0) size 784x18
         RenderText {#text} at (0,0) size 4x18
           text run at (0,0) width 4: " "
-      RenderTable {TABLE} at (0,18) size 206x82 [border: (1px outset #808080)]
-        RenderTableSection {TBODY} at (1,1) size 204x80
-          RenderTableRow {TR} at (0,2) size 204x24
+      RenderTable {TABLE} at (0,18) size 238x82 [border: (1px outset #808080)]
+        RenderTableSection {TBODY} at (1,1) size 236x80
+          RenderTableRow {TR} at (0,2) size 236x24
             RenderTableCell {TD} at (2,2) size 14x24 [border: (1px inset #808080)] [r=0 c=0 rs=1 cs=1]
               RenderText {#text} at (3,3) size 8x18
                 text run at (3,3) width 8: "1"
-            RenderTableCell {TD} at (18,2) size 136x24 [border: (1px inset #808080)] [r=0 c=1 rs=2 cs=2]
+            RenderTableCell {TD} at (18,15) size 136x24 [border: (1px inset #808080)] [r=0 c=1 rs=2 cs=2]
               RenderText {#text} at (3,3) size 130x18
                 text run at (3,3) width 130: "colspan 2 rowspan 2"
             RenderTableCell {TD} at (156,2) size 22x24 [border: (1px inset #808080)] [r=0 c=3 rs=1 cs=1]
               RenderText {#text} at (3,3) size 8x18
                 text run at (3,3) width 8: "3"
-          RenderTableRow {TR} at (0,28) size 204x24
+          RenderTableRow {TR} at (0,28) size 236x24
             RenderTableCell {TD} at (2,28) size 14x24 [border: (1px inset #808080)] [r=1 c=0 rs=1 cs=1]
               RenderText {#text} at (3,3) size 8x18
                 text run at (3,3) width 8: "4"
-            RenderTableCell {TD} at (18,28) size 67x24 [border: (1px inset #808080)] [r=1 c=1 rs=1 cs=1]
+            RenderTableCell {TD} at (156,28) size 22x24 [border: (1px inset #808080)] [r=1 c=3 rs=1 cs=1]
               RenderText {#text} at (3,3) size 8x18
                 text run at (3,3) width 8: "5"
-            RenderTableCell {TD} at (87,28) size 67x24 [border: (1px inset #808080)] [r=1 c=2 rs=1 cs=1]
+            RenderTableCell {TD} at (180,28) size 22x24 [border: (1px inset #808080)] [r=1 c=4 rs=1 cs=1]
               RenderText {#text} at (3,3) size 8x18
                 text run at (3,3) width 8: "6"
-            RenderTableCell {TD} at (156,28) size 22x24 [border: (1px inset #808080)] [r=1 c=3 rs=1 cs=1]
+            RenderTableCell {TD} at (204,28) size 14x24 [border: (1px inset #808080)] [r=1 c=5 rs=1 cs=1]
               RenderText {#text} at (3,3) size 8x18
                 text run at (3,3) width 8: "7"
-            RenderTableCell {TD} at (180,28) size 22x24 [border: (1px inset #808080)] [r=1 c=4 rs=1 cs=1]
+            RenderTableCell {TD} at (220,28) size 14x24 [border: (1px inset #808080)] [r=1 c=6 rs=1 cs=1]
               RenderText {#text} at (3,3) size 8x18
                 text run at (3,3) width 8: "8"
-          RenderTableRow {TR} at (0,54) size 204x24
+          RenderTableRow {TR} at (0,54) size 236x24
             RenderTableCell {TD} at (2,54) size 14x24 [border: (1px inset #808080)] [r=2 c=0 rs=1 cs=1]
               RenderText {#text} at (3,3) size 8x18
                 text run at (3,3) width 8: "9"
diff --git a/LayoutTests/tables/mozilla/bugs/bug22246-3-expected.checksum b/LayoutTests/tables/mozilla/bugs/bug22246-3-expected.checksum
new file mode 100644 (file)
index 0000000..b89da19
--- /dev/null
@@ -0,0 +1 @@
+5f2781bc6bff81d9cb6557b763728de4
\ No newline at end of file
diff --git a/LayoutTests/tables/mozilla/bugs/bug22246-3-expected.png b/LayoutTests/tables/mozilla/bugs/bug22246-3-expected.png
new file mode 100644 (file)
index 0000000..49f80f0
Binary files /dev/null and b/LayoutTests/tables/mozilla/bugs/bug22246-3-expected.png differ
@@ -6,35 +6,35 @@ layer at (0,0) size 800x600
       RenderBlock (anonymous) at (0,0) size 784x18
         RenderText {#text} at (0,0) size 4x18
           text run at (0,0) width 4: " "
-      RenderTable {TABLE} at (0,18) size 182x82 [border: (1px outset #808080)]
-        RenderTableSection {TBODY} at (1,1) size 180x80
-          RenderTableRow {TR} at (0,2) size 180x24
+      RenderTable {TABLE} at (0,18) size 230x82 [border: (1px outset #808080)]
+        RenderTableSection {TBODY} at (1,1) size 228x80
+          RenderTableRow {TR} at (0,2) size 228x24
             RenderTableCell {TD} at (2,2) size 14x24 [border: (1px inset #808080)] [r=0 c=0 rs=1 cs=1]
               RenderText {#text} at (3,3) size 8x18
                 text run at (3,3) width 8: "1"
-            RenderTableCell {TD} at (18,2) size 136x24 [border: (1px inset #808080)] [r=0 c=1 rs=2 cs=3]
+            RenderTableCell {TD} at (18,15) size 136x24 [border: (1px inset #808080)] [r=0 c=1 rs=2 cs=3]
               RenderText {#text} at (3,3) size 130x18
                 text run at (3,3) width 130: "colspan 3 rowspan 2"
-            RenderTableCell {TD} at (110,2) size 44x24 [border: (1px inset #808080)] [r=0 c=3 rs=1 cs=1]
+            RenderTableCell {TD} at (156,2) size 22x24 [border: (1px inset #808080)] [r=0 c=4 rs=1 cs=1]
               RenderText {#text} at (3,3) size 8x18
                 text run at (3,3) width 8: "3"
-          RenderTableRow {TR} at (0,28) size 180x24
+          RenderTableRow {TR} at (0,28) size 228x24
             RenderTableCell {TD} at (2,28) size 14x24 [border: (1px inset #808080)] [r=1 c=0 rs=1 cs=1]
               RenderText {#text} at (3,3) size 8x18
                 text run at (3,3) width 8: "4"
-            RenderTableCell {TD} at (18,28) size 44x24 [border: (1px inset #808080)] [r=1 c=1 rs=1 cs=1]
+            RenderTableCell {TD} at (156,28) size 22x24 [border: (1px inset #808080)] [r=1 c=4 rs=1 cs=1]
               RenderText {#text} at (3,3) size 8x18
                 text run at (3,3) width 8: "5"
-            RenderTableCell {TD} at (64,28) size 44x24 [border: (1px inset #808080)] [r=1 c=2 rs=1 cs=1]
+            RenderTableCell {TD} at (180,28) size 14x24 [border: (1px inset #808080)] [r=1 c=5 rs=1 cs=1]
               RenderText {#text} at (3,3) size 8x18
                 text run at (3,3) width 8: "6"
-            RenderTableCell {TD} at (110,28) size 44x24 [border: (1px inset #808080)] [r=1 c=3 rs=1 cs=1]
+            RenderTableCell {TD} at (196,28) size 14x24 [border: (1px inset #808080)] [r=1 c=6 rs=1 cs=1]
               RenderText {#text} at (3,3) size 8x18
                 text run at (3,3) width 8: "7"
-            RenderTableCell {TD} at (156,28) size 22x24 [border: (1px inset #808080)] [r=1 c=4 rs=1 cs=1]
+            RenderTableCell {TD} at (212,28) size 14x24 [border: (1px inset #808080)] [r=1 c=7 rs=1 cs=1]
               RenderText {#text} at (3,3) size 8x18
                 text run at (3,3) width 8: "8"
-          RenderTableRow {TR} at (0,54) size 180x24
+          RenderTableRow {TR} at (0,54) size 228x24
             RenderTableCell {TD} at (2,54) size 14x24 [border: (1px inset #808080)] [r=2 c=0 rs=1 cs=1]
               RenderText {#text} at (3,3) size 8x18
                 text run at (3,3) width 8: "9"
diff --git a/LayoutTests/tables/mozilla/bugs/bug22246-3a-expected.checksum b/LayoutTests/tables/mozilla/bugs/bug22246-3a-expected.checksum
new file mode 100644 (file)
index 0000000..b89da19
--- /dev/null
@@ -0,0 +1 @@
+5f2781bc6bff81d9cb6557b763728de4
\ No newline at end of file
diff --git a/LayoutTests/tables/mozilla/bugs/bug22246-3a-expected.png b/LayoutTests/tables/mozilla/bugs/bug22246-3a-expected.png
new file mode 100644 (file)
index 0000000..49f80f0
Binary files /dev/null and b/LayoutTests/tables/mozilla/bugs/bug22246-3a-expected.png differ
@@ -6,35 +6,35 @@ layer at (0,0) size 800x116
       RenderBlock (anonymous) at (0,0) size 784x18
         RenderText {#text} at (0,0) size 4x18
           text run at (0,0) width 4: " "
-      RenderTable {TABLE} at (0,18) size 182x82 [border: (1px outset #808080)]
-        RenderTableSection {TBODY} at (1,1) size 180x80
-          RenderTableRow {TR} at (0,2) size 180x24
+      RenderTable {TABLE} at (0,18) size 230x82 [border: (1px outset #808080)]
+        RenderTableSection {TBODY} at (1,1) size 228x80
+          RenderTableRow {TR} at (0,2) size 228x24
             RenderTableCell {TD} at (2,2) size 14x24 [border: (1px inset #808080)] [r=0 c=0 rs=1 cs=1]
               RenderText {#text} at (3,3) size 8x18
                 text run at (3,3) width 8: "1"
-            RenderTableCell {TD} at (18,2) size 136x24 [border: (1px inset #808080)] [r=0 c=1 rs=2 cs=3]
+            RenderTableCell {TD} at (18,15) size 136x24 [border: (1px inset #808080)] [r=0 c=1 rs=2 cs=3]
               RenderText {#text} at (3,3) size 130x18
                 text run at (3,3) width 130: "colspan 3 rowspan 2"
-            RenderTableCell {TD} at (110,2) size 44x24 [border: (1px inset #808080)] [r=0 c=3 rs=1 cs=1]
+            RenderTableCell {TD} at (156,2) size 22x24 [border: (1px inset #808080)] [r=0 c=4 rs=1 cs=1]
               RenderText {#text} at (3,3) size 8x18
                 text run at (3,3) width 8: "3"
-          RenderTableRow {TR} at (0,28) size 180x24
+          RenderTableRow {TR} at (0,28) size 228x24
             RenderTableCell {TD} at (2,28) size 14x24 [border: (1px inset #808080)] [r=1 c=0 rs=1 cs=1]
               RenderText {#text} at (3,3) size 8x18
                 text run at (3,3) width 8: "4"
-            RenderTableCell {TD} at (18,28) size 44x24 [border: (1px inset #808080)] [r=1 c=1 rs=1 cs=1]
+            RenderTableCell {TD} at (156,28) size 22x24 [border: (1px inset #808080)] [r=1 c=4 rs=1 cs=1]
               RenderText {#text} at (3,3) size 8x18
                 text run at (3,3) width 8: "5"
-            RenderTableCell {TD} at (64,28) size 44x24 [border: (1px inset #808080)] [r=1 c=2 rs=1 cs=1]
+            RenderTableCell {TD} at (180,28) size 14x24 [border: (1px inset #808080)] [r=1 c=5 rs=1 cs=1]
               RenderText {#text} at (3,3) size 8x18
                 text run at (3,3) width 8: "6"
-            RenderTableCell {TD} at (110,28) size 44x24 [border: (1px inset #808080)] [r=1 c=3 rs=1 cs=1]
+            RenderTableCell {TD} at (196,28) size 14x24 [border: (1px inset #808080)] [r=1 c=6 rs=1 cs=1]
               RenderText {#text} at (3,3) size 8x18
                 text run at (3,3) width 8: "7"
-            RenderTableCell {TD} at (156,28) size 22x24 [border: (1px inset #808080)] [r=1 c=4 rs=1 cs=1]
+            RenderTableCell {TD} at (212,28) size 14x24 [border: (1px inset #808080)] [r=1 c=7 rs=1 cs=1]
               RenderText {#text} at (3,3) size 8x18
                 text run at (3,3) width 8: "8"
-          RenderTableRow {TR} at (0,54) size 180x24
+          RenderTableRow {TR} at (0,54) size 228x24
             RenderTableCell {TD} at (2,54) size 14x24 [border: (1px inset #808080)] [r=2 c=0 rs=1 cs=1]
               RenderText {#text} at (3,3) size 8x18
                 text run at (3,3) width 8: "9"
diff --git a/LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-2-expected.checksum b/LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-2-expected.checksum
deleted file mode 100644 (file)
index 5eee244..0000000
+++ /dev/null
@@ -1 +0,0 @@
-9967432b9593d81489308a9de8054ff2
\ No newline at end of file
diff --git a/LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-2-expected.png b/LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-2-expected.png
deleted file mode 100644 (file)
index fe76d04..0000000
Binary files a/LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-2-expected.png and /dev/null differ
diff --git a/LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-2a-expected.checksum b/LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-2a-expected.checksum
deleted file mode 100644 (file)
index 5eee244..0000000
+++ /dev/null
@@ -1 +0,0 @@
-9967432b9593d81489308a9de8054ff2
\ No newline at end of file
diff --git a/LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-2a-expected.png b/LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-2a-expected.png
deleted file mode 100644 (file)
index fe76d04..0000000
Binary files a/LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-2a-expected.png and /dev/null differ
diff --git a/LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-3-expected.checksum b/LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-3-expected.checksum
deleted file mode 100644 (file)
index 0fbc2ad..0000000
+++ /dev/null
@@ -1 +0,0 @@
-9653444fc8458bb0e32cf9f726359592
\ No newline at end of file
diff --git a/LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-3-expected.png b/LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-3-expected.png
deleted file mode 100644 (file)
index 5678f1a..0000000
Binary files a/LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-3-expected.png and /dev/null differ
diff --git a/LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-3a-expected.checksum b/LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-3a-expected.checksum
deleted file mode 100644 (file)
index 0fbc2ad..0000000
+++ /dev/null
@@ -1 +0,0 @@
-9653444fc8458bb0e32cf9f726359592
\ No newline at end of file
diff --git a/LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-3a-expected.png b/LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-3a-expected.png
deleted file mode 100644 (file)
index 5678f1a..0000000
Binary files a/LayoutTests/tables/mozilla_expected_failures/bugs/bug22246-3a-expected.png and /dev/null differ
index 2497cdf..af7f7ac 100644 (file)
@@ -1,3 +1,16 @@
+2007-01-15  Mitz Pettel  <mitz@webkit.org>
+
+        Reviewed by Darin.
+
+        - fix http://bugs.webkit.org/show_bug.cgi?id=8604
+          rowspan does not work when set through javascript
+
+        Covered by existing tests.
+
+        * rendering/RenderTableCell.cpp:
+        (WebCore::RenderTableCell::updateFromElement): Mark the section for grid recalc
+        if row span or column span change.
+
 2007-01-15  Adam Roben  <aroben@apple.com>
 
         Reviewed by Darin.
index 2e2ae0b..239bd1e 100644 (file)
@@ -69,8 +69,11 @@ void RenderTableCell::updateFromElement()
 
         m_columnSpan = tc->colSpan();
         m_rowSpan = tc->rowSpan();
-        if ((oldRSpan != m_rowSpan || oldCSpan != m_columnSpan) && style() && parent())
+        if ((oldRSpan != m_rowSpan || oldCSpan != m_columnSpan) && style() && parent()) {
             setNeedsLayoutAndMinMaxRecalc();
+            if (section())
+                section()->setNeedsCellRecalc();
+        }
     }
 }