[Webkit-unassigned] [Bug 90277] Web Inspector: added Paint events for Images to TimelineAgent

[bugzilla-daemon at webkit.org]

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





--- Comment #36 from Andrey Kosyakov <caseq at chromium.org>  2012-07-13 03:08:33 PST ---
(From update of attachment 151904)
View in context: https://bugs.webkit.org/attachment.cgi?id=151904&action=review

>> Source/WebCore/platform/image-decoders/bmp/BMPImageDecoder.cpp:86
>> +        InspectorInstrumentation::willDecodeImage("BMP");
> 
> I can see existing TRACE_EVENT called from within decode. Is there a reason we want to put our instrumentation elsewhere? Also, could use introduce RAII concept (InspectorInstrumentation::TraceEvent("ImageDecode") trace;) for this kind of instrumentation?

We'd like to only show full image decodes, as opposed to size only, as the latter are presumed to be cheap -- hence the instrumentation is at the (only) call site that requests full decode. This may be moved instead to the location of TRACE, but will have an extra conditional -- do you prefer this?

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