Merge pull request #1623 from paulvanbrenk/removeLogger

Remove logger which is no longer used,

Author: paulvanbrenk

Nodejs/Product/Nodejs/Logging/INodejsToolsLogger.cs

@@ -245,11 +245,7 @@
     <Compile Include="Debugger\LocalFileNameMapper.cs" />
     <Compile Include="Debugger\NodeProcess.cs" />
     <Compile Include="Debugger\ScriptTree.cs" />
-    <Compile Include="Logging\InMemoryLogger.cs" />
-    <Compile Include="Logging\INodejsToolsLogger.cs" />
     <Compile Include="Logging\LiveLogger.cs" />
-    <Compile Include="Logging\NodejsToolsLogEvent.cs" />
-    <Compile Include="Logging\NodejsToolsLogger.cs" />
     <Compile Include="NodejsPackage.Debugger.cs" />
     <Compile Include="NpmUI\LastRefreshedMessageProvider.cs" />
     <Compile Include="NpmUI\NpmInstallWindowResources.Designer.cs">

Nodejs/Product/Nodejs/Logging/InMemoryLogger.cs

@@ -6,14 +6,12 @@
 using System.Diagnostics;
 using System.Globalization;
 using System.IO;
-using System.Linq;
 using System.Runtime.InteropServices;
 using System.Windows.Forms;
 using Microsoft.NodejsTools.Commands;
 using Microsoft.NodejsTools.Debugger.DebugEngine;
 using Microsoft.NodejsTools.Debugger.Remote;
 using Microsoft.NodejsTools.Jade;
-using Microsoft.NodejsTools.Logging;
 using Microsoft.NodejsTools.Options;
 using Microsoft.NodejsTools.Project;
 using Microsoft.NodejsTools.ProjectWizard;
@@ -66,13 +64,13 @@ namespace Microsoft.NodejsTools
     internal sealed partial class NodejsPackage : CommonPackage
     {
         internal const string NodeExpressionEvaluatorGuid = "{F16F2A71-1C45-4BAB-BECE-09D28CFDE3E6}";
-        private IContentType _contentType;
+        private IContentType contentType;
         internal static NodejsPackage Instance;
         internal HashSet<ITextBuffer> ChangedBuffers = new HashSet<ITextBuffer>();
-        private NodejsToolsLogger _logger;
+
         // Hold references for the subscribed events. Otherwise the callbacks will be garbage collected
         // after the initialization
-        private List<EnvDTE.CommandEvents> _subscribedCommandEvents = new List<EnvDTE.CommandEvents>();
+        private readonly List<EnvDTE.CommandEvents> subscribedCommandEvents = new List<EnvDTE.CommandEvents>();
 
         /// <summary>
         /// Default constructor of the package.
@@ -125,21 +123,18 @@ protected override void Initialize()
                 new SendFeedbackCommand(),
                 new ShowDocumentationCommand()
             };
+
             try
             {
                 commands.Add(new AzureExplorerAttachDebuggerCommand());
             }
             catch (NotSupportedException)
             {
             }
-            RegisterCommands(commands, Guids.NodejsCmdSet);
 
+            RegisterCommands(commands, Guids.NodejsCmdSet);
             MakeDebuggerContextAvailable();
 
-            InitializeLogging();
-
-            InitializeTelemetry();
-
             // The variable is inherited by child processes backing Test Explorer, and is used in
             // the NTVS test discoverer and test executor to connect back to VS.
             Environment.SetEnvironmentVariable(NodejsConstants.NodeToolsProcessIdEnvironmentVariable, Process.GetCurrentProcess().Id.ToString());
@@ -174,25 +169,11 @@ private void SubscribeToVsCommandEvents(
             {
                 targetEvent.AfterExecute += afterExecute;
             }
-            this._subscribedCommandEvents.Add(targetEvent);
-        }
-
-        private void InitializeLogging()
-        {
-            this._logger = new NodejsToolsLogger(this.ComponentModel.GetExtensions<INodejsToolsLogger>().ToArray());
-        }
-
-        private void InitializeTelemetry()
-        {
-            // Fetch the session synchronously on the UI thread; if this doesn't happen before we try using this on 
-            // the background thread then the VS process will deadlock.
-            TelemetryHelper.Initialize();
+            this.subscribedCommandEvents.Add(targetEvent);
         }
 
         public new IComponentModel ComponentModel => this.GetComponentModel();
 
