[Webkit-unassigned] [Bug 120084] Allow the inspector to show the reason for style invalidation

[bugzilla-daemon at webkit.org]

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





--- Comment #6 from Simon Fraser (smfr) <simon.fraser at apple.com>  2013-08-23 08:57:23 PST ---
> It might be useful to make the reason tracking part of the style resolve code rather than being in Inspector land. We might have internal uses for this information. Specifically we might want to decide what sort of resolve to do at resolve time instead at invalidation time like we do now.
> 
> This is not really a show stopper for this patch (we can switch over later) but something to consider.

Interesting, though in some cases it would require plumbing a "reason" param through several levels of calls.

> >> Source/WebCore/inspector/InspectorInstrumentation.h:196
> >> +    class RecalcStyleReasonSetter {
> > 
> > Would be nice to make this explicitly non-copyable.
> 
> Global variable based hackery like this is pretty distasteful. It is probably ok for Inspector interface but at least we shouldn't propagate these elsewhere.

InspectorInstrumentation consults a global to decide whether to take a fast return, so this isn't really any different, but I agree that global state is icky.

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