RESOLVED FIXED 232847
Convert more wheel event tests to UIHelper.mouseWheelSequence() 
https://bugs.webkit.org/show_bug.cgi?id=232847
Summary Convert more wheel event tests to UIHelper.mouseWheelSequence()
Simon Fraser (smfr)
Reported 2021-11-08 13:43:12 PST
Convert more wheel event tests to UIHelper.mouseWheelSequence()
Attachments
Patch (19.16 KB, patch)
2021-11-08 13:44 PST, Simon Fraser (smfr) 		no flags
DetailsFormatted DiffDiff
Patch (19.78 KB, patch)
2021-11-08 14:10 PST, Simon Fraser (smfr) 		wenson_hsieh: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Simon Fraser (smfr)
Comment 1 2021-11-08 13:44:24 PST
Created attachment 443598 [details] Patch
Tim Nguyen (:ntim)
Comment 2 2021-11-08 13:49:19 PST
Comment on attachment 443598 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=443598&action=review You can simplify the code in ui-helper.js > LayoutTests/resources/ui-helper.js:111 > + static async mouseWheelSequence(eventStream, options) static async mouseWheelSequence(eventStream, { waitForCompletion = true }) > LayoutTests/resources/ui-helper.js:119 > + const waitForScrollCompletion = options?.waitForCompletion ?? true; > + if (waitForScrollCompletion) if (waitForCompletion) > LayoutTests/resources/ui-helper.js:132 > + if (waitForScrollCompletion) ditto
Tim Nguyen (:ntim)
Comment 3 2021-11-08 13:56:36 PST
Comment on attachment 443598 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=443598&action=review > LayoutTests/fast/scrolling/mac/momentum-axis-locking.html:162 > + await UIHelper.mouseWheelSequence({ events: events }); here and below: { events: events } -> { events } works as well
Wenson Hsieh
Comment 4 2021-11-08 14:09:44 PST
Comment on attachment 443598 [details] Patch r=me (with Tim's suggestions)
Simon Fraser (smfr)
Comment 5 2021-11-08 14:10:31 PST
Created attachment 443604 [details] Patch
Simon Fraser (smfr)
Comment 6 2021-11-09 07:28:24 PST
https://trac.webkit.org/changeset/285498/webkit
Radar WebKit Bug Importer
Comment 7 2021-11-09 07:29:21 PST
<rdar://problem/85201668>
Note You need to log in before you can comment on or make changes to this bug.