Elements. Remove V1 'get children'.

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

Author: scheglov

pkg/analyzer/api.txt

@@ -3010,7 +3010,6 @@ package:analyzer/dart/element/element.dart:
     unit (getter: CompilationUnitElement, deprecated)
   Element (class extends Object implements AnalysisTarget, deprecated):
     new (constructor: Element Function())
-    children (getter: List<Element>, deprecated)
     context (getter: AnalysisContext@2)
     declaration (getter: Element?, deprecated)
     displayName (getter: String)

pkg/analyzer/lib/dart/element/element.dart

@@ -223,12 +223,6 @@ abstract class DirectiveUriWithUnit extends DirectiveUriWithSource {
 /// Clients may not extend, implement or mix-in this class.
 @Deprecated(elementModelDeprecationMsg)
 abstract class Element implements AnalysisTarget {
-  /// A list of this element's children.
-  ///
-  /// There is no guarantee of the order in which the children will be included.
-  @Deprecated(elementModelDeprecationMsg)
-  List<Element> get children;
-
   /// The analysis context in which this element is defined.
   AnalysisContext get context;
 

pkg/analyzer/lib/src/dart/element/element.dart

@@ -782,23 +782,6 @@ class CompilationUnitElementImpl extends UriReferencedElementImpl
     _accessors = accessors;
   }
 
-  @Deprecated('Use Element2 instead')
-  @override
-  List<Element> get children => [
-        ...super.children,
-        ...libraryImports,
-        ...libraryExports,
-        ...accessors,
-        ...classes,
-        ...enums,
-        ...extensions,
-        ...extensionTypes,
-        ...functions,
-        ...mixins,
-        ...typeAliases,
-        ...topLevelVariables,
-      ];
-
   @override
   List<Fragment> get children3 {
     return [
@@ -2504,10 +2487,6 @@ abstract class ElementImpl
     reference?.element = this;
   }
 
-  @Deprecated('Use Element2 instead')
-  @override
-  List<Element> get children => const [];
-
   /// The length of the element's code, or `null` if the element is synthetic.
   int? get codeLength => _codeLength;
 
@@ -3532,14 +3511,6 @@ abstract class ExecutableElementImpl extends _ExistingElementImpl
   /// [offset].
   ExecutableElementImpl(String super.name, super.offset, {super.reference});
 
-  @Deprecated('Use Element2 instead')
-  @override
-  List<Element> get children => [
-        ...super.children,
-        ...typeParameters,
-        ...parameters,
-      ];
-
   @override
   List<Fragment> get children3 => [
         ...typeParameters,
@@ -3794,16 +3765,6 @@ class ExtensionElementImpl extends InstanceElementImpl
   /// element.
   ExtensionElementImpl(super.name, super.nameOffset);
 
-  @Deprecated('Use Element2 instead')
-  @override
-  List<Element> get children => [
-        ...super.children,
-        ...accessors,
-        ...fields,
-        ...methods,
-        ...typeParameters,
-      ];
-
   @override
   List<Fragment> get children3 => [
         ...accessors,
@@ -5113,14 +5074,6 @@ class GenericFunctionTypeElementImpl extends _ExistingElementImpl
   GenericFunctionTypeElementImpl.forOffset(int nameOffset)
       : super("", nameOffset);
 
-  @Deprecated('Use Element2 instead')
-  @override
-  List<Element> get children => [
-        ...super.children,
-        ...typeParameters,
-        ...parameters,
-      ];
-
   @override
   List<Fragment> get children3 => [
         ...typeParameters,
@@ -5858,17 +5811,6 @@ abstract class InterfaceElementImpl extends InstanceElementImpl
   /// given [offset] in the file that contains the declaration of this element.
   InterfaceElementImpl(super.name, super.offset);
 
-  @Deprecated('Use Element2 instead')
-  @override
-  List<Element> get children => [
-        ...super.children,
-        ...accessors,
-        ...fields,
-        ...constructors,
-        ...methods,
-        ...typeParameters,
-      ];
-
   @override
   List<Fragment> get children3 => [
         ...accessors,
@@ -6652,13 +6594,6 @@ class LibraryElementImpl extends ElementImpl
   @override
   LibraryElementImpl get baseElement => this;
 
-  @Deprecated('Use Element2 instead')
-  @override
-  List<Element> get children => [
-        definingCompilationUnit,
-        ..._partUnits,
-      ];
-
   @override
   List<Element2> get children2 {
     return [
@@ -8658,10 +8593,6 @@ class ParameterElementImpl extends VariableElementImpl
     return element;
   }
 
-  @Deprecated('Use Element2 instead')
-  @override
-  List<Element> get children => parameters;
-
   @override
   List<Fragment> get children3 => const [];
 

pkg/analyzer/lib/src/dart/element/member.dart

@@ -209,10 +209,6 @@ abstract class ExecutableMember extends Member
     required this.typeParameters,
   });
 
-  @Deprecated('Use Element2 instead')
-  @override
-  List<Element> get children => parameters;
-
   @override
   List<Element2> get children2 {
     return [
@@ -789,10 +785,6 @@ abstract class Member
 
   Element2 get baseElement;
 
-  @Deprecated('Use Element2 instead')
-  @override
-  List<Element> get children => const [];
-
   @override
   AnalysisContext get context => _declaration.context;
 
@@ -1134,10 +1126,6 @@ class ParameterMember extends VariableMember
   @override
   FormalParameterElement get baseElement => _element2;
 
-  @Deprecated('Use Element2 instead')
-  @override
-  List<Element> get children => parameters;
-
   @override
   List<Element2> get children2 {
     return [