DescriptionTelemetry: Add more details to inspector_backend exceptions.
Previously, inspector_backend would always throw a
DevtoolsTargetCrashException. Now, it throws either a
DevtoolsTargetCrashException or a TimeoutException. The CL also adds the
debugger_url to the exception as further debugging information.
There were exactly 2 files that tried to catch a DevtoolsTargetCrashException.
I changed fast_navigation_profile_extender to catch Error instead. I will be
improving this class in the future to have better exception handling.
chrome_proxy_measurements caught a DevtoolsTargetCrashException, but its
behavior is better suited by catching a TimeoutException.
BUG=460625
Committed: https://crrev.com/910d6e733bac30f15f76054eb101a81987f8c675
Cr-Commit-Position: refs/heads/master@{#319193}
Patch Set 1 #Patch Set 2 : #Patch Set 3 : Rebase against top of tree. #
Messages
Total messages: 14 (6 generated)
|