Macro. Remove more from analyzer/ and analysis_seever/.

Change-Id: Id5426cce73395d3d95657cf1e0b5f9d85fe3f574
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/406921
Reviewed-by: Brian Wilkerson <[email protected]>

Author: scheglov

pkg/analysis_server/lib/src/computer/computer_highlights.dart

@@ -806,7 +806,6 @@ class _DartUnitHighlightsComputerVisitor extends RecursiveAstVisitor<void> {
       node.augmentKeyword,
       HighlightRegionType.BUILT_IN,
     );
-    computer._addRegion_token(node.macroKeyword, HighlightRegionType.BUILT_IN);
     computer._addRegion_token(
       node.abstractKeyword,
       HighlightRegionType.BUILT_IN,

pkg/analysis_server/lib/src/handler/legacy/edit_get_assists.dart

@@ -16,7 +16,6 @@ import 'package:analysis_server/src/services/correction/assist_internal.dart';
 import 'package:analysis_server_plugin/src/correction/dart_change_workspace.dart';
 import 'package:analyzer/dart/analysis/session.dart';
 import 'package:analyzer/src/exception/exception.dart';
-import 'package:analyzer/src/util/file_paths.dart';
 import 'package:analyzer_plugin/protocol/protocol_common.dart';
 import 'package:analyzer_plugin/protocol/protocol_generated.dart' as plugin;
 
@@ -45,10 +44,6 @@ class EditGetAssistsHandler extends LegacyHandler
     if (server.sendResponseErrorIfInvalidFilePath(request, file)) {
       return;
     }
-    if (isMacroGenerated(file)) {
-      sendResult(EditGetAssistsResult([]));
-      return;
-    }
     //
     // Allow plugins to start computing assists.
     //

pkg/analysis_server/lib/src/handler/legacy/edit_get_available_refactorings.dart

@@ -9,7 +9,6 @@ import 'package:analysis_server/src/handler/legacy/legacy_handler.dart';
 import 'package:analysis_server/src/services/refactoring/legacy/refactoring.dart';
 import 'package:analyzer/dart/element/element2.dart';
 import 'package:analyzer/src/dart/ast/utilities.dart';
-import 'package:analyzer/src/util/file_paths.dart';
 import 'package:analyzer_plugin/protocol/protocol_common.dart';
 
 /// The handler for the `edit.getAvailableRefactorings` request.
@@ -36,10 +35,6 @@ class EditGetAvailableRefactoringsHandler extends LegacyHandler {
     if (server.sendResponseErrorIfInvalidFilePath(request, file)) {
       return;
     }
-    if (isMacroGenerated(file)) {
-      sendResult(EditGetAvailableRefactoringsResult([]));
-      return;
-    }
 
     // add refactoring kinds
     var kinds = <RefactoringKind>[];

pkg/analysis_server/lib/src/handler/legacy/edit_get_fixes.dart

@@ -28,7 +28,6 @@ import 'package:analyzer/src/generated/source.dart' show SourceFactory;
 import 'package:analyzer/src/pubspec/pubspec_validator.dart';
 import 'package:analyzer/src/task/options.dart';
 import 'package:analyzer/src/util/file_paths.dart' as file_paths;
-import 'package:analyzer/src/util/file_paths.dart';
 import 'package:analyzer/src/workspace/pub.dart';
 import 'package:analyzer_plugin/protocol/protocol_generated.dart' as plugin;
 import 'package:yaml/yaml.dart';
@@ -62,10 +61,6 @@ class EditGetFixesHandler extends LegacyHandler
       server.sendResponse(Response.getFixesInvalidFile(request));
       return;
     }
-    if (isMacroGenerated(file)) {
-      sendResult(EditGetFixesResult([]));
-      return;
-    }
 
     //
     // Allow plugins to start computing fixes.

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

@@ -80,7 +80,6 @@ import 'package:analysis_server_plugin/src/correction/fix_generators.dart';
 import 'package:analyzer/error/error.dart';
 import 'package:analyzer/src/dart/ast/utilities.dart';
 import 'package:analyzer/src/generated/java_core.dart';
-import 'package:analyzer/src/util/file_paths.dart';
 import 'package:analyzer_plugin/utilities/assist/assist.dart'
     hide AssistContributor;
 import 'package:analyzer_plugin/utilities/change_builder/change_builder_core.dart';
@@ -178,9 +177,6 @@ class AssistProcessor {
   AssistProcessor(this._assistContext);
 
   Future<List<Assist>> compute() async {
-    if (isMacroGenerated(_assistContext.unitResult.file.path)) {
-      return _assists;
-    }
     await _addFromProducers();
     return _assists;
   }

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

@@ -227,9 +227,7 @@ class BulkFixProcessor {
   }) async {
     var pathContext = context.contextRoot.resourceProvider.pathContext;
 
-    if (file_paths.isDart(pathContext, path) &&
-        !file_paths.isGenerated(path) &&
-        !file_paths.isMacroGenerated(path)) {
+    if (file_paths.isDart(pathContext, path) && !file_paths.isGenerated(path)) {
       var libraryResult = await performance.runAsync(
         'getResolvedLibrary',
         (_) => context.currentSession.getResolvedContainingLibrary(path),
@@ -257,8 +255,7 @@ class BulkFixProcessor {
       var pathContext = context.contextRoot.resourceProvider.pathContext;
       for (var path in context.contextRoot.analyzedFiles()) {
         if (!file_paths.isDart(pathContext, path) ||
-            file_paths.isGenerated(path) ||
-            file_paths.isMacroGenerated(path)) {
+            file_paths.isGenerated(path)) {
           continue;
         }
 
@@ -397,8 +394,7 @@ class BulkFixProcessor {
 
       for (var path in context.contextRoot.analyzedFiles()) {
         if (!file_paths.isDart(pathContext, path) ||
-            file_paths.isGenerated(path) ||
-            file_paths.isMacroGenerated(path)) {
+            file_paths.isGenerated(path)) {
           continue;
         }
         var package = workspace.findPackageFor(path);
@@ -510,8 +506,7 @@ class BulkFixProcessor {
       var pathContext = context.contextRoot.resourceProvider.pathContext;
       for (var path in context.contextRoot.analyzedFiles()) {
         if (!file_paths.isDart(pathContext, path) ||
-            file_paths.isGenerated(path) ||
-            file_paths.isMacroGenerated(path)) {
+            file_paths.isGenerated(path)) {
           continue;
         }
 
@@ -831,8 +826,7 @@ class BulkFixProcessor {
       for (var path in context.contextRoot.analyzedFiles()) {
         var pathContext = context.contextRoot.resourceProvider.pathContext;
         if (!file_paths.isDart(pathContext, path) ||
-            file_paths.isGenerated(path) ||
-            file_paths.isMacroGenerated(path)) {
+            file_paths.isGenerated(path)) {
           continue;
         }
         var unitResult =
@@ -905,8 +899,7 @@ class BulkFixProcessor {
       for (var path in context.contextRoot.analyzedFiles()) {
         var pathContext = context.contextRoot.resourceProvider.pathContext;
         if (!file_paths.isDart(pathContext, path) ||
-            file_paths.isGenerated(path) ||
-            file_paths.isMacroGenerated(path)) {
+            file_paths.isGenerated(path)) {
           continue;
         }
         var unitResult =

pkg/analysis_server/lib/src/services/correction/error_fix_status.yaml

@@ -966,8 +966,6 @@ CompileTimeErrorCode.INVALID_INLINE_FUNCTION_TYPE:
   status: needsFix
   notes: |-
     Replace with the equivalent generic function type.
-CompileTimeErrorCode.INVALID_MACRO_APPLICATION_TARGET:
-  status: noFix
 CompileTimeErrorCode.INVALID_MODIFIER_ON_CONSTRUCTOR:
   status: hasFix
 CompileTimeErrorCode.INVALID_MODIFIER_ON_SETTER:
@@ -1023,18 +1021,6 @@ CompileTimeErrorCode.LIST_ELEMENT_TYPE_NOT_ASSIGNABLE_NULLABILITY:
   status: hasFix
 CompileTimeErrorCode.LIST_ELEMENT_TYPE_NOT_ASSIGNABLE:
   status: needsEvaluation
-CompileTimeErrorCode.MACRO_DECLARATIONS_PHASE_INTROSPECTION_CYCLE:
-  status: needsEvaluation
-CompileTimeErrorCode.MACRO_DEFINITION_APPLICATION_SAME_LIBRARY_CYCLE:
-  status: noFix
-CompileTimeErrorCode.MACRO_APPLICATION_ARGUMENT_ERROR:
-  status: noFix
-CompileTimeErrorCode.MACRO_ERROR:
-  status: noFix
-CompileTimeErrorCode.MACRO_INTERNAL_EXCEPTION:
-  status: noFix
-CompileTimeErrorCode.MACRO_NOT_ALLOWED_DECLARATION:
-  status: noFix
 CompileTimeErrorCode.MAIN_FIRST_POSITIONAL_PARAMETER_TYPE:
   status: needsFix
   notes: |-
@@ -1844,8 +1830,6 @@ HintCode.DEPRECATED_MEMBER_USE_WITH_MESSAGE:
   status: hasFix
 HintCode.IMPORT_DEFERRED_LIBRARY_WITH_LOAD_FUNCTION:
   status: noFix
-HintCode.MACRO_INFO:
-  status: noFix
 HintCode.UNNECESSARY_IMPORT:
   status: hasFix
 LintCode.always_declare_return_types_of_functions:
@@ -3644,8 +3628,6 @@ WarningCode.INVALID_VISIBLE_FOR_OVERRIDING_ANNOTATION:
   status: hasFix
 WarningCode.INVALID_VISIBLE_OUTSIDE_TEMPLATE_ANNOTATION:
   status: needsEvaluation
-WarningCode.MACRO_WARNING:
-  status: noFix
 WarningCode.MISSING_OVERRIDE_OF_MUST_BE_OVERRIDDEN_ONE:
   status: hasFix
 WarningCode.MISSING_OVERRIDE_OF_MUST_BE_OVERRIDDEN_TWO:

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

@@ -13,7 +13,6 @@ import 'package:analyzer/dart/ast/ast.dart';
 import 'package:analyzer/dart/ast/token.dart';
 import 'package:analyzer/dart/element/element2.dart';
 import 'package:analyzer/src/dart/analysis/session_helper.dart';
-import 'package:analyzer/src/util/file_paths.dart';
 import 'package:analyzer_plugin/utilities/range_factory.dart';
 
 /// [ConvertMethodToGetterRefactoring] implementation.
@@ -141,8 +140,6 @@ class ConvertGetterToMethodRefactoringImpl extends RefactoringImpl
     var matches = await searchEngine.searchReferences(element);
     var references = getSourceReferences(matches);
     for (var reference in references) {
-      // Don't update references in macro-generated files.
-      if (isMacroGenerated(reference.file)) continue;
       var refRange = reference.range;
       // insert "()"
       var edit = SourceEdit(refRange.end, 0, '()');

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

@@ -14,7 +14,6 @@ import 'package:analyzer/dart/element/element2.dart';
 import 'package:analyzer/dart/element/type.dart';
 import 'package:analyzer/src/dart/analysis/session_helper.dart';
 import 'package:analyzer/src/dart/ast/utilities.dart';
-import 'package:analyzer/src/util/file_paths.dart';
 import 'package:analyzer_plugin/utilities/range_factory.dart';
 
 /// [ConvertMethodToGetterRefactoring] implementation.
@@ -141,9 +140,6 @@ class ConvertMethodToGetterRefactoringImpl extends RefactoringImpl
     var matches = await searchEngine.searchReferences(element);
     var references = getSourceReferences(matches);
     for (var reference in references) {
-      // Don't update references in macro-generated files.
-      if (isMacroGenerated(reference.file)) continue;
-
       var refElement = reference.element2;
       var refRange = reference.range;
       // prepare invocation

pkg/analysis_server/test/edit/bulk_fixes_test.dart

@@ -377,20 +377,6 @@ linter:
 class A {}
 //ignore: unnecessary_new
 A f() => new A();
-''');
-    await assertNoEdits();
-  }
-
-  Future<void> test_unnecessaryNew_macroGenerated() async {
-    newAnalysisOptionsYamlFile(testPackageRootPath, '''
-linter:
-  rules:
-    - unnecessary_new
-''');
-    var macroFilePath = join(testPackageLibPath, 'test.macro.dart');
-    newFile(macroFilePath, '''
-class A {}
-A f() => new A();
 ''');
     await assertNoEdits();
   }