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