Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: d19de317fe22a9126816fb10be30ce577548c04c
https://github.com/WebKit/WebKit/commit/d19de317fe22a9126816fb10be30ce577548c04c
Author: Nikolaos Mouchtaris <[email protected]>
Date: 2025-03-21 (Fri, 21 Mar 2025)
Changed paths:
A LayoutTests/fast/scrolling/nested-scroll-into-view-expected.txt
A LayoutTests/fast/scrolling/nested-scroll-into-view.html
M Source/WebCore/page/LocalFrameView.cpp
Log Message:
-----------
engadget.com:Embedded YouTube video shifts upward video frame / container
upon opening resolution options
https://bugs.webkit.org/show_bug.cgi?id=290119
rdar://139008436
Reviewed by Simon Fraser.
When doing a focus on an element in nested scrollers, we may have to issue
multiple scrolls to get
the element visible. When iterating up the containing block chain, we have to
update the location
of the rect we are trying to make visible if there was a scroll issued. We
failed to update the
location of the visibility check rect in this case, which could result in us
thinking the rect was
off screen when it was already visible.
* LayoutTests/fast/scrolling/nested-scroll-into-view-expected.txt: Added.
* LayoutTests/fast/scrolling/nested-scroll-into-view.html: Added.
* Source/WebCore/page/LocalFrameView.cpp:
(WebCore::LocalFrameView::scrollRectToVisible):
Canonical link: https://commits.webkit.org/292485@main
To unsubscribe from these emails, change your notification settings at
https://github.com/WebKit/WebKit/settings/notifications
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
