Remove explicit selection recovery from RenameRefactoring

There is no real need for this anymore given the new automatic recovery.

Author: MDoerner

Rubberduck.Refactorings/EncapsulateField/EncapsulateFieldRefactoring.cs

@@ -1,6 +1,5 @@
 
 using System;
-using System.Collections.Generic;
 using System.Linq;
 using Rubberduck.Parsing;
 using Rubberduck.Parsing.Grammar;
@@ -23,8 +22,6 @@ public class EncapsulateFieldRefactoring : IRefactoring
         private readonly IRewritingManager _rewritingManager;
         private readonly IRefactoringPresenterFactory _factory;
         private EncapsulateFieldModel _model;
-
-        private readonly HashSet<IModuleRewriter> _referenceRewriters = new HashSet<IModuleRewriter>();
 
         public EncapsulateFieldRefactoring(RubberduckParserState state, IVBE vbe, IIndenter indenter, IRefactoringPresenterFactory factory, IRewritingManager rewritingManager)
         {
@@ -69,7 +66,6 @@ public void Refactor()
                     return;
                 }
 
-                var target = _model.TargetDeclaration;
                 var rewriteSession = _rewritingManager.CheckOutCodePaneSession();
                 AddProperty(rewriteSession);
                 rewriteSession.TryRewrite();

Rubberduck.Refactorings/Rename/RenameRefactoring.cs

@@ -31,7 +31,6 @@ public class RenameRefactoring : IRefactoring
         private readonly IProjectsProvider _projectsProvider;
         private readonly IRewritingManager _rewritingManager;
         private RenameModel _model;
-        private QualifiedSelection? _initialSelection;
         private readonly IDictionary<DeclarationType, Action<IRewriteSession>> _renameActions;
         private readonly List<string> _neverRenameIdentifiers;
 
@@ -47,10 +46,7 @@ public RenameRefactoring(IVBE vbe, IRefactoringPresenterFactory factory, IMessag
             _projectsProvider = projectsProvider;
             _rewritingManager = rewritingManager;
             _model = null;
-            using (var activeCodePane = _vbe.ActiveCodePane)
-            {
-                _initialSelection = activeCodePane.GetQualifiedSelection();
-            }
+
             _renameActions = new Dictionary<DeclarationType, Action<IRewriteSession>>
             {
                 {DeclarationType.Member, RenameMember},
@@ -73,7 +69,6 @@ private RenameModel InitializeModel(Declaration target)
 
         public void Refactor(QualifiedSelection qualifiedSelection)
         {
-            CacheInitialSelection(qualifiedSelection);
             Refactor();
         }
 
@@ -91,7 +86,6 @@ public void Refactor()
                 if (presenter != null)
                 {
                     RefactorImpl(presenter.Model.Target, presenter);
-                    RestoreInitialSelection();
                 }
             }
         }
@@ -109,7 +103,6 @@ public void Refactor(Declaration target)
                 var presenter = container.Value;
 
                 RefactorImpl(target, presenter);
-                RestoreInitialSelection();
             }
         }
 
@@ -585,40 +578,6 @@ private IEnumerable<Declaration> FindEventHandlersForControl(Declaration control
             return Enumerable.Empty<Declaration>();
         }
 
-        private void CacheInitialSelection(QualifiedSelection qSelection)
-        {
-            var component = _projectsProvider.Component(qSelection.QualifiedName);
-            using (var codeModule = component.CodeModule)
-            using (var codePane = codeModule.CodePane)
-            {
-                if (!codePane.IsWrappingNullReference)
-                {
-                    _initialSelection = codePane.GetQualifiedSelection();
-                }
-            }
-        }
-
-        private void RestoreInitialSelection()
-        {
-            if (!_initialSelection.HasValue)
-            {
-                return;
-            }
-            
-            var qualifiedSelection = _initialSelection.Value;
-            var component = _projectsProvider.Component(qualifiedSelection.QualifiedName);
-            using (var codeModule = component.CodeModule)
-            {
-                using (var codePane = codeModule.CodePane)
-                {
-                    if (!codePane.IsWrappingNullReference)
-                    {
-                        codePane.Selection = qualifiedSelection.Selection;
-                    }
-                }
-            }
-        }
-
         private void PresentRenameErrorMessage(string errorMsg)
         {
             _messageBox?.NotifyWarn(errorMsg, RubberduckUI.RenameDialog_Caption);