[Webkit-unassigned] [Bug 25503] Add instrumentation framework to WebCore

[bugzilla-daemon at webkit.org]

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


oliver at apple.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #30205|review?                     |review-
               Flag|                            |




------- Comment #7 from oliver at apple.com  2009-05-22 20:22 PDT -------
(From update of attachment 30205)
I am not sure why these need to use client objects -- as this is primarily of
value to developers it should be entirely inside webcore with appropriate
bindings to the inspector.  Currently it sits in the inspector directory but
the inspector does not use it at all.

I feel it's value in DRT is vastly less than in the inspector so there should
be at least rudimentary information displayed in the inspector as part of this
patch.

Also you're adding branches to some very hot code paths so i'd like to be sure
that you've ensured that there isn't a performance regression in layout or
rendering.

Given these issues i'm r-'ing this for now


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