[analyzer] Remove @experimental annotations from API.

Removes all the `@experimental` annotations from the analyzer
API. Since these annotations are not yet enforced through warnings, in
effect everything the analyzer public API has to be supported until
the next breaking change release, whether it's marked `@experimental`
or not. Removing the `@experimental` annotations reflects this
reality.

In follow-up CLs, I will introduce analyzer functionality that causes
warnings to be issued if an API method marked `@experimental` is used
(or overridden) from outside of its declaring package. Once that has
landed, we'll be able to safely start using `@experimental` to tag
parts of the analyzer public API for which we don't guarantee
long-term support.

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

Author: stereotype441

pkg/analyzer/api.txt

@@ -6,7 +6,6 @@ import 'package:analyzer/dart/analysis/context_root.dart';
 import 'package:analyzer/dart/analysis/session.dart';
 import 'package:analyzer/file_system/file_system.dart';
 import 'package:analyzer/src/generated/engine.dart';
-import 'package:meta/meta.dart';
 
 /// A representation of a body of code and the context in which the code is to
 /// be analyzed.
@@ -61,6 +60,5 @@ abstract class AnalysisContext {
   ///
   /// NOTE: this API is experimental and subject to change in a future
   /// release (see https://github.com/dart-lang/sdk/issues/53876 for context).
-  @experimental
   AnalysisOptions getAnalysisOptionsForFile(File file);
 }

pkg/analyzer/lib/dart/analysis/analysis_context.dart

@@ -11,7 +11,6 @@ import 'package:analyzer/dart/element/type_system.dart';
 import 'package:analyzer/diagnostic/diagnostic.dart';
 import 'package:analyzer/file_system/file_system.dart';
 import 'package:analyzer/source/line_info.dart';
-import 'package:meta/meta.dart';
 
 @Deprecated('Use AnalysisResultWithDiagnostics instead')
 typedef AnalysisResultWithErrors = AnalysisResultWithDiagnostics;
@@ -169,12 +168,10 @@ abstract class InvalidResult {}
 /// Clients may not extend, implement or mix-in this class.
 abstract class LibraryElementResult implements SomeLibraryElementResult {
   /// The element representing the library.
-  @experimental
   LibraryElement get element;
 
   /// The element representing the library.
   @Deprecated('Use element instead')
-  @experimental
   LibraryElement get element2;
 }
 
@@ -246,15 +243,13 @@ abstract class ParsedLibraryResult
   ///
   /// Throws [ArgumentError] if the [fragment] is not defined in this library.
   @Deprecated('Use getFragmentDeclaration() instead')
-  @experimental
   ElementDeclarationResult? getElementDeclaration2(Fragment fragment);
 
   /// Returns the declaration of the [fragment].
   ///
   /// Returns `null` if the [fragment] is synthetic.
   ///
   /// Throws [ArgumentError] if the [fragment] is not defined in this library.
-  @experimental
   FragmentDeclarationResult? getFragmentDeclaration(Fragment fragment);
 }
 
