Title: [205436] releases/WebKitGTK/webkit-2.12
- Revision
- 205436
- Author
- [email protected]
- Date
- 2016-09-05 02:02:37 -0700 (Mon, 05 Sep 2016)
Log Message
Merge r204860 - ASSERTION FAILED: hasOverflowingCell == this->hasOverflowingCell() in WebCore::RenderTableSection::computeOverflowFromCells
https://bugs.webkit.org/show_bug.cgi?id=155702
<rdar://problem/27720731>
Reviewed by David Hyatt.
Source/WebCore:
The simplified layout path for table/section should also clear the slow path flag for visual overflow (since it recomputes
the overflow for the descendant cells).
Test: fast/table/assert-when-visual-overflow-is-cleared.html
* rendering/RenderTableSection.cpp:
(WebCore::RenderTableSection::layout):
LayoutTests:
* fast/table/assert-when-visual-overflow-is-cleared-expected.txt: Added.
* fast/table/assert-when-visual-overflow-is-cleared.html: Added.
Modified Paths
Added Paths
Diff
Modified: releases/WebKitGTK/webkit-2.12/LayoutTests/ChangeLog (205435 => 205436)
--- releases/WebKitGTK/webkit-2.12/LayoutTests/ChangeLog 2016-09-05 08:56:49 UTC (rev 205435)
+++ releases/WebKitGTK/webkit-2.12/LayoutTests/ChangeLog 2016-09-05 09:02:37 UTC (rev 205436)
@@ -1,3 +1,14 @@
+2016-08-23 Zalan Bujtas <[email protected]>
+
+ ASSERTION FAILED: hasOverflowingCell == this->hasOverflowingCell() in WebCore::RenderTableSection::computeOverflowFromCells
+ https://bugs.webkit.org/show_bug.cgi?id=155702
+ <rdar://problem/27720731>
+
+ Reviewed by David Hyatt.
+
+ * fast/table/assert-when-visual-overflow-is-cleared-expected.txt: Added.
+ * fast/table/assert-when-visual-overflow-is-cleared.html: Added.
+
2016-08-24 Chris Dumez <[email protected]>
Assertion hit: ASSERT(!m_parserScheduler) in ~HTMLDocumentParser()
Added: releases/WebKitGTK/webkit-2.12/LayoutTests/fast/table/assert-when-visual-overflow-is-cleared-expected.txt (0 => 205436)
--- releases/WebKitGTK/webkit-2.12/LayoutTests/fast/table/assert-when-visual-overflow-is-cleared-expected.txt (rev 0)
+++ releases/WebKitGTK/webkit-2.12/LayoutTests/fast/table/assert-when-visual-overflow-is-cleared-expected.txt 2016-09-05 09:02:37 UTC (rev 205436)
@@ -0,0 +1,3 @@
+PASS if no assert in debug.
+a
+
Added: releases/WebKitGTK/webkit-2.12/LayoutTests/fast/table/assert-when-visual-overflow-is-cleared.html (0 => 205436)
--- releases/WebKitGTK/webkit-2.12/LayoutTests/fast/table/assert-when-visual-overflow-is-cleared.html (rev 0)
+++ releases/WebKitGTK/webkit-2.12/LayoutTests/fast/table/assert-when-visual-overflow-is-cleared.html 2016-09-05 09:02:37 UTC (rev 205436)
@@ -0,0 +1,29 @@
+<!DOCTYPE html>
+<html>
+<head>
+<title>This tests that table does not assert when child visual overflow is cleared.</title>
+<script>
+if (window.testRunner)
+ testRunner.dumpAsText();
+
+window._onload_ = function() {
+ window.scrollTo(0, document.body.scrollHeight);
+ document.designMode = 'on';
+}
+</script>
+<style>
+pre {
+ margin-top: -1px;
+ position: relative;
+ display: inline-table;
+}
+
+span {
+ position: absolute;
+}
+</style>
+</head>
+<body>
+PASS if no assert in debug.
+<table><tr><th><span><div><pre><a>a<pre><span><details></details></span></pre></a></pre></div></span></tr></th></table>
+</body>
\ No newline at end of file
Modified: releases/WebKitGTK/webkit-2.12/Source/WebCore/ChangeLog (205435 => 205436)
--- releases/WebKitGTK/webkit-2.12/Source/WebCore/ChangeLog 2016-09-05 08:56:49 UTC (rev 205435)
+++ releases/WebKitGTK/webkit-2.12/Source/WebCore/ChangeLog 2016-09-05 09:02:37 UTC (rev 205436)
@@ -1,3 +1,19 @@
+2016-08-23 Zalan Bujtas <[email protected]>
+
+ ASSERTION FAILED: hasOverflowingCell == this->hasOverflowingCell() in WebCore::RenderTableSection::computeOverflowFromCells
+ https://bugs.webkit.org/show_bug.cgi?id=155702
+ <rdar://problem/27720731>
+
+ Reviewed by David Hyatt.
+
+ The simplified layout path for table/section should also clear the slow path flag for visual overflow (since it recomputes
+ the overflow for the descendant cells).
+
+ Test: fast/table/assert-when-visual-overflow-is-cleared.html
+
+ * rendering/RenderTableSection.cpp:
+ (WebCore::RenderTableSection::layout):
+
2016-08-24 Chris Dumez <[email protected]>
Assertion hit: ASSERT(!m_parserScheduler) in ~HTMLDocumentParser()
Modified: releases/WebKitGTK/webkit-2.12/Source/WebCore/rendering/RenderTableSection.cpp (205435 => 205436)
--- releases/WebKitGTK/webkit-2.12/Source/WebCore/rendering/RenderTableSection.cpp 2016-09-05 08:56:49 UTC (rev 205435)
+++ releases/WebKitGTK/webkit-2.12/Source/WebCore/rendering/RenderTableSection.cpp 2016-09-05 09:02:37 UTC (rev 205436)
@@ -372,6 +372,7 @@
ASSERT(!needsCellRecalc());
ASSERT(!table()->needsSectionRecalc());
+ m_forceSlowPaintPathWithOverflowingCell = false;
// addChild may over-grow m_grid but we don't want to throw away the memory too early as addChild
// can be called in a loop (e.g during parsing). Doing it now ensures we have a stable-enough structure.
m_grid.shrinkToFit();
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
