[webkit-reviews] review granted: [Bug 234937] Ensure that the scrolling thread always commits layer position changes to reduce scrolling stutters (v2) : [Attachment 448536] Patch

[bugzilla-daemon at webkit.org]

Tim Horton <thorton at apple.com> has granted Simon Fraser (smfr)
<simon.fraser at apple.com>'s request for review:
Bug 234937: Ensure that the scrolling thread always commits layer position
changes to reduce scrolling stutters (v2)
https://bugs.webkit.org/show_bug.cgi?id=234937

Attachment 448536: Patch

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




--- Comment #2 from Tim Horton <thorton at apple.com> ---
Comment on attachment 448536
  --> https://bugs.webkit.org/attachment.cgi?id=448536
Patch

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

> Source/WebCore/ChangeLog:8
> +	   Refine the logic added in r286932, which tries ensure that the
scrolling thread commits a

"tries ensure"

> Source/WebCore/page/Page.h:617
> +    void scheduledRenderingUpdate();

"didSchedule"? (initially I didn't notice the -ed and thought this was a
parameterless version of the line above)