Bug 130976

Summary: [iOS WK2] Hook up scroll events for accelerated overflow:scroll
Product: WebKit Reporter: Simon Fraser (smfr) <simon.fraser>
Component: New BugsAssignee: Simon Fraser (smfr) <simon.fraser>
Status: RESOLVED FIXED    
Severity: Normal CC: andersca, bdakin, cmarcelo, commit-queue, esprehn+autocc, glenn, jamesr, kondapallykalyan, luiz, sam, simon.fraser, thorton, tonikitoo, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
For EWS none

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.