DAS: Correct several broken comment references

Change-Id: Id0e80d5566a5c05296a5941746888a48284c4ae0
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/407861
Reviewed-by: Phil Quitslund <[email protected]>
Commit-Queue: Samuel Rawlins <[email protected]>
Reviewed-by: Brian Wilkerson <[email protected]>

Author: srawlins

pkg/analysis_server/lib/src/analysis_server.dart

@@ -107,7 +107,7 @@ typedef UserPromptSender =
     );
 
 /// Implementations of [AnalysisServer] implement a server that listens
-/// on a [CommunicationChannel] for analysis messages and process them.
+/// on an [AbstractNotificationManager] for analysis messages and process them.
 abstract class AnalysisServer {
   /// A flag indicating whether plugins are supported in this build.
   static final bool supportsPlugins = true;
@@ -418,7 +418,7 @@ abstract class AnalysisServer {
   ///
   /// Request handlers should be careful to use the correct clients capabilities
   /// if they are available to non-editor clients (such as over DTD). The
-  /// capabilities of the caller are available in [MessageInfo] and should
+  /// capabilities of the caller are available in [lsp.MessageInfo] and should
   /// usually be used when computing the results for requests, but if those
   /// requests additionally trigger requests to the editor, those requests to
   /// the editor should consider these capabilities.
@@ -437,7 +437,7 @@ abstract class AnalysisServer {
   /// A [Future] that completes when the LSP server moves into the initialized
   /// state and can handle normal LSP requests.
   ///
-  /// Completes with the [InitializedStateMessageHandler] that is active.
+  /// Completes with the [lsp.InitializedStateMessageHandler] that is active.
   ///
   /// When the server leaves the initialized state, [lspUninitialized] will
   /// complete.
@@ -493,8 +493,8 @@ abstract class AnalysisServer {
 
   void afterContextsDestroyed() {}
 
-  /// Broadcast a request built from the given [params] to all of the plugins
-  /// that are currently associated with the context root from the given
+  /// Broadcast a request built from the given [requestParams] to all of the
+  /// plugins that are currently associated with the context root from the given
   /// [driver]. Return a list containing futures that will complete when each of
   /// the plugins have sent a response, or an empty list if no [driver] is
   /// provided.

pkg/analysis_server/lib/src/services/search/element_visitors.dart

@@ -78,7 +78,7 @@ class _ElementByNameOffsetVisitor extends GeneralizingElementVisitor<void> {
   }
 }
 
-/// A [GeneralizingElementVisitor] adapter for [ElementProcessor].
+/// A [GeneralizingElementVisitor] adapter for [BoolElementProcessor].
 class _ElementVisitorAdapter extends GeneralizingElementVisitor<void> {
   final BoolElementProcessor processor;
 

pkg/analysis_server/lib/src/services/search/hierarchy.dart

@@ -87,12 +87,8 @@ List<Element2> getExtensionMembers(
   return members;
 }
 
-/// Return all implementations of the given [member], including in its
+/// Returns all implementations of the given [member], including in its
 /// superclasses and their subclasses.
-///
-/// If [includeParametersForFields] is true and [member] is a [FieldElement2],
-/// any [FieldFormalParameterElement2]s for the member will also be provided
-/// (otherwise, the parameter set will be empty in the result).
 Future<Set<Element2>> getHierarchyMembers(
   SearchEngine searchEngine,
   Element2 member, {
@@ -106,10 +102,10 @@ Future<Set<Element2>> getHierarchyMembers(
   return members;
 }
 
-/// Return all implementations of the given [member], including in its
+/// Returns all implementations of the given [member2], including in its
 /// superclasses and their subclasses.
 ///
-/// If [includeParametersForFields] is true and [member] is a [FieldElement2],
+/// If [includeParametersForFields] is true and [member2] is a [FieldElement2],
 /// any [FieldFormalParameterElement2]s for the member will also be provided
 /// (otherwise, the parameter set will be empty in the result).
 Future<(Set<Element2>, Set<FormalParameterElement>)>

pkg/analysis_server/lib/src/status/element_writer.dart

@@ -31,7 +31,7 @@ class ElementWriter with TreeWriter {
     }
   }
 
-  /// Write a representation of the properties of the given [node] to the
+  /// Writes a representation of the properties of the given [element] to the
   /// buffer.
   Map<String, Object?> _computeProperties(Element2 element) {
     var properties = <String, Object?>{};

pkg/analysis_server/lib/src/status/tree_writer.dart

@@ -88,7 +88,7 @@ mixin TreeWriter {
     return null;
   }
 
-  /// Write the [value] of the property with the given [name].
+  /// Writes the [value] of the property.
   void _writePropertyValue(Object value, int baseIndent) {
     if (value is List<Object>) {
       if (value.isEmpty) {

pkg/analysis_server/lib/src/utilities/import_analyzer.dart

@@ -34,9 +34,11 @@ class ImportAnalyzer {
   /// the set of imports used to reference those declarations.
   final Map<Element2, Set<LibraryImport>> stayingReferences = {};
 
-  /// Analyze the given library [result] to find the declarations and references
-  /// being moved and that are staying. The declarations being moved are in the
-  /// file at the given [path] in the given [range].
+  /// Analyzes the given library [result] to find the declarations and
+  /// references being moved and that are staying.
+  ///
+  /// The declarations being moved are in the file at the given [path] in the
+  /// given [ranges].
   ImportAnalyzer(this.result, String path, List<SourceRange> ranges) {
     for (var unit in result.units) {
       var finder = _ReferenceFinder(
@@ -92,7 +94,7 @@ class _ElementRecorder {
 
   /// Initialize a newly created recorder to use the [analyzer] to record
   /// declarations of and references to elements, based on whether the reference
-  /// is within the [range].
+  /// is within the [ranges].
   _ElementRecorder(this.analyzer, this.ranges);
 
   /// Record that the [declaredElement] is declared in the library.
@@ -174,7 +176,7 @@ class _ReferenceFinder extends RecursiveAstVisitor<void> {
   /// empty string is used for unprefixed imports.
   ///
   /// Library imports are ordered the same as they appear in the source file
-  /// (since this is a [LinkedHashSet]).
+  /// (since this is a `LinkedHashSet`).
   final _importsByPrefix = <String, Set<LibraryImport>>{};
 
   /// Initialize a newly created finder to send information to the [recorder].

pkg/analysis_server/lib/src/utilities/mocks.dart

@@ -2,6 +2,9 @@
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
+/// @docImport 'package:analysis_server/src/analysis_server.dart';
+library;
+
 import 'dart:async';
 import 'dart:convert';