Package-ify Dromaeo and browser controller functionality

samples/third_party/dromaeo/web/Dromaeo.dart

 library dromaeo_test;
 
 import 'dart:html';
 import 'dart:async';
 import "dart:convert";
 import 'dart:math' as Math;
+import 'dart:js' as js;
 import 'Suites.dart';
 
 main() {
   new Dromaeo().run();
 }
 
 class SuiteController {
   final SuiteDescription _suiteDescription;
   final IFrameElement _suiteIframe;
 
@@ skipping 177 matching lines @@
           _updateTime();
           return running;
 
         case 'over':
           currentSuite++;
           if (currentSuite < _suitesTotal) {
             _suiteControllers[currentSuite].start();
             return running;
           }
           document.body.attributes['class'] = 'alldone';
+          try {

[vsm, 2014/03/13 22:02:21]

The try-catch is a bit ugly here.  Consider something like:

var report = js.context['reportPerformanceTestDone'];
if (report != null) {
  report.apply([]);
} else {
  ...
}

[Emily Fortuna, 2014/03/13 22:43:07]

Fixed!

+            js.context.callMethod('reportPerformanceTestDone');
+          } catch (e) {
+            // This is not running as a performance test. Continue as normal.
+            window.console.log('Failed to call reportPerformanceTestDone. $e');
+          }
           return done;
 
         default:
           throw 'Unknown command ${command} [${data}]';
       }
     };
 
     done = (String command, var data) {
     };
 
     return loading;
   }
 
   _css(Element element, String property, String value) {
     // TODO(antonm): remove the last argument when CallWithDefaultValue
     // is implemented.
     element.style.setProperty(property, value, '');
   }
 
   Element _byId(String id) {
     return document.querySelector('#$id');
   }
 
   int get _suitesTotal {
     return _suiteControllers.length;
   }
 }