Bug 250561 - REGRESSION(257650@main): Multiple regressions on input-events-get-target-ranges* WPT input event tests
Summary: REGRESSION(257650@main): Multiple regressions on input-events-get-target-rang...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: UI Events (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: BrowserCompat, InRadar
Depends on:
Blocks:
 
Reported: 2023-01-13 03:48 PST by Ahmad Saleem
Modified: 2023-01-17 00:37 PST (History)
7 users (show)

See Also:


Attachments
Regressing WPT Tests (377.85 KB, image/png)
2023-01-13 03:48 PST, Ahmad Saleem
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Ahmad Saleem 2023-01-13 03:48:58 PST
Created attachment 464482 [details]
Regressing WPT Tests

Hi Team,

While going through STP161 release and noticing passes, I noticed that we regressed in Input Events test cases:

I don't know whether it will work or not:

https://wpt.fyi/results/input-events?diff&filter=ADC&run_id=5076022454648832&run_id=5119896183111680

But I am attaching screenshot as well.

Thanks!
Comment 1 Ahmad Saleem 2023-01-13 05:32:34 PST
Our failing reduced with exact numbers matching now with Chrome so pretty sure this is regression introduced by myself while doing some "Blink Merges" and if I have to guess, it would be this:

https://github.com/WebKit/WebKit/commit/31be36f212e4872dfd9761736dba5814453ac8a2
Comment 2 Alexey Proskuryakov 2023-01-13 09:46:54 PST
Thank you!
Comment 3 Radar WebKit Bug Importer 2023-01-13 09:47:35 PST
<rdar://problem/104227951>
Comment 4 Karl Dubost 2023-01-15 17:59:15 PST
It's easier to read by changing the order here.
https://wpt.fyi/results/input-events?diff&filter=ADC&run_id=5119896183111680&run_id=5076022454648832&view=subtest
Comment 5 Ahmad Saleem 2023-01-15 18:01:03 PST
Do we have any regression tool similar to mozregression to automate this so I can confirm my culprit from Comment 01? Appreciate if someone can point me to right direction?
Comment 6 Alexey Proskuryakov 2023-01-15 22:03:55 PST
Yes, Tools/Scripts/bisect-builds. It hasn't been used in a while though, so could have bit rotted.
Comment 7 Ahmad Saleem 2023-01-16 09:11:44 PST
(In reply to Alexey Proskuryakov from comment #6)
> Yes, Tools/Scripts/bisect-builds. It hasn't been used in a while though, so
> could have bit rotted.

From where I can download old builds in order to confirm that above commit is culprit?

Or if you know someone in Apple or WebKit team, who could do this or guide me?
Comment 8 Ahmad Saleem 2023-01-16 09:11:58 PST
(In reply to Ahmad Saleem from comment #7)
> (In reply to Alexey Proskuryakov from comment #6)
> > Yes, Tools/Scripts/bisect-builds. It hasn't been used in a while though, so
> > could have bit rotted.
> 
> From where I can download old builds in order to confirm that above commit
> is culprit?
> 
> Or if you know someone in Apple or WebKit team, who could do this or guide
> me?

https://webkit.org/build-archives/#mac-ventura-x86_64%20arm64 <-- This does not have old builds.
Comment 9 Alexey Proskuryakov 2023-01-16 11:28:15 PST
We don't show them on the webpage, but we store them for a long time. In any case, bisect-builds downloads on its own.

Please reach out in Slack or email if it's not working - this bug is probably not the right place to discuss bisection.
Comment 10 Ryosuke Niwa 2023-01-16 23:44:29 PST
Sigh... who the fuck wrote this test? It's totally incomprehensible.
Comment 11 Ryosuke Niwa 2023-01-16 23:44:47 PST
Regression ranges:
Release 160 (Safari 16.4, WebKit 18615.1.14.3)
257350@main

Release 161 (Safari 16.4, WebKit 18615.1.17.6)
257522@main
Comment 12 Ryosuke Niwa 2023-01-17 00:18:53 PST
257522@main
392 pass 48 fail

257612@main
392 pass 48 fail

257634@main
392 pass 48 fail

257646@main
392 pass 48 fail

257649@main
392 pass 48 fail

257650@main
260 pass 180 fail

257652@main
260 pass 180 fail

257657@main
260 pass 180 fail

257703@main
260 pass 180 fail

257884@main
260 pass 180 fail

258246@main
260 pass 180 fail

258970@main
260 pass 180 fail
Comment 13 Ryosuke Niwa 2023-01-17 00:21:00 PST
The above is results for input-events/input-events-get-target-ranges-joining-dl-elements.tentative.html?Backspace
Comment 14 Ryosuke Niwa 2023-01-17 00:26:20 PST
input-events-get-target-ranges-deleting-in-list-items.tentative.html?Backspace,ol has the same regression point:

257649@main
238 pass 76 fail

257650@main
219 pass 95 fail
Comment 15 Ryosuke Niwa 2023-01-17 00:27:43 PST
Reverting the culprit in https://bugs.webkit.org/show_bug.cgi?id=250700