Title: [106228] trunk/Source/WebCore
- Revision
- 106228
- Author
- [email protected]
- Date
- 2012-01-30 03:32:20 -0800 (Mon, 30 Jan 2012)
Log Message
No need to keep anonymous RenderBlock in DETAILS.
https://bugs.webkit.org/show_bug.cgi?id=77322
Patch by Shinya Kawanaka <[email protected]> on 2012-01-30
Reviewed by Hajime Morita.
Anonymous RenderBlock of DETAILS element was not squashed when detaching them.
However, it should be removed.
No new tests. Should be covered by existing tests.
* rendering/RenderBlock.cpp:
(WebCore::canMergeContiguousAnonymousBlocks):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (106227 => 106228)
--- trunk/Source/WebCore/ChangeLog 2012-01-30 11:17:58 UTC (rev 106227)
+++ trunk/Source/WebCore/ChangeLog 2012-01-30 11:32:20 UTC (rev 106228)
@@ -1,3 +1,18 @@
+2012-01-30 Shinya Kawanaka <[email protected]>
+
+ No need to keep anonymous RenderBlock in DETAILS.
+ https://bugs.webkit.org/show_bug.cgi?id=77322
+
+ Reviewed by Hajime Morita.
+
+ Anonymous RenderBlock of DETAILS element was not squashed when detaching them.
+ However, it should be removed.
+
+ No new tests. Should be covered by existing tests.
+
+ * rendering/RenderBlock.cpp:
+ (WebCore::canMergeContiguousAnonymousBlocks):
+
2012-01-30 Zoltan Herczeg <[email protected]>
Custom written CSS lexer
Modified: trunk/Source/WebCore/rendering/RenderBlock.cpp (106227 => 106228)
--- trunk/Source/WebCore/rendering/RenderBlock.cpp 2012-01-30 11:17:58 UTC (rev 106227)
+++ trunk/Source/WebCore/rendering/RenderBlock.cpp 2012-01-30 11:32:20 UTC (rev 106228)
@@ -1000,11 +1000,6 @@
if (oldChild->documentBeingDestroyed() || oldChild->isInline() || oldChild->virtualContinuation())
return false;
-#if ENABLE(DETAILS)
- if (oldChild->parent() && oldChild->parent()->isDetails())
- return false;
-#endif
-
if ((prev && (!prev->isAnonymousBlock() || toRenderBlock(prev)->continuation() || toRenderBlock(prev)->beingDestroyed()))
|| (next && (!next->isAnonymousBlock() || toRenderBlock(next)->continuation() || toRenderBlock(next)->beingDestroyed())))
return false;
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes