Pass target elements being renamed into validator.

pkg/analysis_server/lib/src/services/refactoring/rename_local.dart

 // Copyright (c) 2014, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 import 'dart:async';
 
-import 'package:analysis_server/src/protocol_server.dart' hide Element;
+import 'package:analysis_server/src/protocol_server.dart'
+    hide Element, ElementKind;
 import 'package:analysis_server/src/services/correction/status.dart';
 import 'package:analysis_server/src/services/correction/util.dart';
 import 'package:analysis_server/src/services/refactoring/naming_conventions.dart';
 import 'package:analysis_server/src/services/refactoring/refactoring.dart';
 import 'package:analysis_server/src/services/refactoring/rename.dart';
 import 'package:analysis_server/src/services/search/hierarchy.dart';
 import 'package:analysis_server/src/services/search/search_engine.dart';
 import 'package:analyzer/dart/ast/ast.dart';
 import 'package:analyzer/dart/ast/visitor.dart';
 import 'package:analyzer/dart/element/element.dart';
@@ skipping 29 matching lines @@
     return "Rename Local Variable";
   }
 
   @override
   Future<RefactoringStatus> checkFinalConditions() async {
     RefactoringStatus result = new RefactoringStatus();
     await _prepareElements();
     for (LocalElement element in elements) {
       CompilationUnit unit = await unitCache.getUnit(element);
       if (unit != null) {
-        SourceRange elementRange = element.visibleRange;
-        unit.accept(new _ConflictValidatorVisitor(this, result, elementRange));
+        unit.accept(new _ConflictValidatorVisitor(result, newName, element));
       }
     }
     return result;
   }
 
   @override
   RefactoringStatus checkNewName() {
     RefactoringStatus result = super.checkNewName();
     if (element is LocalVariableElement) {
       result.addStatus(validateVariableName(newName));
@@ skipping 35 matching lines @@
           }
         }
       }
     } else {
       elements = new Set.from([element]);
     }
   }
 }
 
 class _ConflictValidatorVisitor extends RecursiveAstVisitor {
-  final RenameLocalRefactoringImpl refactoring;
   final RefactoringStatus result;
-  final SourceRange elementRange;
+  final String newName;
+  final LocalElement target;
   final Set<Element> conflictingLocals = new Set<Element>();
 
-  _ConflictValidatorVisitor(this.refactoring, this.result, this.elementRange);
+  _ConflictValidatorVisitor(this.result, this.newName, this.target);
 
   @override
   visitSimpleIdentifier(SimpleIdentifier node) {
     Element nodeElement = node.bestElement;
-    String newName = refactoring.newName;
     if (nodeElement != null && nodeElement.name == newName) {
-      // duplicate declaration
-      if (node.inDeclarationContext() &&
-          haveIntersectingRanges(refactoring.element, nodeElement)) {
+      // Duplicate declaration.
+      if (node.inDeclarationContext() && _isVisibleWithTarget(nodeElement)) {
         conflictingLocals.add(nodeElement);
         String nodeKind = nodeElement.kind.displayName;
         String message = "Duplicate $nodeKind '$newName'.";
         result.addError(message, newLocation_fromElement(nodeElement));
         return;
       }
       if (conflictingLocals.contains(nodeElement)) {
         return;
       }
-      // shadowing referenced element
-      if (elementRange != null &&
-          elementRange.contains(node.offset) &&
+      // Shadowing by the target element.
+      SourceRange targetRange = target.visibleRange;
+      if (targetRange != null &&
+          targetRange.contains(node.offset) &&
           !node.isQualified &&
           !_isNamedExpressionName(node)) {
         nodeElement = getSyntheticAccessorVariable(nodeElement);
         String nodeKind = nodeElement.kind.displayName;
         String nodeName = getElementQualifiedName(nodeElement);
         String nameElementSourceName = nodeElement.source.shortName;
-        String refKind = refactoring.element.kind.displayName;
+        String refKind = target.kind.displayName;
         String message = 'Usage of $nodeKind "$nodeName" declared in '
             '"$nameElementSourceName" will be shadowed by renamed $refKind.';
         result.addError(message, newLocation_fromNode(node));
       }
     }
   }
 
+  /**
+   * Returns whether [element] and [target] are visible together.
+   */
+  bool _isVisibleWithTarget(Element element) {
+    if (element is LocalElement) {
+      SourceRange targetRange = target.visibleRange;
+      SourceRange elementRange = element.visibleRange;
+      return targetRange != null &&
+          elementRange != null &&
+          elementRange.intersects(targetRange);
+    }
+    return false;
+  }
+
   static bool _isNamedExpressionName(SimpleIdentifier node) {
     return node.parent is Label && node.parent.parent is NamedExpression;
   }
 }