RESOLVED FIXED 225221
[run-api-tests] Use webkitcorepy's TaskPool 
https://bugs.webkit.org/show_bug.cgi?id=225221
Summary [run-api-tests] Use webkitcorepy's TaskPool
Jonathan Bedard
Reported 2021-04-29 17:29:30 PDT
In order for run-api-tests to support Python 3.8 on Apple's platforms, we need run-api-tests to use webkitcorepy's TaskPool.
Attachments
Patch (35.17 KB, patch)
2021-05-03 17:12 PDT, Jonathan Bedard 		no flags
DetailsFormatted DiffDiff
Patch (43.23 KB, patch)
2021-05-04 15:37 PDT, Jonathan Bedard 		no flags
DetailsFormatted DiffDiff
Patch (35.25 KB, patch)
2021-05-05 08:46 PDT, Jonathan Bedard 		no flags
DetailsFormatted DiffDiff
Patch (37.29 KB, patch)
2021-05-05 10:53 PDT, Jonathan Bedard 		no flags
DetailsFormatted DiffDiff
Patch (34.50 KB, patch)
2021-05-05 15:46 PDT, Jonathan Bedard 		ews-feeder: commit-queue-
DetailsFormatted DiffDiff
Patch for landing (34.53 KB, patch)
2021-05-05 16:55 PDT, Jonathan Bedard 		no flags
DetailsFormatted DiffDiff
Show Obsolete (5) View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2021-04-29 17:30:02 PDT
<rdar://problem/77352465>
Jonathan Bedard
Comment 2 2021-05-03 17:12:40 PDT
Created attachment 427625 [details] Patch
Jonathan Bedard
Comment 3 2021-05-04 15:37:25 PDT
Created attachment 427703 [details] Patch
Jonathan Bedard
Comment 4 2021-05-05 08:46:45 PDT
Created attachment 427768 [details] Patch
Jonathan Bedard
Comment 5 2021-05-05 10:53:16 PDT
Created attachment 427780 [details] Patch
dewei_zhu
Comment 6 2021-05-05 15:06:06 PDT
Comment on attachment 427780 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=427780&action=review > Tools/Scripts/webkitpy/api_tests/runner.py:225 > + if status == Runner.STATUS_PASSED and not output and Runner.instance.port.get_option('quiet'): > + Runner.instance.printer.write_update('{} {} {}'.format(worker, test, Runner.NAME_FOR_STATUS[status])) > + else: > + Runner.instance.printer.write_update('{} {} {}'.format(worker, test, Runner.NAME_FOR_STATUS[status])) Are those expected to be identical? > Tools/Scripts/webkitpy/api_tests/runner.py:233 > +class Runner(object): Moving Runner to the bottom of this file make it more difficult to compare if we made any change to this class. Maybe move it back?
Jonathan Bedard
Comment 7 2021-05-05 15:46:50 PDT
Created attachment 427816 [details] Patch
dewei_zhu
Comment 8 2021-05-05 16:39:42 PDT
Comment on attachment 427816 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=427816&action=review r=me > Tools/ChangeLog:18 > + (_Worker.setup): Pass port object to worker process. > + (_Worker.setup): Pass port object to worker process. Nit: those two lines are identical, we only need one, right? > Tools/Scripts/webkitpy/api_tests/runner.py:118 > + Runner.instance = self Discussed in person, let's add an assertion that `Runner.instance is None` before this assignment.
Jonathan Bedard
Comment 9 2021-05-05 16:55:57 PDT
Created attachment 427822 [details] Patch for landing
EWS
Comment 10 2021-05-05 17:22:22 PDT
Committed r277061 (237367@main): <https://commits.webkit.org/237367@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 427822 [details].
Note You need to log in before you can comment on or make changes to this bug.