Cherry-pick in telemetry changes for 17.9 (#9559)

* Fix type check in IsComplexValue

* Fix telemetry (again!)

When reducing allocations in our telemetry, I inadvertently created one array to be too small, resulting in our correlation ID being overwritten for telemetry with three properties.

---------

Co-authored-by: Dustin Campbell <[email protected]>

Author: phil-allen-msft

src/Razor/src/Microsoft.AspNetCore.Razor.ProjectEngineHost/Telemetry/TelemetryScope.cs

@@ -93,7 +93,7 @@ public static TelemetryScope Create(ITelemetryReporter reporter, string name, Se
 
     public static TelemetryScope Create(ITelemetryReporter reporter, string name, Severity severity, Property property1, Property property2, Property property3)
     {
-        var array = new Property[3];
+        var array = new Property[4];
         array[0] = property1;
         array[1] = property2;
         array[2] = property3;

src/Razor/src/Microsoft.VisualStudio.Editor.Razor/Telemetry/TelemetryReporter.cs

@@ -95,7 +95,9 @@ private static void AddToProperties(IDictionary<string, object?> properties, Pro
         }
 
         static bool IsComplexValue(object? o)
-            => o is Type type && Type.GetTypeCode(type) == TypeCode.Object;
+        {
+            return o?.GetType() is Type type && Type.GetTypeCode(type) == TypeCode.Object;
+        }
     }
 
     public void ReportFault(Exception exception, string? message, params object?[] @params)