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.
<rdar://problem/77352465>
Created attachment 427625 [details] Patch
Created attachment 427703 [details] Patch
Created attachment 427768 [details] Patch
Created attachment 427780 [details] Patch
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?
Created attachment 427816 [details] Patch
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.
Created attachment 427822 [details] Patch for landing
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].