Clean up the test srapper code so that it does not use reportTestError, and made that private.

lib/unittest/interactive_html_config.dart

 // Copyright (c) 2012, 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.
 
 /**
  * This configuration can be used to rerun selected tests, as well
  * as see diagnostic output from tests. It runs each test in its own
  * IFrame, so the configuration consists of two parts - a 'master'
  * config that manages all the tests, and a 'slave' config for the
  * IFrame that runs the individual tests.
@@ skipping 55 matching lines @@
   /** The time at which tests start. */
   Map<int,Date> _testStarts;
 
   SlaveInteractiveHtmlConfiguration() :
       _testStarts = new Map<int,Date>();
 
   /** Don't start running tests automatically. */
   get autoStart() => false;
 
   void onInit() {
-    _onErrorClosure = (e) {
-      // TODO(vsm): figure out how to expose the stack trace here
-      // Currently e.message works in dartium, but not in dartc.
-      reportTestError('(DOM callback has errors) Caught ${e}', '');
-    };
+    _onErrorClosure =
+        (e) => handleExternalError(e, '(DOM callback has errors)');
 
     /**
      *  The master posts a 'start' message to kick things off,
      *  which is handled by this handler. It saves the master
      *  window, gets the test ID from the query parameter in the
      *  IFrame URL, sets that as a solo test and starts test execution.
      */
     window.on.message.add((MessageEvent e) {
       // Get the result, do any logging, then do a pass/fail.
       var m = new _Message.fromString(e.data);
@@ skipping 128 matching lines @@
         currentTestCase.fail(msg.body, _stack);
       } else if (msg.messageType == _Message.ERROR) {
         currentTestCase.error(msg.body, _stack);
       }
       completeTest();
     }
   }
 
   void onInit() {
     _messageHandler = _handleMessage; // We need to make just one closure.
-    _onErrorClosure = (e) {
-      // TODO(vsm): figure out how to expose the stack trace here
-      // Currently e.message works in dartium, but not in dartc.
-      reportTestError('(DOM callback has errors) Caught ${e}', '');
-    };
+    _onErrorClosure =
+        (e) => handleExternalError(e, '(DOM callback has errors)');
     document.query('#group-divs').innerHTML = "";
   }
 
   void onStart() {
     // Listen for uncaught errors.
     window.on.error.add(_onErrorClosure);
     if (!_doneWrap) {
       _doneWrap = true;
       for (int i = 0; i < testCases.length; i++) {
         testCases[i].test = wrapTest(testCases[i]);
@@ skipping 409 matching lines @@
 display: block;
 list-style-type: disc;
 -webkit-margin-before: 1em;
 -webkit-margin-after: 1em;
 -webkit-margin-start: 0px;
 -webkit-margin-end: 0px;
 -webkit-padding-start: 40px;
 }
 
   """;