Title: [106228] trunk/Source/WebCore
Revision
106228
Author
commit-qu...@webkit.org
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 <shin...@google.com> 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  <shin...@google.com>
+
+        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  <zherc...@webkit.org>
 
         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
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to