Merge pull request #3019 from davkean/TimerId

Put constant into own class

Author: davkean

src/Microsoft.VisualStudio.ProjectSystem.Managed.VS/CodeMarkerTimerId.cs

@@ -0,0 +1,17 @@
+// Copyright (c) Microsoft.  All Rights Reserved.  Licensed under the Apache License, Version 2.0.  See License.txt in the project root for license information.
+
+using Microsoft.Internal.Performance;
+
+namespace Microsoft.VisualStudio
+{
+    /// <summary>
+    ///     Represents timer IDs that are passed to <see cref="CodeMarkers.CodeMarker"/>.
+    /// </summary>
+    internal static class CodeMarkerTimerId
+    {
+        /// <summary>
+        ///     Indicates that NuGet package restore has finished.
+        /// </summary>
+        public const int PerfPackageRestoreEnd = 7343;
+    }
+}

src/Microsoft.VisualStudio.ProjectSystem.Managed.VS/ProjectSystem/VS/NuGet/NuGetRestorer.cs

@@ -25,7 +25,7 @@ internal class NuGetRestorer : OnceInitializedOnceDisposedAsync
         private readonly IActiveProjectConfigurationRefreshService _activeProjectConfigurationRefreshService;
         private IDisposable _targetFrameworkSubscriptionLink;
         private DisposableBag _designTimeBuildSubscriptionLink;
-        private const int perfPackageRestoreEnd = 7343;
+        
 
         private static ImmutableHashSet<string> _targetFrameworkWatchedRules = Empty.OrdinalIgnoreCaseStringSet
             .Add(NuGetRestore.SchemaName);
@@ -171,7 +171,7 @@ await _solutionRestoreService
                                     _projectVsServices.Project.Services.ProjectAsynchronousTasks.UnloadCancellationToken)
                                .ConfigureAwait(false);
 
-                        CodeMarkers.Instance.CodeMarker(perfPackageRestoreEnd);
+                        CodeMarkers.Instance.CodeMarker(CodeMarkerTimerId.PerfPackageRestoreEnd);
 
                     }),
                     ProjectCriticalOperation.Build | ProjectCriticalOperation.Unload | ProjectCriticalOperation.Rename,