[webkit-reviews] review requested: [Bug 25503] Add instrumentation framework to WebCore : [Attachment 30205] Addresses some comments and rebases against r43514
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon May 11 15:07:24 PDT 2009
James Robinson <jamesr at google.com> has asked for review:
Bug 25503: Add instrumentation framework to WebCore
https://bugs.webkit.org/show_bug.cgi?id=25503
Attachment 30205: Addresses some comments and rebases against r43514
https://bugs.webkit.org/attachment.cgi?id=30205&action=review
------- Additional Comments from James Robinson <jamesr at google.com>
This patch addresses the naming issues with AnalysisClient and creates a
separate didFoo() callback instead of the generic finished() callback. This
way an implementation only interested in a subset of the instrumentation points
could support only the willFoo()/didFoo() pairs they cared about.
This patch also merges up to r43514 and fixes some whitespace/path issues from
the original patch. I'll address the other comments raised in the bug
directly.
More information about the webkit-reviews
mailing list