Bug 206553

Summary:	REGRESSION (r254807): Page Up/Page Down to scroll no longer work
Product:	WebKit	Reporter:	Simon Fraser (smfr) <simon.fraser>
Component:	Scrolling	Assignee:	Nobody <webkit-unassigned>
Status:	RESOLVED DUPLICATE
Severity:	Normal	CC:	cathiechen, fred.wang, simon.fraser, webkit-bug-importer
Priority:	P2	Keywords:	InRadar
Version:	Safari Technology Preview
Hardware:	Unspecified
OS:	Unspecified
See Also:	https://bugs.webkit.org/show_bug.cgi?id=204882

Simon Fraser (smfr)

Reported 2020-01-21 14:01:08 PST

Can't scroll from the keyboard after r254807.

Attachments
Add attachment proposed patch, testcase, etc.

Radar WebKit Bug Importer

Comment 1 2020-01-21 14:01:36 PST

<rdar://problem/58772661>

Simon Fraser (smfr)

Comment 2 2020-01-21 15:17:09 PST

The change in ScrollableArea::setScrollOffsetFromAnimation(), that removed the call to requestScrollPositionUpdate() broke this. The changelog mentions this was removed to avoid iteration (I think it meant recursion), but through what codepath?

Simon Fraser (smfr)

Comment 3 2020-01-21 15:31:42 PST

We don't hit any of the newly added calls to requestScrollPositionUpdate() on macOS.

Simon Fraser (smfr)

Comment 4 2020-01-21 15:37:17 PST

*** This bug has been marked as a duplicate of bug 206559 ***

Note You need to log in before you can comment on or make changes to this bug.