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 Rendering

Assignee:

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:

Keywords:

InRadar

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Bug Depends on:

145140

Bug Blocks:

Attachments:

Description	Flags
Patch	bdakin: review+

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>