Fix for bugzilla bug 8060.
authorhyatt <hyatt@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 8 May 2006 04:14:27 +0000 (04:14 +0000)
committerhyatt <hyatt@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 8 May 2006 04:14:27 +0000 (04:14 +0000)
        Fixed width table cells weren't having their border widths
        included in the effective width of the column (and thus could end up being
        too small).

        Reviewed by darin

        Added fast/tables/fixed-cell-growth.html

        * rendering/table_layout.cpp:
        (WebCore::AutoTableLayout::recalcColumn):

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

WebCore/ChangeLog
WebCore/rendering/table_layout.cpp

index 1e32c2de01b3da1c5c1717061d8106edea649964..554bdd87939dde774ca7eb72007956d1b4d8b952 100644 (file)
@@ -1,3 +1,18 @@
+2006-05-07  David Hyatt  <hyatt@apple.com>
+
+        Fix for bugzilla bug 8060.
+
+        Fixed width table cells weren't having their border widths
+        included in the effective width of the column (and thus could end up being
+        too small).
+
+        Reviewed by darin
+
+        Added fast/tables/fixed-cell-growth.html
+
+        * rendering/table_layout.cpp:
+        (WebCore::AutoTableLayout::recalcColumn):
+
 2006-05-07  Alexey Proskuryakov  <ap@nypop.com>
 
         Reviewed by Darin.
index 3b35f91ad5a7b6ca932f8c68a2b78e248976a1da..7c8bf5ac7f79da67fa015f2dce4269e7b38a4720 100644 (file)
@@ -403,7 +403,7 @@ void AutoTableLayout::recalcColumn( int effCol )
                     case Fixed:
                         // ignore width=0
                         if ( w.value() > 0 && (int)l.width.type() != Percent ) {
-                            int wval = w.value() + (cell->paddingLeft() + cell->paddingRight());
+                            int wval = cell->calcBorderBoxWidth(w.value());
                             if ( l.width.isFixed() ) {
                                 // Nav/IE weirdness
                                 if ((wval > l.width.value()) ||