Bug 253070 - Fix alignToClosestEdge semantics in ScrollableArea::getRectToExposeForScrollIntoView
Summary: Fix alignToClosestEdge semantics in ScrollableArea::getRectToExposeForScrollI...
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Scrolling (show other bugs)
Version: Safari Technology Preview
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL: https://jsfiddle.net/ubw216ze/show
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2023-02-28 04:27 PST by Ahmad Saleem
Modified: 2023-06-03 08:46 PDT (History)
3 users (show)

See Also:

Attachments
Patch GitHub Desktop Screenshot (505.52 KB, image/png)
2023-02-28 04:27 PST, Ahmad Saleem 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ahmad Saleem 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!
Comment 1 Radar WebKit Bug Importer 2023-03-07 04:28:14 PST 
<rdar://problem/106356373>
Comment 2 Karl Dubost 2023-03-26 20:39:47 PDT 
I'm not sure. 
Safari and Firefox are behaving the same in these tests.