Bug 145175

Summary: REGRESSION(142590): Part 2: Scroll-snap points are improperly snapping to earlier index values (Scrolling Thread)
Product: WebKit Reporter: Brent Fulgham <bfulgham>
Component: Layout and RenderingAssignee: Brent Fulgham <bfulgham>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, cmarcelo, commit-queue, esprehn+autocc, glenn, jamesr, kondapallykalyan, luiz, simon.fraser, tonikitoo, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on: 145140    
Bug Blocks:    
Attachments:
Description Flags
Patch bdakin: review+

Brent Fulgham
Reported 2015-05-19 12:48:10 PDT
I thought I fixed this yesterday with Bug 145140, but that actually only dealt with the overflow scrolling case. If I had followed the lessons of the 'isRubberBandInProgress', I would have noticed that we also need to recognize when a scrolling thread scroll is running the rubber band snap. This patch completes the work I started in Bug 145140.
Attachments
Patch (21.57 KB, patch)
2015-05-19 14:48 PDT, Brent Fulgham 		bdakin: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2015-05-19 12:49:26 PDT
<rdar://problem/21023785>
Brent Fulgham
Comment 2 2015-05-19 14:48:48 PDT
Created attachment 253404 [details] Patch
Brent Fulgham
Comment 3 2015-05-19 14:58:35 PDT
Committed r184596: <http://trac.webkit.org/changeset/184596>
Note You need to log in before you can comment on or make changes to this bug.