Changeset 143555 in webkit

Timestamp:

Feb 20, 2013, 7:26:10 PM (13 years ago)

Author:

ojan@chromium.org

Message:

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.

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• rendering/AutoTableLayout.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+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>
 

trunk/Source/WebCore/rendering/AutoTableLayout.cpp

@@ -261 +261 @@
         maxWidth = tableMaxWidth;
     }
-
-    Length tableLogicalMinWidth = m_table->style()->logicalMinWidth();
-    if (tableLogicalMinWidth.isFixed() && tableLogicalMinWidth.isPositive()) {
-        minWidth = max<int>(minWidth, tableLogicalMinWidth.value());
-        maxWidth = max<int>(minWidth, maxWidth);
-    }
 }