-        internal NodejsToolsLogger Logger => this._logger;
-
         /// <summary>
         /// Makes the debugger context available - this enables our debugger when we're installed into
         /// a SKU which doesn't support every installed debugger.
@@ -201,8 +182,8 @@ private void MakeDebuggerContextAvailable()
         {
             var monitorSelection = (IVsMonitorSelection)GetService(typeof(SVsShellMonitorSelection));
             var debugEngineGuid = AD7Engine.DebugEngineGuid;
-            uint contextCookie;
-            if (ErrorHandler.Succeeded(monitorSelection.GetCmdUIContextCookie(ref debugEngineGuid, out contextCookie)))
+
+            if (ErrorHandler.Succeeded(monitorSelection.GetCmdUIContextCookie(ref debugEngineGuid, out var contextCookie)))
             {
                 ErrorHandler.ThrowOnFailure(monitorSelection.SetCmdUIContext(contextCookie, 1));
             }
@@ -262,11 +243,11 @@ private IContentType ReplContentType
         {
             get
             {
-                if (this._contentType == null)
+                if (this.contentType == null)
                 {
-                    this._contentType = this.ComponentModel.GetService<IContentTypeRegistryService>().GetContentType(NodejsConstants.TypeScript);
+                    this.contentType = this.ComponentModel.GetService<IContentTypeRegistryService>().GetContentType(NodejsConstants.TypeScript);
                 }
-                return this._contentType;
+                return this.contentType;
             }
         }
 
@@ -358,15 +339,5 @@ public string BrowseForDirectory(IntPtr owner, string initialDirectory = null)
                 }
             }
         }
-
-        internal static void NavigateTo(string filename, int line, int col)
-        {
-            VsUtilities.NavigateTo(Instance, filename, Guid.Empty, line, col);
-        }
-
-        internal static void NavigateTo(string filename, int pos)
-        {
-            VsUtilities.NavigateTo(Instance, filename, Guid.Empty, pos);
-        }
     }
 }

Nodejs/Product/Nodejs/Logging/NodejsToolsLogEvent.cs

@@ -52,17 +52,10 @@ public NodejsProjectLauncher(NodejsProjectNode project)
         #region IProjectLauncher Members
         public int LaunchProject(bool debug)
         {
-            NodejsPackage.Instance.Logger.LogEvent(Logging.NodejsToolsLogEvent.Launch, debug ? 1 : 0);
-            return Start(ResolveStartupFile(), debug);
+            return LaunchFile(ResolveStartupFile(), debug);
         }
 
         public int LaunchFile(string file, bool debug)
-        {
-            NodejsPackage.Instance.Logger.LogEvent(Logging.NodejsToolsLogEvent.Launch, debug ? 1 : 0);
-            return Start(file, debug);
-        }
-
-        private int Start(string file, bool debug)
         {
             var nodePath = GetNodePath();
 

Nodejs/Product/Nodejs/Logging/NodejsToolsLogger.cs

@@ -1,6 +1,5 @@
 // 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.VisualStudio.Shell;
 using Microsoft.VisualStudio.Telemetry;
 using static Microsoft.NodejsTools.Telemetry.TelemetryEvents;
 using static Microsoft.NodejsTools.Telemetry.TelemetryProperties;
@@ -12,12 +11,6 @@ namespace Microsoft.NodejsTools.Telemetry
     /// </summary>
     internal static class TelemetryHelper
     {
-        public static void Initialize()
-        {
-            ThreadHelper.ThrowIfNotOnUIThread();
-            var defaultSession = TelemetryService.DefaultSession;
-        }
-
         public static void LogProjectImported()
         {
             TelemetryService.DefaultSession.PostUserTask(ProjectImported, TelemetryResult.Success);