Title: [206884] trunk/Source/WebCore
Revision
206884
Author
[email protected]
Date
2016-10-06 14:40:38 -0700 (Thu, 06 Oct 2016)

Log Message

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

Reviewed by Dan Bernstein.

LayoutTests pass fine now with this assert on.

Covered by existing tests.

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

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (206883 => 206884)


--- trunk/Source/WebCore/ChangeLog	2016-10-06 21:32:02 UTC (rev 206883)
+++ trunk/Source/WebCore/ChangeLog	2016-10-06 21:40:38 UTC (rev 206884)
@@ -1,3 +1,18 @@
+2016-10-06  Zalan Bujtas  <[email protected]>
+
+        Add back ASSERT(!needsLayout) to RenderTableSection which is now valid
+        https://bugs.webkit.org/show_bug.cgi?id=92954
+        <rdar://problem/12147973>
+
+        Reviewed by Dan Bernstein.
+
+        LayoutTests pass fine now with this assert on.
+
+        Covered by existing tests.
+
+        * rendering/RenderTableSection.cpp:
+        (WebCore::RenderTableSection::paint):
+
 2016-10-06  Jiewen Tan  <[email protected]>
 
         Add a dummy SubtleCrypto interface

Modified: trunk/Source/WebCore/rendering/RenderTableSection.cpp (206883 => 206884)


--- trunk/Source/WebCore/rendering/RenderTableSection.cpp	2016-10-06 21:32:02 UTC (rev 206883)
+++ trunk/Source/WebCore/rendering/RenderTableSection.cpp	2016-10-06 21:40:38 UTC (rev 206884)
@@ -943,8 +943,7 @@
 
 void RenderTableSection::paint(PaintInfo& paintInfo, const LayoutPoint& paintOffset)
 {
-    // put this back in when all layout tests can handle it
-    // ASSERT(!needsLayout());
+    ASSERT(!needsLayout());
     // avoid crashing on bugs that cause us to paint with dirty layout
     if (needsLayout())
         return;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to