[element model] migrate use_string_buffers

Bug: https://github.com/dart-lang/linter/issues/5099
Change-Id: If08bf48b2d7caa9e54868f383b94cdccd5766197
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/393060
Commit-Queue: Brian Wilkerson <[email protected]>
Auto-Submit: Phil Quitslund <[email protected]>
Reviewed-by: Brian Wilkerson <[email protected]>

Author: pq

pkg/linter/analyzer_use_new_elements.txt

@@ -23,6 +23,7 @@ lib/src/rules/avoid_dynamic_calls.dart
 lib/src/rules/avoid_empty_else.dart
 lib/src/rules/avoid_equals_and_hash_code_on_mutable_classes.dart
 lib/src/rules/avoid_escaping_inner_quotes.dart
+lib/src/rules/avoid_field_initializers_in_const_classes.dart
 lib/src/rules/avoid_final_parameters.dart
 lib/src/rules/avoid_function_literals_in_foreach_calls.dart
 lib/src/rules/avoid_futureor_void.dart
@@ -56,6 +57,7 @@ lib/src/rules/camel_case_types.dart
 lib/src/rules/cancel_subscriptions.dart
 lib/src/rules/cast_nullable_to_non_nullable.dart
 lib/src/rules/close_sinks.dart
+lib/src/rules/collection_methods_unrelated_type.dart
 lib/src/rules/combinators_ordering.dart
 lib/src/rules/comment_references.dart
 lib/src/rules/conditional_uri_does_not_exist.dart
@@ -212,6 +214,7 @@ lib/src/rules/use_is_even_rather_than_modulo.dart
 lib/src/rules/use_named_constants.dart
 lib/src/rules/use_raw_strings.dart
 lib/src/rules/use_rethrow_when_possible.dart
+lib/src/rules/use_string_buffers.dart
 lib/src/rules/use_string_in_part_of_directives.dart
 lib/src/rules/use_super_parameters.dart
 lib/src/rules/use_test_throws_matchers.dart

pkg/linter/lib/src/rules/use_string_buffers.dart

@@ -5,11 +5,10 @@
 import 'package:analyzer/dart/ast/ast.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 '../analyzer.dart';
-import '../extensions.dart';
 
 const _desc = r'Use string buffers to compose strings.';
 
@@ -67,7 +66,7 @@ class _IdentifierIsPrefixVisitor extends SimpleAstVisitor<void> {
 
   @override
   void visitSimpleIdentifier(SimpleIdentifier node) {
-    if (node.staticElement == identifier.staticElement) {
+    if (node.element == identifier.element) {
       rule.reportLint(identifier);
     }
   }
@@ -83,7 +82,7 @@ class _IdentifierIsPrefixVisitor extends SimpleAstVisitor<void> {
 
 class _UseStringBufferVisitor extends SimpleAstVisitor<void> {
   final LintRule rule;
-  final localElements = <Element?>{};
+  final localElements = <Element2?>{};
 
   _UseStringBufferVisitor(this.rule);
 
@@ -100,7 +99,7 @@ class _UseStringBufferVisitor extends SimpleAstVisitor<void> {
         writeType is InterfaceType &&
         writeType.isDartCoreString) {
       if (node.operator.type == TokenType.PLUS_EQ &&
-          !localElements.contains(node.writeElement?.canonicalElement)) {
+          !localElements.contains(node.writeElement2)) {
         rule.reportLint(node);
       }
       if (node.operator.type == TokenType.EQ) {
@@ -128,7 +127,7 @@ class _UseStringBufferVisitor extends SimpleAstVisitor<void> {
   @override
   void visitVariableDeclarationStatement(VariableDeclarationStatement node) {
     for (var variable in node.variables.variables) {
-      localElements.add(variable.declaredElement);
+      localElements.add(variable.declaredElement2);
     }
   }
 }