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

Unified Diff: pkg/analysis_server/lib/src/plugin/server_plugin.dart

Issue 1073383002: Create a public API for contributing assists and make assists pluggable (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 5 years, 8 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/plugin/server_plugin.dart
diff --git a/pkg/analysis_server/lib/src/plugin/server_plugin.dart b/pkg/analysis_server/lib/src/plugin/server_plugin.dart
index a5cea51320368d9eef8d9f7289c86433a6949bc8..3d02551cada7fb264b9196ed5c5a526b164a2667 100644
--- a/pkg/analysis_server/lib/src/plugin/server_plugin.dart
+++ b/pkg/analysis_server/lib/src/plugin/server_plugin.dart
@@ -4,7 +4,9 @@
library analysis_server.src.plugin.server_plugin;
+import 'package:analysis_server/edit/assist/assist_core.dart';
import 'package:analysis_server/edit/fix/fix_core.dart';
+import 'package:analysis_server/plugin/assist.dart';
import 'package:analysis_server/plugin/fix.dart';
import 'package:analysis_server/src/analysis_server.dart';
import 'package:analysis_server/src/domain_analysis.dart';
@@ -14,6 +16,7 @@ import 'package:analysis_server/src/domain_server.dart';
import 'package:analysis_server/src/edit/edit_domain.dart';
import 'package:analysis_server/src/protocol.dart';
import 'package:analysis_server/src/search/search_domain.dart';
+import 'package:analysis_server/src/services/correction/assist_internal.dart';
import 'package:analysis_server/src/services/correction/fix_internal.dart';
import 'package:analyzer/plugin/plugin.dart';
@@ -30,6 +33,12 @@ typedef RequestHandler RequestHandlerFactory(AnalysisServer server);
class ServerPlugin implements Plugin {
/**
* The simple identifier of the extension point that allows plugins to
+ * register new assist contributors with the server.
+ */
+ static const String ASSIST_CONTRIBUTOR_EXTENSION_POINT = 'assistContributor';
+
+ /**
+ * The simple identifier of the extension point that allows plugins to
* register new domains with the server.
*/
static const String DOMAIN_EXTENSION_POINT = 'domain';
@@ -46,6 +55,12 @@ class ServerPlugin implements Plugin {
static const String UNIQUE_IDENTIFIER = 'analysis_server.core';
/**
+ * The extension point that allows plugins to register new assist contributors
+ * with the server.
+ */
+ ExtensionPoint assistContributorExtensionPoint;
+
+ /**
* The extension point that allows plugins to register new domains with the
* server.
*/
@@ -66,6 +81,13 @@ class ServerPlugin implements Plugin {
String get uniqueIdentifier => UNIQUE_IDENTIFIER;
/**
+ * Return a list containing all of the fix contributors that were contributed.
+ */
+ List<AssistContributor> assistContributors() {
+ return assistContributorExtensionPoint.extensions;
+ }
+
+ /**
* Use the given [server] to create all of the domains ([RequestHandler]'s)
* that have been registered and return the newly created domains.
*/
@@ -87,6 +109,9 @@ class ServerPlugin implements Plugin {
@override
void registerExtensionPoints(RegisterExtensionPoint registerExtensionPoint) {
+ assistContributorExtensionPoint = registerExtensionPoint(
+ ASSIST_CONTRIBUTOR_EXTENSION_POINT,
+ _validateAssistContributorExtension);
domainExtensionPoint = registerExtensionPoint(
DOMAIN_EXTENSION_POINT, _validateDomainExtension);
fixContributorExtensionPoint = registerExtensionPoint(
@@ -96,6 +121,11 @@ class ServerPlugin implements Plugin {
@override
void registerExtensions(RegisterExtension registerExtension) {
//
+ // Register assist contributors.
+ //
+ registerExtension(
+ ASSIST_CONTRIBUTOR_EXTENSION_POINT_ID, new DefaultAssistContributor());
+ //
// Register domains.
//
String domainId = Plugin.join(UNIQUE_IDENTIFIER, DOMAIN_EXTENSION_POINT);
@@ -120,6 +150,18 @@ class ServerPlugin implements Plugin {
/**
* Validate the given extension by throwing an [ExtensionError] if it is not a
+ * valid fix contributor.
+ */
+ void _validateAssistContributorExtension(Object extension) {
+ if (extension is! AssistContributor) {
+ String id = assistContributorExtensionPoint.uniqueIdentifier;
+ throw new ExtensionError(
+ 'Extensions to $id must be an AssistContributor');
+ }
+ }
+
+ /**
+ * Validate the given extension by throwing an [ExtensionError] if it is not a
* valid domain.
*/
void _validateDomainExtension(Object extension) {
@@ -136,7 +178,7 @@ class ServerPlugin implements Plugin {
*/
void _validateFixContributorExtension(Object extension) {
if (extension is! FixContributor) {
- String id = domainExtensionPoint.uniqueIdentifier;
+ String id = fixContributorExtensionPoint.uniqueIdentifier;
throw new ExtensionError('Extensions to $id must be a FixContributor');
}
}

Powered by Google App Engine
This is Rietveld 408576698