[webkit-reviews] review granted: [Bug 230445] Remove use of NSScrollAnimationHelper for animated scrolls : [Attachment 438565] Patch

[bugzilla-daemon at webkit.org]

Martin Robinson <mrobinson at webkit.org> has granted Simon Fraser (smfr)
<simon.fraser at apple.com>'s request for review:
Bug 230445: Remove use of NSScrollAnimationHelper for animated scrolls
https://bugs.webkit.org/show_bug.cgi?id=230445

Attachment 438565: Patch

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




--- Comment #2 from Martin Robinson <mrobinson at webkit.org> ---
Comment on attachment 438565
  --> https://bugs.webkit.org/attachment.cgi?id=438565
Patch

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

This is a great cleanup!

> Source/WebCore/platform/ScrollAnimator.cpp:109
> +    WTFLogAlways("scrollToPositionWithoutAnimation to %.2f",
adjustedPosition.y());

Are these WTFLogAlways calls left over or did you mean to add them?

> Source/WebCore/platform/mac/ScrollAnimatorMac.mm:54
>  static bool scrollAnimationEnabledForSystem()

I think this can be removed and moved into the body of
platformAllowsScrollAnimation. I don't think it's used anywhere else.