Skip to content
  • timothy@apple.com's avatar
    Fixes a bug in the profiler where call and apply would show up · e4f53f8a
    timothy@apple.com authored
    and double the time spent in a function. We don't want to show call
    and apply at all in the profiles. This change excludes them.
    
    Reviewed by Kevin McCullough.
    
    * profiler/ProfileNode.cpp:
    (KJS::ProfileNode::stopProfiling): Remove a second for loop and
    calculate self time in the existing loop.
    * profiler/Profiler.cpp:
    (KJS::shouldExcludeFunction): Helper inline function that returns
    true in the current function in an InternalFunctionImp and it is
    has the functionName call or apply.
    (KJS::Profiler::willExecute): Call shouldExcludeFunction and return
    early if if returns true.
    (KJS::Profiler::didExecute): Ditto.
    
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@33928 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    e4f53f8a