Bug 130976 - [iOS WK2] Hook up scroll events for accelerated overflow:scroll
Summary: [iOS WK2] Hook up scroll events for accelerated overflow:scroll
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Simon Fraser (smfr)
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2014-03-31 12:08 PDT by Simon Fraser (smfr)
Modified: 2014-03-31 15:32 PDT (History)
14 users (show)

See Also:


Attachments
Patch (32.12 KB, patch)
2014-03-31 12:30 PDT, Simon Fraser (smfr)
no flags Details | Formatted Diff | Diff
For EWS (32.12 KB, patch)
2014-03-31 12:44 PDT, Simon Fraser (smfr)
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Simon Fraser (smfr) 2014-03-31 12:08:20 PDT
[iOS WK2] Hook up scroll events for accelerated overflow:scroll
Comment 1 Simon Fraser (smfr) 2014-03-31 12:30:29 PDT
Created attachment 228186 [details]
Patch
Comment 2 Radar WebKit Bug Importer 2014-03-31 12:36:31 PDT
<rdar://problem/16476012>
Comment 3 Tim Horton 2014-03-31 12:41:22 PDT
Comment on attachment 228186 [details]
Patch

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?
Comment 4 Simon Fraser (smfr) 2014-03-31 12:44:09 PDT
Created attachment 228188 [details]
For EWS
Comment 5 Simon Fraser (smfr) 2014-03-31 14:24:26 PDT
https://trac.webkit.org/r166531
Comment 6 Alexey Proskuryakov 2014-03-31 15:32:55 PDT
Comment on attachment 228188 [details]
For EWS

Clearing review flag in a fixed bug.