Bug 253070

Summary: Fix alignToClosestEdge semantics in ScrollableArea::getRectToExposeForScrollIntoView
Product: WebKit Reporter: Ahmad Saleem <ahmad.saleem792>
Component: ScrollingAssignee: Nobody <webkit-unassigned>
Status: NEW    
Severity: Normal CC: karlcow, simon.fraser, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: Safari Technology Preview   
Hardware: Unspecified   
OS: Unspecified   
URL: https://jsfiddle.net/ubw216ze/show
Attachments:
Description Flags
Patch GitHub Desktop Screenshot
none
Test Case from Blink Issue Tracker
none
rendering in safari, firefox, chrome none

Ahmad Saleem
Reported 2023-02-28 04:27:51 PST
Created attachment 465218 [details] Patch GitHub Desktop Screenshot Hi Team, While going through Blink's commit, I came across another where we are failing tests but Chrome Canary 113 is passing them while Firefox Nightly 112 is matching Safari Technology Preview 164. Test case - https://jsfiddle.net/tzj4kq28/show Blink Commit - https://chromium.googlesource.com/chromium/blink/+/ae8d18613905f501a3ce9ba83193fe77e920afbf WebKit Source - https://searchfox.org/wubkat/source/Source/WebCore/platform/ScrollableArea.cpp#857 & https://searchfox.org/wubkat/source/Source/WebCore/platform/ScrollableArea.cpp#899 I have done local merge of this patch but it does not lead to pass on this testcase but it would be good to track and get input whether this is legit bug in WebKit or not. Just raising for tracking. Thanks!
Attachments
Patch GitHub Desktop Screenshot (505.52 KB, image/png)
2023-02-28 04:27 PST, Ahmad Saleem 		no flags
Details
Test Case from Blink Issue Tracker (594 bytes, text/html)
2024-09-08 11:39 PDT, Ahmad Saleem 		no flags
Details
rendering in safari, firefox, chrome (193.16 KB, image/png)
2024-09-08 18:19 PDT, Karl Dubost 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2023-03-07 04:28:14 PST
<rdar://problem/106356373>
Karl Dubost
Comment 2 2023-03-26 20:39:47 PDT
I'm not sure. Safari and Firefox are behaving the same in these tests.
Ahmad Saleem
Comment 3 2024-09-08 11:39:44 PDT
Created attachment 472492 [details] Test Case from Blink Issue Tracker In this attached - at least for me, Chrome Canary 130 and Firefox Nightly 132 are matching and showing 'gray' container while Safari Technology Preview 202 does not.
Karl Dubost
Comment 4 2024-09-08 18:19:44 PDT
Created attachment 472496 [details] rendering in safari, firefox, chrome Very interesting. In Ahmad test case, none of the browsers are behaving the same way. * Safari aligns on top of the scrolling context * Firefox aligns the grey square in a way that it is fully visible and so align to the bottom of the scrolling context. * Chrome aligns in a way that the bottom of the green area is aligned and so it shows half of the grey box. (so… in between?)
Note You need to log in before you can comment on or make changes to this bug.