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

Unified Diff: editor/tools/plugins/com.google.dart.server/src/com/google/dart/server/internal/remote/processor/NotificationServerConnectedProcessor.java

Issue 943723002: Return version with connected notification (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Touched missed Java files Created 5 years, 10 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: editor/tools/plugins/com.google.dart.server/src/com/google/dart/server/internal/remote/processor/NotificationServerConnectedProcessor.java
diff --git a/editor/tools/plugins/com.google.dart.server/src/com/google/dart/server/internal/remote/processor/NotificationServerConnectedProcessor.java b/editor/tools/plugins/com.google.dart.server/src/com/google/dart/server/internal/remote/processor/NotificationServerConnectedProcessor.java
index 8821a7296714a0c55f247276fe5e10e73ea76732..aff172933da8a69310b8e747df189a4edce7227f 100644
--- a/editor/tools/plugins/com.google.dart.server/src/com/google/dart/server/internal/remote/processor/NotificationServerConnectedProcessor.java
+++ b/editor/tools/plugins/com.google.dart.server/src/com/google/dart/server/internal/remote/processor/NotificationServerConnectedProcessor.java
@@ -14,6 +14,7 @@
package com.google.dart.server.internal.remote.processor;
import com.google.dart.server.AnalysisServerListener;
+import com.google.gson.JsonElement;
import com.google.gson.JsonObject;
/**
@@ -29,6 +30,17 @@ public class NotificationServerConnectedProcessor extends NotificationProcessor
@Override
public void process(JsonObject response) throws Exception {
- getListener().serverConnected();
+ getListener().serverConnected(getVersion(response));
+ }
+
+ private String getVersion(JsonObject response) {
+ JsonElement paramsElement = response.get("params");
+ if (paramsElement != null) {
+ JsonElement versionElement = paramsElement.getAsJsonObject().get("version");
+ if (versionElement != null) {
+ return versionElement.getAsString();
+ }
+ }
+ return null;
}
}

Powered by Google App Engine
This is Rietveld 408576698