Changeset 32354 in webkit for trunk/JavaScriptCore/profiler/FunctionCallProfile.cpp

Timestamp:

Apr 21, 2008, 4:13:27 PM (17 years ago)

Author:

[email protected]

Message:

2008-04-21 Kevin McCullough <​[email protected]>

Reviewed by Adam.

<rdar://problem/5770054> JavaScript profiler (10928)

• When stop profiling is called we need to stop the timers on all the functions that are still running.

• profiler/FunctionCallProfile.cpp: (KJS::FunctionCallProfile::didExecute): (KJS::FunctionCallProfile::stopProfiling):
• profiler/FunctionCallProfile.h:
• profiler/Profiler.cpp: (KJS::Profiler::stopProfiling):

File:

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

trunk/JavaScriptCore/profiler/FunctionCallProfile.cpp

@@ -61 +61 @@
 
         m_timeSum += getCurrentUTCTime() - m_startTime;
+        m_startTime = 0.0;
 
         // FIXME: We may need something with higher resolution than ms as some functions will take 0ms.
@@ -95 +96 @@
 
     return 0;
+}
+
+void FunctionCallProfile::stopProfiling()
+{
+    if (m_startTime)
+        m_timeSum += getCurrentUTCTime() - m_startTime;
+
+    StackIterator endOfChildren = m_children.end();
+    for (StackIterator it = m_children.begin(); it != endOfChildren; ++it)
+        (*it)->stopProfiling();
 }