DevtoolsClientConfig instance is not closed properly in DevtoolsClientBackend

catapult-project / catapult

Deprecated Catapult GitHub. Please instead use http://crbug.com "Speed>Benchmarks" component for bugs and https://chromium.googlesource.com/catapult for downloading and editing source code..

https://chromium.googlesource.com/catapult

BSD 3-Clause "New" or "Revised" License

1.91k stars 562 forks source link

DevtoolsClientConfig instance is not closed properly in DevtoolsClientBackend #4607

Closed andrey-malets closed 5 years ago

andrey-malets commented 5 years ago

This must happen in DevtoolsClientBackend.Close(): https://github.com/catapult-project/catapult/blob/08081e7b7e2d692933fa276ff2cfaae194956d3a/telemetry/telemetry/internal/backends/chrome_inspector/devtools_client_backend.py#L270

andrey-malets commented 5 years ago

Fixed in a923c2a6de79f0f209157ab09849d695a98f4470.