Send 'execution.launchData' only after full resolution.

pkg/analysis_server/lib/src/domain_execution.dart

Index: pkg/analysis_server/lib/src/domain_execution.dart
diff --git a/pkg/analysis_server/lib/src/domain_execution.dart b/pkg/analysis_server/lib/src/domain_execution.dart
index 145f1ba8fb64e77806916bd3e840edadd7b1ff47..88cc8705faf72ce61faecff051802e677a76e473 100644
--- a/pkg/analysis_server/lib/src/domain_execution.dart
+++ b/pkg/analysis_server/lib/src/domain_execution.dart
@@ -154,11 +154,19 @@ class ExecutionDomainHandler implements RequestHandler {
     ServerPerformanceStatistics.executionNotifications.makeCurrentWhile(() {
       Source source = notice.source;
       String filePath = source.fullName;
+      // check files
+      bool isDartFile = notice.resolvedDartUnit != null;
+      bool isHtmlFile = AnalysisEngine.isHtmlFileName(filePath);

[Brian Wilkerson, 2015/05/24 15:53:31]

Why not "notice.resolvedHtmlUnit != null"?

+      if (!isDartFile && !isHtmlFile) {
+        return;
+      }
+      // prepare context
       AnalysisContext context = server.getContainingContext(filePath);
       if (context == null) {
         return;
       }
-      if (AnalysisEngine.isDartFileName(filePath)) {
+      // analyze the file
+      if (isDartFile) {
         ExecutableKind kind = ExecutableKind.NOT_EXECUTABLE;
         if (context.isClientLibrary(source)) {
           kind = ExecutableKind.CLIENT;
@@ -171,7 +179,7 @@ class ExecutionDomainHandler implements RequestHandler {
         server.sendNotification(
             new ExecutionLaunchDataParams(filePath, kind: kind)
                 .toNotification());
-      } else if (AnalysisEngine.isHtmlFileName(filePath)) {
+      } else if (isHtmlFile) {
         List<Source> libraries = context.getLibrariesReferencedFromHtml(source);
         server.sendNotification(new ExecutionLaunchDataParams(filePath,
             referencedFiles: _getFullNames(libraries)).toNotification());