Hi Team, While going through I came across another failing test in Safari 16.2 & STP160, while the test case passes in Chrome Canary 110 only. Even Firefox Nightly 110 fail it with single point (dot). Test Case - https://jsfiddle.net/54b80mLc/show ^ Make sure to run it in /show mode full width to see the results else on small window even Chrome Canary 111 will fail. Blink Commit - https://chromium.googlesource.com/chromium/blink/+/fb46751341e395b641b691670c1c70e90bae30d4 It might be possible to merge this 1-1 but I haven't looked completely but from quick glance, it seems that it is possible. Just wanted to raise bug so in future, I can give it a try. Thanks!
<rdar://problem/103756227>
It might have some impact on WPT as well: https://wpt.fyi/results/svg/interact/scripted/ellipse-hittest.html?label=experimental&label=master&aligned
Follow-up commit as well - https://src.chromium.org/viewvc/blink?view=revision&revision=189271
Created attachment 465627 [details] GitHub Desktop Patch I took Blink's commit and applied without comment changes and it compiles. We only have to remove one reference variable, since now it would be unused. Also on local testing on WebKit ToT (262194@main) with above leads to passing all WPT tests. Hence, I added WPTImpact tag. Although, I am not sure whether we would have similar performance impacts which was fixed in the follow-up fix but just wanted to keep it here for someone to confirm.
Pre-requisite - https://src.chromium.org/viewvc/blink?view=revision&revision=186650
Right. This is exactly what I pointed at https://github.com/WebKit/WebKit/pull/18740#discussion_r1353911754 I'll create a simple patch which makes this fixed.
Pull request: https://github.com/WebKit/WebKit/pull/19320
Committed 269554@main (339f49b72a95): <https://commits.webkit.org/269554@main> Reviewed commits have been landed. Closing PR #19320 and removing active labels.