Changeset 34043 in webkit for trunk/JavaScriptCore/profiler/ProfileNode.cpp

Timestamp:

May 22, 2008, 6:03:17 PM (17 years ago)

Author:

[email protected]

Message:

2008-05-22 Kevin McCullough <​[email protected]>

Reviewed by Darin.

<rdar://problem/5951529> JSProfiler: Allow the profiler to "Exclude" a
profile node.
-Implement 'exclude'; where the excluded node attributes its time to its
parent's self time.

• JavaScriptCore.exp: Export the exclude function.
• profiler/Profile.h: (KJS::Profile::exclude):
• profiler/ProfileNode.cpp: (KJS::ProfileNode::setTreeVisible): New function that allows a change in visiblitiy to be propogated to all the children of a node. (KJS::ProfileNode::exclude): If the node matches the callIdentifier then set the visiblity of this node and all of its children to false and attribute it's total time to it's caller's self time.
• profiler/ProfileNode.h:

File:

• trunk/JavaScriptCore/profiler/ProfileNode.cpp (modified) (2 diffs)

trunk/JavaScriptCore/profiler/ProfileNode.cpp

@@ -239 +239 @@
 }
 
+void ProfileNode::setTreeVisible(bool visible)
+{
+    m_visible = visible;
+
+    for (StackIterator currentChild = m_children.begin(); currentChild != m_children.end(); ++currentChild)
+        (*currentChild)->setTreeVisible(visible);
+}
+
 void ProfileNode::focus(const CallIdentifier& callIdentifier, bool forceVisible)
 {
@@ -261 +269 @@
 }
 
+double ProfileNode::exclude(const CallIdentifier& callIdentifier)
+{
+    if (m_callIdentifier == callIdentifier) {
+        m_visible = false;
+
+        for (StackIterator currentChild = m_children.begin(); currentChild != m_children.end(); ++currentChild)
+            (*currentChild)->setTreeVisible(false);
+
+        return m_visibleTotalTime;
+    }
+
+    for (StackIterator currentChild = m_children.begin(); currentChild != m_children.end(); ++currentChild)
+        m_visibleSelfTime += (*currentChild)->exclude(callIdentifier);
+
+    return 0;
+}
+
 void ProfileNode::restoreAll()
 {