Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5037)

Unified Diff: pkg/analysis_server/lib/src/context_manager.dart

Issue 2552023002: Send 'analysis.flushResults' on file removal. (Closed)
Patch Set: Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: pkg/analysis_server/lib/src/context_manager.dart
diff --git a/pkg/analysis_server/lib/src/context_manager.dart b/pkg/analysis_server/lib/src/context_manager.dart
index 41af5d9316c7eaa5667ced58e2c08671e22df760..4333b45473bfc3325df2c24b4aa627f86e2d756c 100644
--- a/pkg/analysis_server/lib/src/context_manager.dart
+++ b/pkg/analysis_server/lib/src/context_manager.dart
@@ -346,6 +346,11 @@ abstract class ContextManagerCallbacks {
AnalysisContext addContext(Folder folder, AnalysisOptions options);
/**
+ * The given [file] was removed from the folder analyzed in the [driver].
+ */
+ void applyFileRemoved(AnalysisDriver driver, String file);
+
+ /**
* Called when the set of files associated with a context have changed (or
* some of those files have been modified). [changeSet] is the set of
* changes that need to be applied to the context.
@@ -1479,7 +1484,7 @@ class ContextManagerImpl implements ContextManager {
}
if (enableNewAnalysisDriver) {
- info.analysisDriver.removeFile(path);
+ callbacks.applyFileRemoved(info.analysisDriver, path);
} else {
List<Source> sources = info.context.getSourcesWithFullName(path);
if (!sources.isEmpty) {
« no previous file with comments | « pkg/analysis_server/lib/src/analysis_server.dart ('k') | pkg/analysis_server/test/context_manager_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698