Changeset 139680 in webkit

Timestamp:

Jan 14, 2013 3:53:51 PM (11 years ago)

Author:

ojan@chromium.org

Message:

Remove unnecessary setNeedsLayoutAndPrefWidthsRecalc from RenderTable
​https://bugs.webkit.org/show_bug.cgi?id=106832

Reviewed by Levi Weintraub.

These are both called from locations that either set these bits themselves
or clearly don't need these bits set (e.g. computePreferredLogicalWidths).

• rendering/RenderTable.cpp:

(WebCore::RenderTable::splitColumn):
(WebCore::RenderTable::appendColumn):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• rendering/RenderTable.cpp (modified) (2 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2013-01-14  Ojan Vafai  <ojan@chromium.org>
+
+        Remove unnecessary setNeedsLayoutAndPrefWidthsRecalc from RenderTable
+        https://bugs.webkit.org/show_bug.cgi?id=106832
+
+        Reviewed by Levi Weintraub.
+
+        These are both called from locations that either set these bits themselves
+        or clearly don't need these bits set (e.g. computePreferredLogicalWidths).
+
+        * rendering/RenderTable.cpp:
+        (WebCore::RenderTable::splitColumn):
+        (WebCore::RenderTable::appendColumn):
+
 2013-01-14  Chris Rogers  <crogers@google.com>
 

trunk/Source/WebCore/rendering/RenderTable.cpp

@@ -759 +759 @@
 
     m_columnPos.grow(numEffCols() + 1);
-    setNeedsLayoutAndPrefWidthsRecalc();
 }
 
@@ -781 +780 @@
 
     m_columnPos.grow(numEffCols() + 1);
-    setNeedsLayoutAndPrefWidthsRecalc();
 }