[Webkit-unassigned] [Bug 56112] Web Inspector: Heavy Profile view should only include leaf functions as top-level nodes

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=56112


Adam Roben (:aroben) <aroben at apple.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|Leaks Viewer should omit    |Web Inspector: Heavy
                   |top-level nodes with 0 self |Profile view should only
                   |count in Heavy view         |include leaf functions as
                   |                            |top-level nodes
          Component|Tools / Tests               |Web Inspector
                 CC|                            |apavlov at chromium.org,
                   |                            |bweinstein at apple.com,
                   |                            |joepeck at webkit.org,
                   |                            |keishi at webkit.org,
                   |                            |loislo at chromium.org,
                   |                            |pfeldman at chromium.org,
                   |                            |pmuellr at yahoo.com,
                   |                            |rik at webkit.org,
                   |                            |timothy at apple.com,
                   |                            |yurys at chromium.org




--- Comment #1 from Adam Roben (:aroben) <aroben at apple.com>  2011-03-11 08:01:55 PST ---
I think this is actually a bug in the Inspector's Heavy profile view. This page gives a good description of how it's supposed to work:

http://developer.apple.com/library/mac/#documentation/DeveloperTools/Conceptual/SharkUserGuide/AdvancedProfileAnalysisandDataMining/AdvancedProfileAnalysisandDataMining.html#//apple_ref/doc/uid/TP40005233-CH7-SW2

> As you can see, the “heavy” view starts from the leaf functions and builds towards the base of the callstack

Any functions that are not leaf functions should not be shown as top-level nodes in the Heavy view.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.