|Summary:||Web Inspector: [Chromium] profiler - differentiate between native code (program) and idle time|
|Product:||WebKit||Reporter:||Paul Irish <paulirish>|
|Component:||Web Inspector (Deprecated)||Assignee:||Eugene Klyuchnikov <eustas>|
|Severity:||Normal||CC:||apavlov, bweinstein, caseq, eustas, joepeck, keishi, loislo, mnaganov, pfeldman, pmuellr, rik, webkit, webkit.review.bot, yurys|
|Version:||528+ (Nightly build)|
|Bug Depends on:||103120|
Description Paul Irish 2012-06-06 13:07:59 PDT
The term (program) in the profiler is unclear to developers. It accounts for native code execution (captured by about:profiler in chromium) as well as idle time. Is it possible to identify what is real idle time? A common use case is: * User starts recording a profiler session * User does an action which could use optimization * User stops recording What often happens is that the added time to manually start/stop the profiler in the GUI adds so much idle time that (program) can have %s of 90% or more. This makes the remaining percentages quite small and the user has to find "very expensive" operations that are taking < 1%. "(idle)" would would help clarify this situation as well as perhaps an ability to filter out Idle time from the profiling totals. From mnaganov: > Idle time is still native code time, so they both are combined in (program). To distinguish between them, we could tell the profiler that certain native functions it sees on the call stack actually mean idleness.
Comment 1 Mikhail Naganov 2012-06-07 01:49:56 PDT
I think this only relates to the Chromium port, as the JSC version already displays distinct entries for "(idle)" and "(program)"
Comment 2 Nicholas C. Zakas 2012-11-28 10:27:04 PST
Comment 4 Pavel Feldman 2012-12-18 09:43:21 PST
Comment on attachment 179933 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=179933&action=review > Source/WebCore/ChangeLog:10 > + injected as a top level node ti profile. to profile.
Comment 6 Eugene Klyuchnikov 2012-12-18 21:13:02 PST
Comment on attachment 179933 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=179933&action=review >> Source/WebCore/ChangeLog:10 >> + injected as a top level node ti profile. > > to profile. Fixed.