AutoTableLayout applies min-width redundantly with RenderTable
authorojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 21 Feb 2013 03:26:10 +0000 (03:26 +0000)
committerojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 21 Feb 2013 03:26:10 +0000 (03:26 +0000)
https://bugs.webkit.org/show_bug.cgi?id=110426

Reviewed by Emil A Eklund.

* rendering/AutoTableLayout.cpp:
(WebCore::AutoTableLayout::computePreferredLogicalWidths):
This code used to do something, but has since been superceded by
code in RenderTable::computePreferredLogicalWidths.

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

Source/WebCore/ChangeLog
Source/WebCore/rendering/AutoTableLayout.cpp

index 29f5aaa..a878962 100644 (file)
@@ -1,3 +1,15 @@
+2013-02-20  Ojan Vafai  <ojan@chromium.org>
+
+        AutoTableLayout applies min-width redundantly with RenderTable
+        https://bugs.webkit.org/show_bug.cgi?id=110426
+
+        Reviewed by Emil A Eklund.
+
+        * rendering/AutoTableLayout.cpp:
+        (WebCore::AutoTableLayout::computePreferredLogicalWidths):
+        This code used to do something, but has since been superceded by
+        code in RenderTable::computePreferredLogicalWidths.
+
 2013-02-20  Dirk Schulze  <krit@webkit.org>
 
         Enable CANVAS_PATH flag
index 6abc0d0..bacf2ab 100644 (file)
@@ -260,12 +260,6 @@ void AutoTableLayout::computePreferredLogicalWidths(LayoutUnit& minWidth, Layout
         // if there was no remaining percent, maxWidth is invalid
         maxWidth = tableMaxWidth;
     }
-
-    Length tableLogicalMinWidth = m_table->style()->logicalMinWidth();
-    if (tableLogicalMinWidth.isFixed() && tableLogicalMinWidth.isPositive()) {
-        minWidth = max<int>(minWidth, tableLogicalMinWidth.value());
-        maxWidth = max<int>(minWidth, maxWidth);
-    }
 }
 
 /*