[das] legacy server support for analytics checking

Fixes: #60853

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

Author: pq

pkg/analysis_server/lib/src/analysis_server.dart

@@ -520,6 +520,20 @@ abstract class AnalysisServer {
     );
   }
 
+  /// Display a message that will allow us to enable analytics on the next run.
+  void checkAnalytics() {
+    var unifiedAnalytics = analyticsManager.analytics;
+    var prompt = userPromptSender;
+    if (!unifiedAnalytics.shouldShowMessage || prompt == null) {
+      return;
+    }
+
+    unawaited(
+      prompt(MessageType.info, unifiedAnalytics.getConsentMessage, ['Ok']),
+    );
+    unifiedAnalytics.clientShowedMessage();
+  }
+
   /// Checks that all [sessions] are still consistent, throwing
   /// [InconsistentAnalysisException] if not.
   void checkConsistency(List<AnalysisSessionImpl> sessions) {

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

@@ -26,6 +26,8 @@ class ServerSetClientCapabilitiesHandler extends LegacyHandler {
         request,
         clientUriConverter: server.uriConverter,
       );
+
+      server.checkAnalytics();
     } on RequestFailure catch (exception) {
       sendResponse(exception.response);
       return;

pkg/analysis_server/lib/src/lsp/lsp_analysis_server.dart

@@ -419,7 +419,7 @@ class LspAnalysisServer extends AnalysisServer {
     performanceAfterStartup = ServerPerformance();
     performance = performanceAfterStartup!;
 
-    _checkAnalytics();
+    checkAnalytics();
     enableSurveys();
 
     // Notify the client of any issues parsing experimental capabilities.
@@ -1073,21 +1073,6 @@ class LspAnalysisServer extends AnalysisServer {
     notifyFlutterWidgetDescriptions(path);
   }
 
-  /// Display a message that will allow us to enable analytics on the next run.
-  void _checkAnalytics() {
-    // TODO(dantup): This code should move to base server.
-    var unifiedAnalytics = analyticsManager.analytics;
-    var prompt = userPromptSender;
-    if (!unifiedAnalytics.shouldShowMessage || prompt == null) {
-      return;
-    }
-
-    unawaited(
-      prompt(MessageType.info, unifiedAnalytics.getConsentMessage, ['Ok']),
-    );
-    unifiedAnalytics.clientShowedMessage();
-  }
-
   /// Computes analysis roots for a set of open files.
   ///
   /// This is used when there are no workspace folders open directly.