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

Side by Side Diff: pkg/analysis_server/lib/src/analysis_server.dart

Issue 988593002: Don't send notifications for disposed contexts. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 5 years, 9 months 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 library analysis.server; 5 library analysis.server;
6 6
7 import 'dart:async'; 7 import 'dart:async';
8 import 'dart:collection'; 8 import 'dart:collection';
9 import 'dart:math' show max; 9 import 'dart:math' show max;
10 10
(...skipping 1211 matching lines...) Expand 10 before | Expand all | Expand 10 after
1222 void endComputePackageMap() { 1222 void endComputePackageMap() {
1223 _computingPackageMap(false); 1223 _computingPackageMap(false);
1224 } 1224 }
1225 1225
1226 @override 1226 @override
1227 void removeContext(Folder folder) { 1227 void removeContext(Folder folder) {
1228 AnalysisContext context = analysisServer.folderMap.remove(folder); 1228 AnalysisContext context = analysisServer.folderMap.remove(folder);
1229 if (analysisServer.index != null) { 1229 if (analysisServer.index != null) {
1230 analysisServer.index.removeContext(context); 1230 analysisServer.index.removeContext(context);
1231 } 1231 }
1232 analysisServer.operationQueue.contextRemoved(context);
1232 _onContextsChangedController 1233 _onContextsChangedController
1233 .add(new ContextsChangedEvent(removed: [context])); 1234 .add(new ContextsChangedEvent(removed: [context]));
1234 analysisServer.sendContextAnalysisDoneNotifications( 1235 analysisServer.sendContextAnalysisDoneNotifications(
1235 context, AnalysisDoneReason.CONTEXT_REMOVED); 1236 context, AnalysisDoneReason.CONTEXT_REMOVED);
1236 context.dispose(); 1237 context.dispose();
1237 } 1238 }
1238 1239
1239 @override 1240 @override
1240 void updateContextPackageUriResolver( 1241 void updateContextPackageUriResolver(
1241 Folder contextFolder, UriResolver packageUriResolver) { 1242 Folder contextFolder, UriResolver packageUriResolver) {
(...skipping 121 matching lines...) Expand 10 before | Expand all | Expand 10 after
1363 /** 1364 /**
1364 * The [PerformanceTag] for time spent in server request handlers. 1365 * The [PerformanceTag] for time spent in server request handlers.
1365 */ 1366 */
1366 static PerformanceTag serverRequests = new PerformanceTag('serverRequests'); 1367 static PerformanceTag serverRequests = new PerformanceTag('serverRequests');
1367 1368
1368 /** 1369 /**
1369 * The [PerformanceTag] for time spent in split store microtasks. 1370 * The [PerformanceTag] for time spent in split store microtasks.
1370 */ 1371 */
1371 static PerformanceTag splitStore = new PerformanceTag('splitStore'); 1372 static PerformanceTag splitStore = new PerformanceTag('splitStore');
1372 } 1373 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698