Bug 170560

Summary: Scroll offset jumps after a programmatic scroll in an overflow container with scroll snapping
Product: WebKit Reporter: Wenson Hsieh <wenson_hsieh>
Component: Layout and RenderingAssignee: Wenson Hsieh <wenson_hsieh>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, cathyxz, commit-queue, manian, simon.fraser, sriramkrish85, thorton, webkit-bug-importer, zalan
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Reduced test case adapted from <http://output.jsbin.com/fiwiqo>
none
Patch
thorton: review+
Patch for landing none

Wenson Hsieh
Reported 2017-04-06 13:03:30 PDT
Created attachment 306413 [details] Reduced test case adapted from <http://output.jsbin.com/fiwiqo> See whsieh.github.io/examples/scroll-snap-fixed-div.html on an iOS device.
Attachments
Reduced test case adapted from <http://output.jsbin.com/fiwiqo> (2.27 KB, text/html)
2017-04-06 13:03 PDT, Wenson Hsieh 		no flags
Details
Patch (9.48 KB, patch)
2017-04-06 13:32 PDT, Wenson Hsieh 		thorton: review+
DetailsFormatted DiffDiff
Patch for landing (9.33 KB, patch)
2017-04-06 16:58 PDT, Wenson Hsieh 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Wenson Hsieh
Comment 1 2017-04-06 13:04:42 PDT
This is the bug in <https://bugs.webkit.org/show_bug.cgi?id=169800> pertaining to overflow scrolling.
Radar WebKit Bug Importer
Comment 2 2017-04-06 13:06:10 PDT
<rdar://problem/31484693>
Wenson Hsieh
Comment 3 2017-04-06 13:32:44 PDT
Created attachment 306422 [details] Patch
Wenson Hsieh
Comment 4 2017-04-06 16:58:41 PDT
Created attachment 306438 [details] Patch for landing
WebKit Commit Bot
Comment 5 2017-04-06 17:38:45 PDT
Comment on attachment 306438 [details] Patch for landing Clearing flags on attachment: 306438 Committed r215075: <http://trac.webkit.org/changeset/215075>
Sriram Krishnan
Comment 6 2017-05-31 15:48:38 PDT
@Wenson Hsieh - looks like the patch failed to apply on IOS - could some one take a look at this please?
Wenson Hsieh
Comment 7 2017-05-31 16:30:32 PDT
(In reply to Sriram Krishnan from comment #6) > @Wenson Hsieh - looks like the patch failed to apply on IOS - could some > one take a look at this please? The "Failed to apply" message is only on an iOS EWS bot.
Note You need to log in before you can comment on or make changes to this bug.