cli: stop treating chrome launcher kill() as async

GoogleChrome / lighthouse

Automated auditing, performance metrics, and best practices for the web.

https://developer.chrome.com/docs/lighthouse/overview/

Apache License 2.0

28.01k stars 9.31k forks source link

cli: stop treating chrome launcher kill() as async #15998

Closed connorjclark closed 1 month ago

connorjclark commented 1 month ago

since https://github.com/GoogleChrome/chrome-launcher/pull/268 kill() is sync. so the timeout thing we are doing (which only really did anything on windows when we waited for taskkill to finish - otherwise sending a kill signal is instant) is no longer needed.