Title: [110378] trunk/Source/WebCore
Revision
110378
Author
[email protected]
Date
2012-03-10 09:08:02 -0800 (Sat, 10 Mar 2012)

Log Message

Unreviewed, rolling out r110358.
http://trac.webkit.org/changeset/110358
https://bugs.webkit.org/show_bug.cgi?id=80706

No improvement on specified benchmarks.

* dom/ContainerNode.cpp:
(WebCore::collectNodes):
(WebCore::collectTargetNodes):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (110377 => 110378)


--- trunk/Source/WebCore/ChangeLog	2012-03-10 16:01:13 UTC (rev 110377)
+++ trunk/Source/WebCore/ChangeLog	2012-03-10 17:08:02 UTC (rev 110378)
@@ -1,3 +1,15 @@
+2012-03-10  Stephen White  <[email protected]>
+
+        Unreviewed, rolling out r110358.
+        http://trac.webkit.org/changeset/110358
+        https://bugs.webkit.org/show_bug.cgi?id=80706
+
+        No improvement on specified benchmarks.
+
+        * dom/ContainerNode.cpp:
+        (WebCore::collectNodes):
+        (WebCore::collectTargetNodes):
+
 2012-03-10  MORITA Hajime  <[email protected]>
 
         ShadowTree uses weak iteration patterns

Modified: trunk/Source/WebCore/dom/ContainerNode.cpp (110377 => 110378)


--- trunk/Source/WebCore/dom/ContainerNode.cpp	2012-03-10 16:01:13 UTC (rev 110377)
+++ trunk/Source/WebCore/dom/ContainerNode.cpp	2012-03-10 17:08:02 UTC (rev 110378)
@@ -26,7 +26,6 @@
 #include "ChildListMutationScope.h"
 #include "ContainerNodeAlgorithms.h"
 #include "DeleteButtonController.h"
-#include "DocumentFragment.h"
 #include "EventNames.h"
 #include "ExceptionCode.h"
 #include "FloatRect.h"
@@ -62,9 +61,8 @@
 static size_t s_attachDepth;
 static bool s_shouldReEnableMemoryCacheCallsAfterAttach;
 
-static inline void collectNodes(ContainerNode* node, NodeVector& nodes)
+static inline void collectNodes(Node* node, NodeVector& nodes)
 {
-    nodes.reserveCapacity(nodes.size() + node->childNodeCount());
     for (Node* child = node->firstChild(); child; child = child->nextSibling())
         nodes.append(child);
 }
@@ -75,7 +73,7 @@
         nodes.append(node);
         return;
     }
-    collectNodes(static_cast<DocumentFragment*>(node), nodes);
+    collectNodes(node, nodes);
 }
 
 void ContainerNode::removeAllChildren()
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to