Migrate rename_class_member

Change-Id: I5a574c3858a9d59d01952052941e08dde4f1e6b9
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/398924
Reviewed-by: Konstantin Shcheglov <[email protected]>
Commit-Queue: Brian Wilkerson <[email protected]>

Author: bwilkerson

pkg/analysis_server/analyzer_use_new_elements.txt

@@ -48,7 +48,6 @@ lib/src/services/refactoring/legacy/refactoring.dart
 lib/src/services/refactoring/legacy/refactoring_internal.dart
 lib/src/services/refactoring/legacy/refactoring_manager.dart
 lib/src/services/refactoring/legacy/rename.dart
-lib/src/services/refactoring/legacy/rename_class_member.dart
 lib/src/services/refactoring/legacy/rename_constructor.dart
 lib/src/services/refactoring/legacy/rename_extension_member.dart
 lib/src/services/refactoring/legacy/rename_import.dart

pkg/analysis_server/lib/src/services/correction/util.dart

@@ -109,6 +109,11 @@ String getElementKindName(Element element) {
   return element.kind.displayName;
 }
 
+/// Return the name of the kind of the [element].
+String getElementKindName2(Element2 element) {
+  return element.kind.displayName;
+}
+
 /// Returns the name to display in the UI for the given [Element].
 String getElementQualifiedName(Element element) {
   var kind = element.kind;
@@ -127,6 +132,24 @@ String getElementQualifiedName(Element element) {
   }
 }
 
+/// Returns the name to display in the UI for the given [element].
+String getElementQualifiedName2(Element2 element) {
+  var kind = element.kind;
+  if (kind == ElementKind.FIELD || kind == ElementKind.METHOD) {
+    return '${element.enclosingElement2!.displayName}.${element.displayName}';
+  } else if (kind == ElementKind.LIBRARY) {
+    // Libraries may not have names, so use a path relative to the context root.
+    var session = element.session!;
+    var pathContext = session.resourceProvider.pathContext;
+    var rootPath = session.analysisContext.contextRoot.root.path;
+    var library = element as LibraryElement;
+
+    return pathContext.relative(library.source.fullName, from: rootPath);
+  } else {
+    return element.displayName;
+  }
+}
+
 /// Returns a class or an unit member enclosing the given [input].
 AstNode? getEnclosingClassOrUnitMember(AstNode input) {
   var member = input;

pkg/analysis_server/lib/src/services/refactoring/legacy/extract_method.dart

@@ -646,15 +646,15 @@ final class ExtractMethodRefactoringImpl extends RefactoringImpl
       return validateCreateFunction(_searchEngine, libraryElement, name);
     }
     // method of class
-    InterfaceElement? interfaceElement;
+    InterfaceElement2? interfaceElement;
     if (parent is ClassDeclaration) {
-      interfaceElement = parent.declaredElement!;
+      interfaceElement = parent.declaredFragment?.element;
     } else if (parent is EnumDeclaration) {
-      interfaceElement = parent.declaredElement!;
+      interfaceElement = parent.declaredFragment?.element;
     } else if (parent is ExtensionTypeDeclaration) {
-      interfaceElement = parent.declaredElement!;
+      interfaceElement = parent.declaredFragment?.element;
     } else if (parent is MixinDeclaration) {
-      interfaceElement = parent.declaredElement!;
+      interfaceElement = parent.declaredFragment?.element;
     }
     if (interfaceElement != null) {
       return validateCreateMethod(

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

@@ -504,8 +504,8 @@ abstract class RenameRefactoring implements Refactoring {
       return RenameClassMemberRefactoringImpl(
         workspace,
         sessionHelper,
-        enclosingElement,
-        element,
+        enclosingElement.asElement2,
+        element.asElement2!,
       );
     }
     if (enclosingElement is ExtensionElement) {

pkg/analysis_server/lib/src/services/refactoring/legacy/refactoring_internal.dart

@@ -93,6 +93,9 @@ class SourceReference {
   /// Return the [LibraryElement] for the [file].
   LibraryElement get libraryElement => _match.libraryElement;
 
+  /// Return the [LibraryElement2] for the [file].
+  LibraryElement2 get libraryElement2 => _match.libraryElement2;
+
   SourceRange get range => _match.sourceRange;
 
   Source get unitSource => _match.unitSource;

pkg/analysis_server/lib/src/services/refactoring/legacy/rename.dart

@@ -9,8 +9,10 @@ import 'package:analysis_server/src/services/refactoring/legacy/refactoring.dart
 import 'package:analysis_server/src/services/refactoring/legacy/refactoring_internal.dart';
 import 'package:analysis_server/src/services/search/search_engine.dart';
 import 'package:analyzer/dart/element/element.dart';
+import 'package:analyzer/dart/element/element2.dart';
 import 'package:analyzer/src/dart/analysis/session_helper.dart';
 import 'package:analyzer/src/generated/java_core.dart';
+import 'package:analyzer/src/utilities/extensions/element.dart';
 import 'package:analyzer_plugin/utilities/range_factory.dart';
 
 /// Helper for renaming one or more [Element]s.
@@ -35,6 +37,18 @@ class RenameProcessor {
     }
   }
 
+  /// Add the edit that updates the [element] declaration.
+  void addDeclarationEdit2(Element2? element) {
+    if (element != null && workspace.containsElement2(element)) {
+      Fragment? fragment = element.firstFragment;
+      while (fragment != null) {
+        var edit = newSourceEdit_range(range.fragmentName(fragment)!, newName);
+        doSourceChange_addFragmentEdit(change, fragment, edit);
+        fragment = fragment.nextFragment;
+      }
+    }
+  }
+
   /// Add edits that update [matches].
   void addReferenceEdits(List<SearchMatch> matches) {
     var references = getSourceReferences(matches);
@@ -88,8 +102,16 @@ abstract class RenameRefactoringImpl extends RefactoringImpl
       elementKindName = element.kind.displayName,
       oldName = _getOldName(element);
 
+  RenameRefactoringImpl.c2(this.workspace, this.sessionHelper, Element2 element)
+    : searchEngine = workspace.searchEngine,
+      _element = element.asElement!,
+      elementKindName = element.kind.displayName,
+      oldName = _getOldName(element.asElement!);
+
   Element get element => _element;
 
+  Element2 get element2 => _element.asElement2!;
+
   @override
   Future<RefactoringStatus> checkInitialConditions() {
     var result = RefactoringStatus();