[webkit-reviews] review cancelled: [Bug 25503] Add instrumentation framework to WebCore : [Attachment 29934] Implementation of instrumentation and simple aggregate in DRT

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon May 11 15:07:23 PDT 2009


James Robinson <jamesr at google.com> has cancelled James Robinson
<jamesr at google.com>'s request for review:
Bug 25503: Add instrumentation framework to WebCore
https://bugs.webkit.org/show_bug.cgi?id=25503

Attachment 29934: Implementation of instrumentation and simple aggregate in DRT
https://bugs.webkit.org/attachment.cgi?id=29934&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