Changeset 163754 in webkit

Timestamp:

Feb 9, 2014, 1:32:34 PM (12 years ago)

Author:

akling@apple.com

Message:

Reduce ref churn in ChildNodesLazySnapshot iteration.
<​https://webkit.org/b/128492>

Add a missing release() in ChildNodesLazySnapshot::nextNode()
so we don't have to churn the ref count.

Reviewed by Sam Weinig.

• dom/ContainerNode.h:

(WebCore::ChildNodesLazySnapshot::nextNode):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• dom/ContainerNode.h (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2014-02-09  Andreas Kling  <akling@apple.com>
+
+        Reduce ref churn in ChildNodesLazySnapshot iteration.
+        <https://webkit.org/b/128492>
+
+        Add a missing release() in ChildNodesLazySnapshot::nextNode()
+        so we don't have to churn the ref count.
+
+        Reviewed by Sam Weinig.
+
+        * dom/ContainerNode.h:
+        (WebCore::ChildNodesLazySnapshot::nextNode):
+
 2014-02-09  Andreas Kling  <akling@apple.com>
 

trunk/Source/WebCore/dom/ContainerNode.h

@@ -274 +274 @@
     {
         if (LIKELY(!hasSnapshot())) {
-            RefPtr<Node> node = m_currentNode;
+            RefPtr<Node> node = m_currentNode.release();
             if (node)
                 m_currentNode = node->nextSibling();