[webkit-reviews] review granted: [Bug 35949] Hover states not updated when overflow section scrolls under stationary mouse pointer : [Attachment 50576] Dispatch a fake mouse move event shrotly after scrolling

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Mar 12 07:42:22 PST 2010


Darin Adler <darin at apple.com> has granted mitz at webkit.org's request for review:
Bug 35949: Hover states not updated when overflow section scrolls under
stationary mouse pointer
https://bugs.webkit.org/show_bug.cgi?id=35949

Attachment 50576: Dispatch a fake mouse move event shrotly after scrolling
https://bugs.webkit.org/attachment.cgi?id=50576&action=review

------- Additional Comments from Darin Adler <darin at apple.com>
> +void EventHandler::cancelFakeMouseMoveEvent()
> +{
> +    if (!m_fakeMouseMoveEventTimer.isActive())
> +	   return;
> +
> +    m_fakeMouseMoveEventTimer.stop();
> +}

I don't think you need to check isActive before calling stop. It's safe to call
it even if the timer is not active.

> +void EventHandler::fakeMouseMoveEventTimerFired(Timer<EventHandler>* timer)
> +{
> +    ASSERT_ARG(timer, timer == &m_fakeMouseMoveEventTimer);

You need to use ASSERT_UNUSED here because "timer" is otherwise unused in this
function. I think this is why the Mac EWS build indicates a failure.

I'm a little sad that the test requires a timeout and can't run
instantaneously. Could it instead be based on the timing of the first mouse
move event?

r=me, but please fix the Mac build issue


More information about the webkit-reviews mailing list