Do not listen to errors twice in test_controller.js

pkg/unittest/lib/test_controller.js

 // Copyright (c) 2011, 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.
 
 /**
  * Test controller logic - used by unit test harness to embed tests in
  * conent shell.
  */
 
 // Clear the console before every test run - this is Firebug specific code.
@@ skipping 103 matching lines @@
 
 // Note: before renaming this function, note that it is also included in an
 // inlined error handler in the HTML files that wrap DRT tests.
 // See: tools/testing/dart/browser_test.dart
 function externalError(e) {
   // needed for dartium compilation errors.
   showErrorAndExit(e && e.message);
   window.postMessage('unittest-suite-external-error', '*');
 }
 
-// If nobody intercepts the error, finish the test.
-window.addEventListener("error", externalError, false);
-
 document.addEventListener('readystatechange', function () {
   if (document.readyState != "loaded") return;
   // If 'startedDartTest' is not set, that means that the test did not have
   // a chance to load. This will happen when a load error occurs in the VM.
   // Give the machine time to start up.
   setTimeout(function() {
     // A window.postMessage might have been enqueued after this timeout.
     // Just sleep another time to give the browser the time to process the
     // posted message.
     setTimeout(function() {
@@ skipping 29 matching lines @@
   try {
     main();
   } catch (e) {
     dartPrint(e);
     if (e.stack) dartPrint(e.stack);
     window.postMessage('unittest-suite-fail', '*');
     return;
   }
   window.postMessage('dart-main-done', '*');
 }