@@ -295,12 +290,10 @@ abstract class ParseStringResult {
 abstract class ResolvedLibraryResult
     implements ParsedLibraryResult, SomeResolvedLibraryResult {
   /// The element representing this library.
-  @experimental
   LibraryElement get element;
 
   /// The element representing this library.
   @Deprecated('Use element instead')
-  @experimental
   LibraryElement get element2;
 
   /// The type provider used when resolving the library.
@@ -325,16 +318,13 @@ abstract class ResolvedUnitResult
   bool get exists;
 
   /// The element representing the library containing the compilation [unit].
-  @experimental
   LibraryElement get libraryElement;
 
   /// The element representing the library containing the compilation [unit].
   @Deprecated('Use libraryElement instead')
-  @experimental
   LibraryElement get libraryElement2;
 
   /// The fragment corresponding to the [unit].
-  @experimental
   LibraryFragment get libraryFragment;
 
   /// The type provider used when resolving the compilation [unit].
@@ -419,7 +409,6 @@ abstract class SomeUnitElementResult {}
 // TODO(scheglov): Stop implementing [FileResult].
 abstract class UnitElementResult implements SomeUnitElementResult, FileResult {
   /// The fragment representing the content of the file.
-  @experimental
   LibraryFragment get fragment;
 }
 

pkg/analyzer/lib/dart/analysis/results.dart

@@ -9,7 +9,6 @@ import 'package:analyzer/dart/analysis/uri_converter.dart';
 import 'package:analyzer/dart/element/element.dart';
 import 'package:analyzer/exception/exception.dart';
 import 'package:analyzer/file_system/file_system.dart';
-import 'package:meta/meta.dart';
 
 /// A consistent view of the results of analyzing one or more files.
 ///
@@ -52,13 +51,11 @@ abstract class AnalysisSession {
 
   /// Return information about the results of parsing units of the library file
   /// with the given library [element].
-  @experimental
   SomeParsedLibraryResult getParsedLibraryByElement(LibraryElement element);
 
   /// Return information about the results of parsing units of the library file
   /// with the given library [element].
   @Deprecated('Use getParsedLibraryByElement instead')
-  @experimental
   SomeParsedLibraryResult getParsedLibraryByElement2(LibraryElement element);
 
   /// Return information about the results of parsing the file with the given
@@ -74,7 +71,6 @@ abstract class AnalysisSession {
   /// resolving all of the files in the library with the library [element].
   ///
   /// Throw [ArgumentError] if the [element] was not produced by this session.
-  @experimental
   Future<SomeResolvedLibraryResult> getResolvedLibraryByElement(
     LibraryElement element,
   );
@@ -84,7 +80,6 @@ abstract class AnalysisSession {
   ///
   /// Throw [ArgumentError] if the [element] was not produced by this session.
   @Deprecated('Use getResolvedLibraryByElement instead')
-  @experimental
   Future<SomeResolvedLibraryResult> getResolvedLibraryByElement2(
     LibraryElement element,
   );

pkg/analyzer/lib/dart/analysis/session.dart

@@ -6,14 +6,12 @@
 library;
 
 import 'package:analyzer/dart/ast/ast.dart';
-import 'package:meta/meta.dart';
 
 /// A block doc directive, denoted by an opening tag, and a closing tag.
 ///
 /// The text in between the two tags is not explicitly called out. It can be
 /// read from the original compilation unit, between the offsets of the opening
 /// and closing tags.
-@experimental
 final class BlockDocDirective implements DocDirective {
   final DocDirectiveTag openingTag;
   final DocDirectiveTag? closingTag;
@@ -25,7 +23,6 @@ final class BlockDocDirective implements DocDirective {
 }
 
 /// The type of Markdown code block in a documentation comment.
-@experimental
 enum CodeBlockType {
   /// Fenced code blocks begin with a code fence, preceded by up to three spaces
   /// of indentation.
@@ -43,13 +40,11 @@ enum CodeBlockType {
 /// as a [SimpleDocDirective], represented by a single [DocDirectiveTag], or a
 /// [BlockDocDirective], represented by an opening [DocDirectiveTag] and a
 /// closing one (in well-formed text).
-@experimental
 sealed class DocDirective {
   DocDirectiveType get type;
 }
 
 /// An argument in a doc directive. See [DocDirective] for their syntax.
-@experimental
 sealed class DocDirectiveArgument {
   /// The offset of the start of the argument, from the beginning of the
   /// compilation unit.
@@ -70,7 +65,6 @@ sealed class DocDirectiveArgument {
 }
 
 /// A named argument in a doc directive. See [DocDirective] for their syntax.
-@experimental
 final class DocDirectiveNamedArgument extends DocDirectiveArgument {
   /// The name of the argument.
   final String name;
@@ -84,7 +78,6 @@ final class DocDirectiveNamedArgument extends DocDirectiveArgument {
 }
 
 /// A parameter in a doc directive, with it's expected format, if it has one.
-@experimental
 final class DocDirectiveParameter {
   final String name;
   final DocDirectiveParameterFormat expectedFormat;
@@ -94,7 +87,6 @@ final class DocDirectiveParameter {
 
 /// The expected format of a doc directive parameter, which indicates some
 /// minimal validation that can produce diagnostics.
-@experimental
 enum DocDirectiveParameterFormat {
   /// A format indicating that arguments are not validated.
   any('any'),
@@ -118,7 +110,6 @@ enum DocDirectiveParameterFormat {
 
 /// A positional argument in a doc directive. See [DocDirective] for their
 /// syntax.
-@experimental
 final class DocDirectivePositionalArgument extends DocDirectiveArgument {
   DocDirectivePositionalArgument({
     required super.offset,
@@ -137,7 +128,6 @@ final class DocDirectivePositionalArgument extends DocDirectiveArgument {
 /// whitespace. There are two types of arguments: positional and named. Named
 /// arguments are written as `NAME=VALUE`, without any internal whitespace.
 /// Named arguments can be optional.
-@experimental
 final class DocDirectiveTag {
   /// The offset of the starting text; for example: '@animation'.
   final int offset;
@@ -161,7 +151,6 @@ final class DocDirectiveTag {
   });
 }
 
-@experimental
 enum DocDirectiveType {
   /// A [DocDirective] declaring an embedded video with HTML video controls.
   ///
@@ -391,7 +380,6 @@ enum DocDirectiveType {
 /// Documentation imports are declared with `@docImport` at the start of a line
 /// of a documentation comment, followed by regular import elements (URI,
 /// optional prefix, optional combinators), ending with a semicolon.
-@experimental
 final class DocImport {
   /// The offset of the starting text, '@docImport'.
   int offset;
@@ -402,7 +390,6 @@ final class DocImport {
 }
 
 /// A Markdown fenced code block found in a documentation comment.
-@experimental
 final class MdCodeBlock {
   /// The 'info string'.
   ///
@@ -434,7 +421,6 @@ final class MdCodeBlock {
 }
 
 /// A Markdown code block line found in a documentation comment.
-@experimental
 final class MdCodeBlockLine {
   /// The offset of the start of the code block, from the beginning of the
   /// compilation unit.
@@ -446,7 +432,6 @@ final class MdCodeBlockLine {
   MdCodeBlockLine({required this.offset, required this.length});
 }
 
-@experimental
 final class SimpleDocDirective implements DocDirective {
   final DocDirectiveTag tag;
 

pkg/analyzer/lib/dart/ast/doc_comment.dart

@@ -14,7 +14,6 @@ library;
 
 import 'package:analyzer/dart/element/element.dart';
 import 'package:analyzer/dart/element/type.dart';
-import 'package:meta/meta.dart';
 
 /// Information about a const constructor invocation.
 abstract class ConstructorInvocation {
@@ -125,7 +124,6 @@ abstract class DartObject {
   /// * this object is not of a function type,
   /// * the value of the object being represented is not known, or
   /// * the value of the object being represented is `null`.
-  @experimental
   ExecutableElement? toFunctionValue();
 
   /// Return an element corresponding to the value of the object being
@@ -135,7 +133,6 @@ abstract class DartObject {
   /// * the value of the object being represented is not known, or
   /// * the value of the object being represented is `null`.
   @Deprecated('Use toFunctionValue instead')
-  @experimental
   ExecutableElement? toFunctionValue2();
 
   /// Return an integer corresponding to the value of the object being

pkg/analyzer/lib/dart/constant/value.dart

@@ -59,7 +59,6 @@ import 'package:analyzer/source/line_info.dart';
 import 'package:analyzer/source/source.dart';
 import 'package:analyzer/src/dart/element/inheritance_manager3.dart' show Name;
 import 'package:analyzer/src/dart/resolver/scope.dart';
-import 'package:meta/meta.dart';
 import 'package:pub_semver/pub_semver.dart';
 
 export 'package:analyzer/src/dart/element/inheritance_manager3.dart' show Name;
@@ -552,7 +551,6 @@ abstract class DirectiveUriWithSource extends DirectiveUriWithRelativeUri {
 /// Clients may not extend, implement or mix-in this class.
 abstract class DirectiveUriWithUnit extends DirectiveUriWithSource {
   /// The library fragment referenced by the [source].
-  @experimental
   LibraryFragment get libraryFragment;
 }