Elements. Migrate DartCompletionRequestExtensions.

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

Author: scheglov

pkg/analysis_server/analyzer_use_new_elements.txt

@@ -65,7 +65,6 @@ lib/src/services/search/hierarchy.dart
 lib/src/services/search/search_engine.dart
 lib/src/services/search/search_engine_internal.dart
 lib/src/utilities/extensions/ast.dart
-lib/src/utilities/extensions/completion_request.dart
 lib/src/utilities/extensions/element.dart
 test/abstract_single_unit.dart
 test/plugin/protocol_dart_test.dart

pkg/analysis_server/lib/src/services/completion/dart/in_scope_completion_pass.dart

@@ -23,6 +23,7 @@ import 'package:analyzer/dart/ast/syntactic_entity.dart';
 import 'package:analyzer/dart/ast/token.dart';
 import 'package:analyzer/dart/ast/visitor.dart';
 import 'package:analyzer/dart/element/element.dart';
+import 'package:analyzer/dart/element/element2.dart';
 import 'package:analyzer/dart/element/type.dart';
 import 'package:analyzer/source/source_range.dart';
 import 'package:analyzer/src/dart/ast/ast.dart';
@@ -2248,22 +2249,22 @@ class InScopeCompletionPass extends SimpleAstVisitor<void> {
       if (type != null) {
         _forMemberAccess(node, type, onlySuper: target is SuperExpression);
       } else {
-        var element = target.staticElement;
+        var element = target.element;
         if (element != null) {
           var parent = node.parent;
           var mustBeAssignable =
               parent is AssignmentExpression && node == parent.leftHandSide;
-          if (element is PrefixElement) {
+          if (element is PrefixElement2) {
             declarationHelper(
               mustBeAssignable: mustBeAssignable,
-            ).addDeclarationsThroughImportPrefix(element);
+            ).addDeclarationsThroughImportPrefix(element.asElement);
           } else {
             declarationHelper(
               mustBeAssignable: mustBeAssignable,
               preferNonInvocation:
-                  element is InterfaceElement &&
+                  element is InterfaceElement2 &&
                   state.request.shouldSuggestTearOff(element),
-            ).addStaticMembersOfElement(element);
+            ).addStaticMembersOfElement(element.asElement!);
           }
         }
       }

pkg/analysis_server/lib/src/utilities/extensions/completion_request.dart

@@ -4,7 +4,7 @@
 
 import 'package:analysis_server/src/services/completion/dart/completion_manager.dart';
 import 'package:analyzer/dart/analysis/features.dart';
-import 'package:analyzer/dart/element/element.dart';
+import 'package:analyzer/dart/element/element2.dart';
 import 'package:analyzer/dart/element/nullability_suffix.dart';
 import 'package:analyzer/dart/element/type.dart';
 
@@ -15,8 +15,8 @@ extension DartCompletionRequestExtensions on DartCompletionRequest {
   ///
   // TODO(scheglov): Validate that suggesting a tear-off instead of invocation
   // is statistically a good choice.
-  bool shouldSuggestTearOff(InterfaceElement element) {
-    if (!libraryElement.featureSet.isEnabled(Feature.constructor_tearoffs)) {
+  bool shouldSuggestTearOff(InterfaceElement2 element) {
+    if (!libraryElement2.featureSet.isEnabled(Feature.constructor_tearoffs)) {
       return false;
     }
 
@@ -27,13 +27,13 @@ extension DartCompletionRequestExtensions on DartCompletionRequest {
 
     var bottomInstance = element.instantiate(
       typeArguments: List.filled(
-        element.typeParameters.length,
-        libraryElement.typeProvider.neverType,
+        element.typeParameters2.length,
+        libraryElement2.typeProvider.neverType,
       ),
       nullabilitySuffix: NullabilitySuffix.none,
     );
 
-    return libraryElement.typeSystem.isSubtypeOf(
+    return libraryElement2.typeSystem.isSubtypeOf(
       bottomInstance,
       contextType.returnType,
     );