Bug 145175 - REGRESSION(142590): Part 2: Scroll-snap points are improperly snapping to earlier index values (Scrolling Thread)
Summary: REGRESSION(142590): Part 2: Scroll-snap points are improperly snapping to ear...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Brent Fulgham
URL:
Keywords: InRadar
Depends on: 145140
Blocks:
  Show dependency tree / graph
 
Reported: 2015-05-19 12:48 PDT by Brent Fulgham
Modified: 2015-05-19 14:58 PDT (History)
11 users (show)

See Also:

Attachments
Patch (21.57 KB, patch)
2015-05-19 14:48 PDT, Brent Fulgham 		bdakin: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Brent Fulgham 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.
Comment 1 Radar WebKit Bug Importer 2015-05-19 12:49:26 PDT 
<rdar://problem/21023785>
Comment 2 Brent Fulgham 2015-05-19 14:48:48 PDT 
Created attachment 253404 [details]
Patch
Comment 3 Brent Fulgham 2015-05-19 14:58:35 PDT 
Committed r184596: <http://trac.webkit.org/changeset/184596>