[webkit-reviews] review granted: [Bug 130976] [iOS WK2] Hook up scroll events for accelerated overflow:scroll : [Attachment 228186] 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 130976: [iOS WK2] Hook up scroll events for accelerated overflow:scroll
https://bugs.webkit.org/show_bug.cgi?id=130976

Attachment 228186: Patch
https://bugs.webkit.org/attachment.cgi?id=228186&action=review

------- Additional Comments from Tim Horton <thorton at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=228186&action=review


> Source/WebCore/ChangeLog:9
> +	   tell the WebProcess that the scroll happened to udpate RenderLayer

update

>
Source/WebKit2/UIProcess/Scrolling/ios/ScrollingTreeOverflowScrollingNodeIOS.mm
:72
> +- (void)scrollViewWillBeginDragging:(UIScrollView *)scrollView
> +{
> +}
> +
> +- (void)scrollViewDidEndDragging:(UIScrollView *)scrollView
willDecelerate:(BOOL)decelerate
> +{
> +}
> +
> +- (void)scrollViewDidEndScrollingAnimation:(UIScrollView *)scrollView
> +{
> +}

Is there a reason for these? Aren't they optional?