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

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Dec 3 00:55:45 PST 2012


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





--- Comment #49 from Allan Sandfeld Jensen <allan.jensen at digia.com>  2012-12-03 00:58:07 PST ---
(From update of attachment 177187)
View in context: https://bugs.webkit.org/attachment.cgi?id=177187&action=review

Nice to see the improvements, but since you made a change in behavior that was obviously intended for some reason, we should change that as part of another bug. Since it was an optimization, this patch can land without.

> Source/WebCore/dom/Document.cpp:-5755
> -    // If the mouse is down and if this is a mouse move event, we want to restrict changes in
> -    // :hover/:active to only apply to elements that are in the :active chain that we froze
> -    // at the time the mouse went down.
> -    bool mustBeInActiveChain = request.active() && request.move();
> -

I don't think it is a good idea to change this behaviour in this patch. We should figure out why it was introduced in the first place and fix it in another bug if we think it is still a good idea.

-- 
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