Minor performance improvements for InspectionResults View

Ignoring an inspection type no longer triggers a full reparse.
Grouping behaviour changes are in a defer-cycle to avoid a rerender without any grouping.

Author: Vogel612

Rubberduck.Core/UI/Inspections/InspectionResultsViewModel.cs

@@ -230,9 +230,12 @@ public InspectionResultGrouping Grouping
                 }
 
                 _grouping = value;
-                // updating GroupDescriptions forces a Refresh
-                Results.GroupDescriptions.Clear();
-                Results.GroupDescriptions.Add(GroupDescriptions[_grouping]);
+                // Deferring refresh to avoid a rerendering without grouping
+                using (Results.DeferRefresh())
+                {
+                    Results.GroupDescriptions.Clear();
+                    Results.GroupDescriptions.Add(GroupDescriptions[_grouping]);
+                }
                 OnPropertyChanged();
             }
         }
@@ -308,7 +311,6 @@ private void OpenSettings(object param)
         }
 
         private bool _canQuickFix;
-
         public bool CanQuickFix
         {
             get => _canQuickFix;
@@ -445,13 +447,12 @@ private async void RefreshInspections(CancellationToken token)
                 }
 
                 stopwatch.Stop();
-                LogManager.GetCurrentClassLogger().Trace("Inspection results rendered in {0}ms", stopwatch.ElapsedMilliseconds);
+                Logger.Trace("Inspection results rendered in {0}ms", stopwatch.ElapsedMilliseconds);
             });
         }
 
-        private void InvalidateStaleInspectionResults(ICollection<QualifiedModuleName> modifiedModules)
+        private void InvalidateUIStaleInspectionResults(ICollection<IInspectionResult> staleResults)
         {
-            var staleResults = _results.Where(result => result.ChangesInvalidateResult(modifiedModules)).ToList();
             _uiDispatcher.Invoke(() =>
             {
                 foreach (var staleResult in staleResults)
@@ -462,6 +463,13 @@ private void InvalidateStaleInspectionResults(ICollection<QualifiedModuleName> m
             });
         }
 
+        private void InvalidateStaleInspectionResults(ICollection<QualifiedModuleName> modifiedModules)
+        {
+            // materialize the collection to take work off of the UI thread
+            var staleResults = _results.Where(result => result.ChangesInvalidateResult(modifiedModules)).ToList();
+            InvalidateUIStaleInspectionResults(staleResults);
+        }
+
         private void ExecuteQuickFixCommand(object parameter)
         {
             var quickFix = parameter as IQuickFix;
@@ -554,14 +562,12 @@ private void ExecuteDisableInspectionCommand(object parameter)
 
             Task.Run(() => _configService.Save(config));
 
-            _uiDispatcher.Invoke(() =>
-            {
-                RefreshCommand.Execute(null);
-            });
+            // remove inspection results of the selected inspection from the UI
+            // collection is materialized to take work off of the UI thread
+            InvalidateUIStaleInspectionResults(_results.Where(i => i.Inspection == _selectedInspection).ToList());
         }
 
         private bool _canDisableInspection;
-
         public bool CanDisableInspection
         {
             get => _canDisableInspection;