Bug 171820 - Drop non-standard Element.scrollByLines() / scrollByPages()
Summary: Drop non-standard Element.scrollByLines() / scrollByPages()
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Chris Dumez
URL: https://dom.spec.whatwg.org/#interfac...
Keywords: WebExposed
Depends on:
Blocks:
 
Reported: 2017-05-08 11:55 PDT by Chris Dumez
Modified: 2017-05-08 15:15 PDT (History)
13 users (show)

See Also:


Attachments
Patch (10.41 KB, patch)
2017-05-08 13:14 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff
Patch (12.02 KB, patch)
2017-05-08 14:32 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Dumez 2017-05-08 11:55:25 PDT
Drop non-standard Element.scrollByLines() / scrollByPages().

Those were added a very long time ago (https://trac.webkit.org/changeset/7901/webkit) for Safari RSS but does not seem to be used anymore.

Blink has already dropped it without issue back in 2014:
- https://bugs.chromium.org/p/chromium/issues/detail?id=398356

Their UseCounter data showed no usage:
- https://www.chromestatus.com/metrics/feature/timeline/popularity/390
Comment 1 Chris Dumez 2017-05-08 13:14:00 PDT
Created attachment 309402 [details]
Patch
Comment 2 Chris Dumez 2017-05-08 14:32:06 PDT
Created attachment 309410 [details]
Patch
Comment 3 WebKit Commit Bot 2017-05-08 15:15:18 PDT
Comment on attachment 309410 [details]
Patch

Clearing flags on attachment: 309410

Committed r216456: <http://trac.webkit.org/changeset/216456>
Comment 4 WebKit Commit Bot 2017-05-08 15:15:20 PDT
All reviewed patches have been landed.  Closing bug.