Revert r13286 "Fix issues with test.dart that appeared on Mac."

tools/testing/dart/test_progress.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.
 
 #library("test_progress");
 
 #import("dart:io");
 #import("test_runner.dart");
 #import("test_suite.dart");
 
@@ skipping 68 matching lines @@
       }
     }
   }
 
   void allDone() {
     _printFailureSummary();
     _printStatus();
     _printTimingInformation();
     stdout.close();
     stderr.close();
-    exit(_failedTests > 0 ? 1 : 0);
+    if (_failedTests > 0) exit(1);

[gram, 2012/10/08 22:34:08]

Did you intend to make this change? It seems the older version is more correct;
either way you want to call exit don't you?

[Emily Fortuna, 2012/10/08 22:37:23]

Yes, this was intentional. As mentioned in the commit message -- this reverts
the change I made earlier this morning back to the way Mads had written it.

   }
 
   void _printStartProgress(TestCase test) {}
   void _printDoneProgress(TestCase test) {}
 
   String _pad(String s, int length) {
     StringBuffer buffer = new StringBuffer();
     for (int i = s.length; i < length; i++) {
       buffer.add(' ');
     }
@@ skipping 113 matching lines @@
 
 class SilentProgressIndicator extends ProgressIndicator {
   SilentProgressIndicator(Date startTime, bool printTiming)
       : super(startTime, printTiming);
   void testAdded() { }
   void start(TestCase test) { }
   void done(TestCase test) { }
   void _printStartProgress(TestCase test) { }
   void _printDoneProgress(TestCase test) { }
   void allTestsKnown() { }
-  void allDone() { 
-    exit(0);
-  }
+  void allDone() { }
 }
 
 abstract class CompactIndicator extends ProgressIndicator {
   CompactIndicator(Date startTime, bool printTiming)
       : super(startTime, printTiming);
 
   void allDone() {
     stdout.write('\n'.charCodes());
     _printFailureSummary();
     _printTimingInformation();
     if (_failedTests > 0) {
       // We may have printed many failure logs, so reprint the summary data.
       _printProgress();
       print('');
     }
     stdout.close();
     stderr.close();
-    exit(_failedTests > 0 ? 1 : 0);
+    if (_failedTests > 0) exit(1);

[gram, 2012/10/08 22:34:08]

Ditto

   }
 
   void allTestsKnown() {
     if (!_allTestsKnown && SummaryReport.total > 0) {
       // Clear progress indicator before printing summary report.
       stdout.write(
           '\r                                               \r'.charCodes());
       SummaryReport.printReport();
     }
     _allTestsKnown = true;
@@ skipping 159 matching lines @@
       print('');
       print('$key:');
       lines.sort((a, b) => a.compareTo(b));
       for (String line in lines) {
         print('  $line');
       }
     });
     _printStatus();
   }
 }