First cut at server timing tests

pkg/analysis_server/lib/src/analysis_server.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 analysis.server;
 
 import 'dart:async';
 
 import 'package:analysis_server/src/analysis_logger.dart';
 import 'package:analysis_server/src/channel.dart';
@@ skipping 193 matching lines @@
       contextWorkQueue.clear();
     }
     if (contextWorkQueue.isEmpty) {
       // Nothing to do.
       return;
     }
     //
     // Look for a context that has work to be done and then perform one task.
     //
     List<ChangeNotice> notices = null;
-//    String contextId;
     try {
       AnalysisContext context = contextWorkQueue[0];
-//      contextId = contextIdMap[context];
-      // TODO(danrubel): Replace with context identifier or similar
+      //
+      // TODO(brianwilkerson) Add an optional function-valued parameter to
+      // performAnalysisTask that will be called when the task has been computed
+      // but before it is performed and send notification in the function:
+      //
+      // AnalysisResult result = context.performAnalysisTask((taskDescription) {
+      //   sendStatusNotification(context.toString(), taskDescription);
+      // });
+      //
       sendStatusNotification(context.toString());
       AnalysisResult result = context.performAnalysisTask();
       notices = result.changeNotices;
     } finally {
       if (notices == null) {
         // Either we have no more work to do for this context, or there was an
         // unhandled exception trying to perform the analysis.  In either case,
         // remove the context form the work queue so we won't try to do more
         // analysis on it.
         contextWorkQueue.removeAt(0);
+      } else {
+        sendNotices(notices);
       }
       //
       // Schedule this method to be run again if there is any more work to be
       // done.
       //
-      if (!contextWorkQueue.isEmpty) {
+      if (contextWorkQueue.isEmpty) {
+        sendStatusNotification(null);
+      } else {
         _scheduleTask();
       }
     }
-    if (notices != null) {
-      sendNotices(notices);
-    } else {
-      sendStatusNotification(null);
-    }
   }
 
   /**
    * Send the information in the given list of notices back to the client.
    */
   void sendNotices(List<ChangeNotice> notices) {
     for (int i = 0; i < notices.length; i++) {
       ChangeNotice notice = notices[i];
       Source source = notice.source;
       CompilationUnit dartUnit = notice.compilationUnit;
@@ skipping 26 matching lines @@
         REGIONS,
         new DartUnitHighlightsComputer(dartUnit).compute());
     sendNotification(notification);
   }
 
   /**
    * Send status notification to the client. The `contextId` indicates
    * the current context being analyzed or `null` if analysis is complete.
    */
   void sendStatusNotification(String contextId) {
-    if (contextId == lastStatusNotificationContextId) {
-      return;
-    }
+//    if (contextId == lastStatusNotificationContextId) {
+//      return;
+//    }
     lastStatusNotificationContextId = contextId;
     Notification notification = new Notification(NOTIFICATION_STATUS);
     Map<String, Object> analysis = new Map();
     if (contextId != null) {
       analysis['analyzing'] = true;
       // TODO(danrubel): replace contextId with real analysisTarget
       analysis['analysisTarget'] = contextId;
     } else {
       analysis['analyzing'] = false;
     }
@@ skipping 214 matching lines @@
 /**
  * An enumeration of the services provided by the server domain.
  */
 class ServerService extends Enum2<ServerService> {
   static const ServerService STATUS = const ServerService('STATUS', 0);
 
   static const List<ServerService> VALUES = const [STATUS];
 
   const ServerService(String name, int ordinal) : super(name, ordinal);
 }