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):

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

diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index 1e32c2de01b3da1c5c1717061d8106edea649964..554bdd87939dde774ca7eb72007956d1b4d8b952 100644 (file)
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -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.

diff --git a/WebCore/rendering/table_layout.cpp b/WebCore/rendering/table_layout.cpp
index 3b35f91ad5a7b6ca932f8c68a2b78e248976a1da..7c8bf5ac7f79da67fa015f2dce4269e7b38a4720 100644 (file)
--- a/WebCore/rendering/table_layout.cpp
+++ b/WebCore/rendering/table_layout.cpp
@@ -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()) ||