Title: [124475] trunk/Source/WebCore
Revision
124475
Author
[email protected]
Date
2012-08-02 10:25:10 -0700 (Thu, 02 Aug 2012)

Log Message

Add back ASSERT(!needsLayout) to RenderTableSection which is now valid
https://bugs.webkit.org/show_bug.cgi?id=92954

Reviewed by Julien Chaffraix.

* rendering/RenderTableSection.cpp:
(WebCore::RenderTableSection::paint):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (124474 => 124475)


--- trunk/Source/WebCore/ChangeLog	2012-08-02 17:14:03 UTC (rev 124474)
+++ trunk/Source/WebCore/ChangeLog	2012-08-02 17:25:10 UTC (rev 124475)
@@ -1,3 +1,13 @@
+2012-08-02  Eric Seidel  <[email protected]>
+
+        Add back ASSERT(!needsLayout) to RenderTableSection which is now valid
+        https://bugs.webkit.org/show_bug.cgi?id=92954
+
+        Reviewed by Julien Chaffraix.
+
+        * rendering/RenderTableSection.cpp:
+        (WebCore::RenderTableSection::paint):
+
 2012-08-02  Kevin Ellis  <[email protected]>
 
         Month-year selector on calendar picker should be touch friendly.

Modified: trunk/Source/WebCore/rendering/RenderTableSection.cpp (124474 => 124475)


--- trunk/Source/WebCore/rendering/RenderTableSection.cpp	2012-08-02 17:14:03 UTC (rev 124474)
+++ trunk/Source/WebCore/rendering/RenderTableSection.cpp	2012-08-02 17:25:10 UTC (rev 124475)
@@ -967,12 +967,8 @@
 
 void RenderTableSection::paint(PaintInfo& paintInfo, const LayoutPoint& paintOffset)
 {
-    // put this back in when all layout tests can handle it
-    // ASSERT(!needsLayout());
-    // avoid crashing on bugs that cause us to paint with dirty layout
-    if (needsLayout())
-        return;
-    
+    ASSERT(!needsLayout());
+
     unsigned totalRows = m_grid.size();
     unsigned totalCols = table()->columns().size();
 
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to