[webkit-reviews] review granted: [Bug 211884] Cursor should not update on a 20ms timer : [Attachment 399349] Patch

[bugzilla-daemon at webkit.org]

Antti Koivisto <koivisto at iki.fi> has granted Antoine Quint
<graouts at apple.com>'s request for review:
Bug 211884: Cursor should not update on a 20ms timer
https://bugs.webkit.org/show_bug.cgi?id=211884

Attachment 399349: Patch

https://bugs.webkit.org/attachment.cgi?id=399349&action=review




--- Comment #8 from Antti Koivisto <koivisto at iki.fi> ---
Comment on attachment 399349
  --> https://bugs.webkit.org/attachment.cgi?id=399349
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=399349&action=review

> Source/WebCore/dom/Document.cpp:8520
> +void Document::updateCursorIfNeeded()
> +{
> +    if (frame())
> +	   frame()->eventHandler().updateCursorIfNeeded();
> +}

Maybe this function is not needed at all?

> Source/WebCore/page/EventHandler.cpp:3153
> +    m_scheduledCursorUpdate = true;

m_hasScheduledCursorUpdate?