[webkit-changes] [WebKit/WebKit] 7b193c: ScrollView::convertChildToSelf / convertSelfToChil...

Simon Fraser noreply at github.com
Thu Oct 5 14:11:46 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 7b193c245074e736d83051b75490e4e6dfe0baa1
      https://github.com/WebKit/WebKit/commit/7b193c245074e736d83051b75490e4e6dfe0baa1
  Author: Simon Fraser <simon.fraser at apple.com>
  Date:   2023-10-05 (Thu, 05 Oct 2023)

  Changed paths:
    M Source/WebCore/platform/ScrollView.cpp
    M Source/WebCore/platform/ScrollView.h
    M Source/WebCore/platform/Widget.cpp

  Log Message:
  -----------
  ScrollView::convertChildToSelf / convertSelfToChild fail to take top content inset into account
https://bugs.webkit.org/show_bug.cgi?id=262585
rdar://116430905

Reviewed by Tim Horton.

ScrollView::convertChildToSelf() and convertSelfToChild() offset the point by the scroll position, but this
fails to take top content inset, and left-side scrollbars into account.

This code path isn't hit in general browsing, but future PDF plugin code needs it to work.

Move the implementations to the .cpp file, and use documentScrollPositionRelativeToViewOrigin()
instead of scrollPosition(). Pass the points by value.

* Source/WebCore/platform/ScrollView.cpp:
(WebCore::ScrollView::convertChildToSelf const):
(WebCore::ScrollView::convertSelfToChild const):
* Source/WebCore/platform/ScrollView.h:
(WebCore::ScrollView::convertChildToSelf const): Deleted.
(WebCore::ScrollView::convertSelfToChild const): Deleted.
* Source/WebCore/platform/Widget.cpp:
(WebCore::Widget::convertFromContainingView const): auto
(WebCore::Widget::convertToContainingView const): auto

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




More information about the webkit-changes mailing list