[Webkit-unassigned] [Bug 101857] Updating mouse cursor on style changes without emitting fake mousemove event

[bugzilla-daemon at webkit.org]

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


Simon Fraser (smfr) <simon.fraser at apple.com> changed:

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




--- Comment #60 from Simon Fraser (smfr) <simon.fraser at apple.com>  2012-12-03 13:29:50 PST ---
(From update of attachment 177187)
View in context: https://bugs.webkit.org/attachment.cgi?id=177187&action=review

> Source/WebCore/rendering/RenderObject.cpp:1813
> +    if (oldStyle.get() && !areCursorsEqual(oldStyle.get(), this->style())) {
> +        if (Frame* frame = this->frame()) {
> +            if (node() && node()->hovered())
> +                frame->eventHandler()->updateCursor();
> +        }
> +    }

The old code did some coalescing (by virtue of the "soon"), and yours does not. I'd be worried that this could create a performance issue.

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