Summary: | Web Inspector: enable instrumentation of platform code | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Andrey Kosyakov <caseq> | ||||||||
Component: | Web Inspector (Deprecated) | Assignee: | Andrey Kosyakov <caseq> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | apavlov, bweinstein, gyuyoung.kim, joepeck, keishi, loislo, pfeldman, pmuellr, rakuco, rik, senorblanco, timothy, webkit.review.bot, yurys | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | All | ||||||||||
OS: | All | ||||||||||
Bug Depends on: | |||||||||||
Bug Blocks: | 90277 | ||||||||||
Attachments: |
|
Description
Andrey Kosyakov
2012-08-15 10:54:35 PDT
Created attachment 158595 [details]
Patch
Comment on attachment 158595 [details]
Patch
This looks great.
Comment on attachment 158595 [details] Patch Attachment 158595 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/13509282 Created attachment 158756 [details]
Patch
Comment on attachment 158756 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=158756&action=review Minor nits, otherwise looks good. > Source/WebCore/inspector/InspectorTimelineAgent.cpp:-491 > - InspectorInstrumentation::setTimelineAgentForOrphanEvents(0); You should remove InspectorInstrumentation::setTimelineAgentForOrphanEvents, InspectorInstrumentation::threadSpecificTimelineAgentForOrphanEvents, etc. as well > Source/WebCore/inspector/InspectorTimelineAgent.cpp:556 > + m_platformInstrumentationClientSet = m_recordStack.size(); m_platformInstrumentationClientInstalledAtStackDepth > Source/WebCore/platform/PlatformInstrumentation.h:46 > + virtual void willDecodeImage(const WTF::String& imageType) = 0; Here and below drop WTF:: > Source/WebCore/platform/image-decoders/png/PNGImageDecoder.cpp:265 > + if (frame.status() != ImageFrame::FrameComplete) { You did not remove TRACE_EVENT macro / include. Created attachment 158775 [details]
Patch
Committed r125769: <http://trac.webkit.org/changeset/125769> |