Elements. Remove V1 PatternVariableElement.

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

Author: scheglov

pkg/analyzer/api.txt

@@ -2967,8 +2967,6 @@ package:analyzer/dart/constant/value.dart:
     toSymbolValue (method: String? Function())
     toTypeValue (method: DartType? Function())
 package:analyzer/dart/element/element.dart:
-  BindPatternVariableElement (class extends Object implements PatternVariableElement, deprecated):
-    new (constructor: BindPatternVariableElement Function())
   ClassMemberElement (class extends Object implements Element, deprecated):
     new (constructor: ClassMemberElement Function())
     enclosingElement3 (getter: Element)
@@ -3149,10 +3147,6 @@ package:analyzer/dart/element/element.dart:
   ImportElementPrefix (class extends Object, deprecated):
     new (constructor: ImportElementPrefix Function())
     element (getter: PrefixElement)
-  JoinPatternVariableElement (class extends Object implements PatternVariableElement, deprecated):
-    new (constructor: JoinPatternVariableElement Function())
-    isConsistent (getter: bool)
-    variables (getter: List<PatternVariableElement>)
   LibraryElement (class extends Object implements _ExistingElement, deprecated):
     new (constructor: LibraryElement Function())
     definingCompilationUnit (getter: CompilationUnitElement, deprecated)
@@ -3228,9 +3222,6 @@ package:analyzer/dart/element/element.dart:
   PartElement (class extends Object implements _ExistingElement, deprecated):
     new (constructor: PartElement Function())
     uri (getter: DirectiveUri)
-  PatternVariableElement (class extends Object implements LocalVariableElement, deprecated):
-    new (constructor: PatternVariableElement Function())
-    join (getter: JoinPatternVariableElement?)
   PrefixElement (class extends Object implements _ExistingElement, deprecated):
     new (constructor: PrefixElement Function())
     enclosingElement3 (getter: CompilationUnitElement, deprecated)

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

@@ -60,13 +60,6 @@ import 'package:analyzer/src/task/api/model.dart' show AnalysisTarget;
 import 'package:meta/meta.dart';
 import 'package:pub_semver/pub_semver.dart';
 
-/// A pattern variable that is explicitly declared.
-///
-/// Clients may not extend, implement or mix-in this class.
-@Deprecated(
-    'Use BindPatternVariableFragment and BindPatternVariableElement2 instead')
-abstract class BindPatternVariableElement implements PatternVariableElement {}
-
 /// An element that is contained within a [ClassElement].
 ///
 /// Clients may not extend, implement or mix-in this class.
@@ -818,21 +811,6 @@ abstract class ImportElementPrefix {
   PrefixElement get element;
 }
 
-/// A pattern variable that is a join of other pattern variables, created
-/// for a logical-or patterns, or shared `case` bodies in `switch` statements.
-///
-/// Clients may not extend, implement or mix-in this class.
-@Deprecated(
-    'Use JoinPatternVariableFragment and JoinPatternVariableElement2 instead')
-abstract class JoinPatternVariableElement implements PatternVariableElement {
-  /// Whether the [variables] are consistent, present in all branches,
-  /// and have the same type and finality.
-  bool get isConsistent;
-
-  /// The variables that join into this variable.
-  List<PatternVariableElement> get variables;
-}
-
 /// A library.
 ///
 /// Clients may not extend, implement or mix-in this class.
@@ -1126,16 +1104,6 @@ abstract class PartElement implements _ExistingElement {
   DirectiveUri get uri;
 }
 
-/// A pattern variable.
-///
-/// Clients may not extend, implement or mix-in this class.
-@Deprecated('Use PatternVariableFragment and PatternVariableElement2 instead')
-abstract class PatternVariableElement implements LocalVariableElement {
-  /// The variable in which this variable joins with other pattern variables
-  /// with the same name, in a logical-or pattern, or shared case scope.
-  JoinPatternVariableElement? get join;
-}
-
 /// A prefix used to import one or more libraries into another library.
 ///
 /// Clients may not extend, implement or mix-in this class.

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

@@ -90,10 +90,7 @@ mixin AugmentableFragment on ElementImpl {
 }
 
 class BindPatternVariableElementImpl extends PatternVariableElementImpl
-    implements
-        // ignore: deprecated_member_use_from_same_package,analyzer_use_new_elements
-        BindPatternVariableElement,
-        BindPatternVariableFragment {
+    implements BindPatternVariableFragment {
   final DeclaredVariablePatternImpl node;
 
   /// This flag is set to `true` if this variable clashes with another
@@ -6326,11 +6323,8 @@ abstract class InterfaceElementImpl2 extends InstanceElementImpl2
 }
 
 class JoinPatternVariableElementImpl extends PatternVariableElementImpl
-    implements
-        // ignore: deprecated_member_use_from_same_package,analyzer_use_new_elements
-        JoinPatternVariableElement,
-        JoinPatternVariableFragment {
-  @override
+    implements JoinPatternVariableFragment {
+  /// The variables that join into this variable.
   final List<PatternVariableElementImpl> variables;
 
   shared.JoinedPatternVariableInconsistency inconsistency;
@@ -8942,11 +8936,9 @@ class PartElementImpl extends _ExistingElementImpl
 }
 
 class PatternVariableElementImpl extends LocalVariableElementImpl
-    implements
-        // ignore: deprecated_member_use_from_same_package,analyzer_use_new_elements
-        PatternVariableElement,
-        PatternVariableFragment {
-  @override
+    implements PatternVariableFragment {
+  /// The variable in which this variable joins with other pattern variables
+  /// with the same name, in a logical-or pattern, or shared case scope.
   JoinPatternVariableElementImpl? join;
 
   /// This flag is set to `true` while we are visiting the [WhenClause] of