Flush results when contexts are made idle.

pkg/analysis_server/lib/src/operation/operation_analysis.dart

 // Copyright (c) 2014, the Dart project authors.  Please see the AUTHORS file
 // 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.
 
 library operation.analysis;
 
 import 'package:analysis_server/src/analysis_server.dart';
 import 'package:analysis_server/src/computer/computer_highlights.dart';
 import 'package:analysis_server/src/computer/computer_highlights2.dart';
 import 'package:analysis_server/src/computer/computer_outline.dart';
 import 'package:analysis_server/src/computer/computer_overrides.dart';
 import 'package:analysis_server/src/domains/analysis/implemented_dart.dart';
 import 'package:analysis_server/src/domains/analysis/navigation.dart';
 import 'package:analysis_server/src/domains/analysis/occurrences.dart';
 import 'package:analysis_server/src/operation/operation.dart';
 import 'package:analysis_server/src/protocol_server.dart' as protocol;
 import 'package:analysis_server/src/services/dependencies/library_dependencies.dart';
 import 'package:analysis_server/src/services/search/search_engine.dart';
 import 'package:analyzer/dart/ast/ast.dart';
 import 'package:analyzer/dart/element/element.dart';
 import 'package:analyzer/src/generated/engine.dart';
 import 'package:analyzer/src/generated/error.dart';
 import 'package:analyzer/src/generated/source.dart';
 
 /**
- * Runs the given function [f] with the working cache size in [context].
- * Returns the result of [f] invocation.
+ * Run the given function [f] with the given [context] made active.
+ * Return the result of [f] invocation.
  */
-runWithWorkingCacheSize(AnalysisContext context, f()) {
-  int currentCacheSize = context.analysisOptions.cacheSize;
-  if (currentCacheSize < PerformAnalysisOperation.WORKING_CACHE_SIZE) {
-    setCacheSize(context, PerformAnalysisOperation.WORKING_CACHE_SIZE);
+runWithActiveContext(AnalysisContext context, f()) {
+  if (context is InternalAnalysisContext && !context.isActive) {
+    context.isActive = true;
     try {
       return f();
     } finally {
-      setCacheSize(context, currentCacheSize);
+      context.isActive = false;
     }
   } else {
-    return f();
+    f();
   }
 }
 
 scheduleImplementedNotification(
     AnalysisServer server, Iterable<String> files) async {
   SearchEngine searchEngine = server.searchEngine;
   if (searchEngine == null) {
     return;
   }
   for (String file in files) {
@@ skipping 188 matching lines @@
 
 void sendAnalysisNotificationOverrides(
     AnalysisServer server, String file, CompilationUnit dartUnit) {
   _sendNotification(server, () {
     var overrides = new DartUnitOverridesComputer(dartUnit).compute();
     var params = new protocol.AnalysisOverridesParams(file, overrides);
     server.sendNotification(params.toNotification());
   });
 }
 
-/**
- * Sets the cache size in the given [context] to the given value.
- */
-void setCacheSize(AnalysisContext context, int cacheSize) {
-  // TODO(scheglov) The cache size cannot be changed with task model.
-  // TODO(scheglov) Consider removing this function.
-//  AnalysisOptionsImpl options =
-//      new AnalysisOptionsImpl.from(context.analysisOptions);
-//  options.cacheSize = cacheSize;
-//  context.analysisOptions = options;
-}
-
 String _computeLibraryName(CompilationUnit unit) {
   for (Directive directive in unit.directives) {
     if (directive is LibraryDirective && directive.name != null) {
       return directive.name.name;
     }
   }
   for (Directive directive in unit.directives) {
     if (directive is PartOfDirective && directive.libraryName != null) {
       return directive.libraryName.name;
     }
@@ skipping 48 matching lines @@
   @override
   void perform(AnalysisServer server) {
     sendAnalysisNotificationOccurrences(server, context, source);
   }
 }
 
 /**
  * Instances of [PerformAnalysisOperation] perform a single analysis task.
  */
 class PerformAnalysisOperation extends ServerOperation {
-  static const int IDLE_CACHE_SIZE = AnalysisOptionsImpl.DEFAULT_CACHE_SIZE;
-  static const int WORKING_CACHE_SIZE = 512;
-
   final bool isContinue;
 
   PerformAnalysisOperation(AnalysisContext context, this.isContinue)
       : super(context);
 
   @override
   ServerOperationPriority get priority {
     if (_isPriorityContext) {
       if (isContinue) {
         return ServerOperationPriority.PRIORITY_ANALYSIS_CONTINUE;
@@ skipping 17 matching lines @@
   void perform(AnalysisServer server) {
     //
     // TODO(brianwilkerson) Add an optional function-valued parameter to
     // performAnalysisTask that will be called when the task has been computed
     // but before it is performed and send notification in the function:
     //
     // AnalysisResult result = context.performAnalysisTask((taskDescription) {
     //   sendStatusNotification(context.toString(), taskDescription);
     // });
     if (!isContinue) {
-      setCacheSize(context, WORKING_CACHE_SIZE);
+      _setContextActive(true);
     }
     // prepare results
     AnalysisResult result = context.performAnalysisTask();
     List<ChangeNotice> notices = result.changeNotices;
     // nothing to analyze
     if (notices == null) {
       server.scheduleCacheConsistencyValidation(context);
-      setCacheSize(context, IDLE_CACHE_SIZE);
+      _setContextActive(false);
       server.sendContextAnalysisDoneNotifications(
           context, AnalysisDoneReason.COMPLETE);
       return;
     }
     // process results
     ServerPerformanceStatistics.notices.makeCurrentWhile(() {
       _sendNotices(server, notices);
       _updateIndex(server, notices);
     });
     // continue analysis
@@ skipping 11 matching lines @@
       // Dart
       CompilationUnit parsedDartUnit = notice.parsedDartUnit;
       CompilationUnit resolvedDartUnit = notice.resolvedDartUnit;
       scheduleNotificationOperations(server, source, file, notice.lineInfo,
           context, parsedDartUnit, resolvedDartUnit, notice.errors);
       // done
       server.fileAnalyzed(notice);
     }
   }
 
+  /**
+   * Make the [context] active or idle.
+   */
+  void _setContextActive(bool active) {
+    AnalysisContext context = this.context;
+    if (context is InternalAnalysisContext) {
+      context.isActive = active;
+    }
+  }
+
   void _updateIndex(AnalysisServer server, List<ChangeNotice> notices) {
     if (server.index == null) {
       return;
     }
     for (ChangeNotice notice in notices) {
       String file = notice.source.fullName;
       // Dart
       try {
         CompilationUnit dartUnit = notice.resolvedDartUnit;
         if (dartUnit != null) {
@@ skipping 117 matching lines @@
 abstract class _SingleFileOperation extends SourceSensitiveOperation {
   final String file;
 
   _SingleFileOperation(AnalysisContext context, this.file) : super(context);
 
   @override
   bool shouldBeDiscardedOnSourceChange(Source source) {
     return source.fullName == file;
   }
 }