[webkit-changes] [WebKit/WebKit] 819e12: Remove Element.scrollByLines() and Element.scrollB...

Karl Dubost noreply at github.com
Wed Jan 10 19:40:07 PST 2024


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 819e12af49a3d44661f118eda128fe30fc5c4a72
      https://github.com/WebKit/WebKit/commit/819e12af49a3d44661f118eda128fe30fc5c4a72
  Author: Karl Dubost <karlcow at apple.com>
  Date:   2024-01-10 (Wed, 10 Jan 2024)

  Changed paths:
    R LayoutTests/fast/css/element-scrollByLines-scrollByPages-obsolete-expected.txt
    R LayoutTests/fast/css/element-scrollByLines-scrollByPages-obsolete.html
    M LayoutTests/webgl/2.0.0/resources/webgl_test_files/deqp/temp_externs/gecko_dom.js
    M Source/WebCore/dom/Element.cpp
    M Source/WebCore/dom/Element.h
    M Source/WebKit/WebProcess/InjectedBundle/API/gtk/DOM/WebKitDOMElementGtk.cpp
    M Source/WebKitLegacy/mac/DOM/DOMElement.h
    M Source/WebKitLegacy/mac/DOM/DOMElement.mm

  Log Message:
  -----------
  Remove Element.scrollByLines() and Element.scrollByPages()
https://bugs.webkit.org/show_bug.cgi?id=264732
rdar://118647942

Reviewed by Simon Fraser and Dan Glastonbury.

Chromium removed this code in 2014. It seems that there is no internal
dependencies on these features.
Thanks to Anne van Kesteren for giving a better understanding of what
needed to be done.

* LayoutTests/fast/css/element-scrollByLines-scrollByPages-obsolete-expected.txt: Removed.
* LayoutTests/fast/css/element-scrollByLines-scrollByPages-obsolete.html: Removed.
* LayoutTests/webgl/2.0.0/resources/webgl_test_files/deqp/temp_externs/gecko_dom.js:
* Source/WebCore/dom/Element.cpp:
(WebCore::Element::scrollByUnits): Deleted.
(WebCore::Element::scrollByLines): Deleted.
(WebCore::Element::scrollByPages): Deleted.
* Source/WebCore/dom/Element.h:
* Source/WebKit/WebProcess/InjectedBundle/API/gtk/DOM/WebKitDOMElementGtk.cpp:
(webkit_dom_element_scroll_by_lines): Made the function no-op.
(webkit_dom_element_scroll_by_pages): Made the function no-op.
* Source/WebKitLegacy/mac/DOM/DOMElement.h: Moved the definitions to the deprecated section.
* Source/WebKitLegacy/mac/DOM/DOMElement.mm: Made the function no-op and moved them to the deprecated section.
(-[DOMElement scrollByLines:]):
(-[DOMElement scrollByPages:]):

Canonical link: https://commits.webkit.org/272881@main




More information about the webkit-changes mailing list