Send 'execution.launchData' only after full resolution.

pkg/analysis_server/lib/src/domain_execution.dart

 // Copyright (c) 2014, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 library domain.execution;
 
 import 'dart:async';
 import 'dart:collection';
 
 import 'package:analysis_server/src/analysis_server.dart';
@@ skipping 136 matching lines @@
         onFileAnalyzed = null;
       }
     }
     return new ExecutionSetSubscriptionsResult().toResponse(request.id);
   }
 
   void _fileAnalyzed(ChangeNotice notice) {
     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;
           if (context.isServerLibrary(source)) {
             kind = ExecutableKind.EITHER;
           }
         } else if (context.isServerLibrary(source)) {
           kind = ExecutableKind.SERVER;
         }
         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());
       }
     });
   }
 
   /**
    * Return `true` if the given [filePath] represents a file that is in an
    * analysis root.
@@ skipping 41 matching lines @@
     if (_isInAnalysisRoot(filePath)) {
       server.sendNotification(
           new ExecutionLaunchDataParams(filePath, kind: kind).toNotification());
     }
   }
 
   static List<String> _getFullNames(List<Source> sources) {
     return sources.map((Source source) => source.fullName).toList();
   }
 }