Do not post fault telemetry for cancellation exceptions

[jeffkl]

Bug

Fixes: https://github.com/NuGet/Client.Engineering/issues/3013

Description

Our code is posting fault telemetry for cancellations and it should not be. The PackageItemViewModel was assuming that HTTP timeouts were throwing a TaskCanceledException but that is not the case. In TimeoutUtility we detect a timeout and throw a TimeoutException instead:

NuGet.Client/src/NuGet.Core/NuGet.Protocol/Utility/TimeoutUtility.cs

Line 46 in 3872959

throw new TimeoutException(timeoutMessage);

This change updates the view model based on my findings and blocks any OperationCancelledException from posting telemetry. Any code in the future where we want to post fault telemetry but its not really a cancellation should wrap the exception as something else like a TimeoutException.

PR Checklist

• Meaningful title, helpful description and a linked NuGet/Home issue
• Added tests
• Link to an issue or pull request to update docs if this PR changes settings, environment variables, new feature, etc.

[jeffkl]

I had to rebase, re-requesting reviews...