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

[bugzilla-daemon at webkit.org]

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


Stephen White <senorblanco at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jamesr at chromium.org,
                   |                            |nduca at chromium.org




--- Comment #48 from Stephen White <senorblanco at chromium.org>  2012-08-03 08:43:10 PST ---
(In reply to comment #47)
> (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?

You'd have to ask Nat Duca about that.

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