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

Unified Diff: pkg/analysis_server/lib/src/operation/operation_analysis.dart

Issue 1303033010: Navigation extension point. (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: tweaks Created 5 years, 4 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 side-by-side diff with in-line comments
Download patch
Index: pkg/analysis_server/lib/src/operation/operation_analysis.dart
diff --git a/pkg/analysis_server/lib/src/operation/operation_analysis.dart b/pkg/analysis_server/lib/src/operation/operation_analysis.dart
index 70b1e9bafd8f7e219eb921246170a5dcc7c6f525..f18ad32139ce226dada6fb92319c8ffa74e2f190 100644
--- a/pkg/analysis_server/lib/src/operation/operation_analysis.dart
+++ b/pkg/analysis_server/lib/src/operation/operation_analysis.dart
@@ -7,10 +7,10 @@ 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_navigation.dart';
import 'package:analysis_server/src/computer/computer_occurrences.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/navigation.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';
@@ -53,9 +53,14 @@ void scheduleIndexOperation(AnalysisServer server, String file,
* Schedules sending notifications for the given [file] using the resolved
* [resolvedDartUnit].
*/
-void scheduleNotificationOperations(AnalysisServer server, String file,
- LineInfo lineInfo, AnalysisContext context, CompilationUnit parsedDartUnit,
- CompilationUnit resolvedDartUnit, List<AnalysisError> errors) {
+void scheduleNotificationOperations(
+ AnalysisServer server,
+ String file,
+ LineInfo lineInfo,
+ AnalysisContext context,
+ CompilationUnit parsedDartUnit,
+ CompilationUnit resolvedDartUnit,
+ List<AnalysisError> errors) {
// If the file belongs to any analysis root, check whether we're in it now.
AnalysisContext containingContext = server.getContainingContext(file);
if (containingContext != null && context != containingContext) {
@@ -162,12 +167,13 @@ void sendAnalysisNotificationHighlights(
}
void sendAnalysisNotificationNavigation(
- AnalysisServer server, String file, CompilationUnit dartUnit) {
+ AnalysisServer server, AnalysisContext context, Source source) {
_sendNotification(server, () {
- var computer = new DartUnitNavigationComputer();
- computer.compute(dartUnit);
+ NavigationHolderImpl holder =
+ computeNavigation(server, context, source, null, null);
+ String file = source.fullName;
var params = new protocol.AnalysisNavigationParams(
- file, computer.regions, computer.targets, computer.files);
+ file, holder.regions, holder.targets, holder.files);
server.sendNotification(params.toNotification());
});
}
@@ -381,7 +387,8 @@ class _DartNavigationOperation extends _DartNotificationOperation {
@override
void perform(AnalysisServer server) {
- sendAnalysisNotificationNavigation(server, file, unit);
+ Source source = unit.element.source;
+ sendAnalysisNotificationNavigation(server, context, source);
}
}

Powered by Google App Engine
This is Rietveld 408576698