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

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Aug 3 08:41:24 PDT 2012


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





--- Comment #47 from Pavel Feldman <pfeldman at chromium.org>  2012-08-03 08:41:22 PST ---
(In reply to comment #46)
> I'd like to re-iterate:  this is a layering violation.  Nothing in platform/ should depend on files outside of platform/.  (Yes, there are exceptions, but we should be removing those, rather than adding new ones).
> 
> In particular, I see that inspector/ depends on dom/ and friends, which means this change introduces a new cyclic dependency between platform/ and dom/.
> 
> See https://lists.webkit.org/pipermail/webkit-dev/2012-January/019043.html.

Fixing it would require addition indirection or piggy-backing on the TRACE_EVENT. Is the latter kosher?

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



More information about the webkit-unassigned mailing list