Title: [143555] trunk/Source/WebCore
Revision
143555
Author
o...@chromium.org
Date
2013-02-20 19:26:10 -0800 (Wed, 20 Feb 2013)

Log 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.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (143554 => 143555)


--- trunk/Source/WebCore/ChangeLog	2013-02-21 03:22:31 UTC (rev 143554)
+++ trunk/Source/WebCore/ChangeLog	2013-02-21 03:26:10 UTC (rev 143555)
@@ -1,3 +1,15 @@
+2013-02-20  Ojan Vafai  <o...@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  <k...@webkit.org>
 
         Enable CANVAS_PATH flag

Modified: trunk/Source/WebCore/rendering/AutoTableLayout.cpp (143554 => 143555)


--- trunk/Source/WebCore/rendering/AutoTableLayout.cpp	2013-02-21 03:22:31 UTC (rev 143554)
+++ trunk/Source/WebCore/rendering/AutoTableLayout.cpp	2013-02-21 03:26:10 UTC (rev 143555)
@@ -260,12 +260,6 @@
         // 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);
-    }
 }
 
 /*
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to