More fixes for compact_vm_config. The previous CL handled prints that occurred

pkg/unittest/lib/compact_vm_config.dart

 // Copyright (c) 2013, 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.
 
 /**
  * A test configuration that generates a compact 1-line progress bar. The bar is
  * updated in-place before and after each test is executed. If all test pass,
  * you should only see a couple lines in the terminal. If a test fails, the
  * failure is shown and the progress bar continues to be updated below it.
  */
@@ skipping 38 matching lines @@
   }
 
   void onTestResult(TestCase test) {
     super.onTestResult(test);
     if (test.result == PASS) {
       _pass++;
       _progressLine(_start, _pass, _fail, test.description);
     } else {
       _fail++;
       _progressLine(_start, _pass, _fail, test.description);
-      stdout.write('\n');
+      _print();
       if (test.message != '') {
-        print(indent(test.message));
+        _print(indent(test.message));
       }
 
       if (test.stackTrace != null) {
-        print(indent(test.stackTrace.toString()));
+        _print(indent(test.stackTrace.toString()));
       }
     }
   }
 
   void onTestResultChanged(TestCase test) {
     _pass--;
     _fail++;
     _progressLine(_start, _pass, _fail, test.description);
-    stdout.write('\n');
+    _print();
     if (test.message != '') {
-      print(indent(test.message));
+      _print(indent(test.message));
     }
 
     if (test.stackTrace != null) {
-      print(indent(test.stackTrace.toString()));
+      _print(indent(test.stackTrace.toString()));
     }
   }
 
   void onDone(bool success) {
     // Override and don't call the superclass onDone() to avoid printing the
     // "unittest-suite-..." boilerplate.
     Future.wait([stdout.close(), stderr.close()]).then((_) {
       _receivePort.close();
       exit(success ? 0 : 1);
     });
   }
 
   void onSummary(int passed, int failed, int errors, List<TestCase> results,
       String uncaughtError) {
     var success = false;
     if (passed == 0 && failed == 0 && errors == 0 && uncaughtError == null) {
-      print('\nNo tests ran.');
+      _print('\nNo tests ran.');
     } else if (failed == 0 && errors == 0 && uncaughtError == null) {
       _progressLine(_start, _pass, _fail, 'All tests passed!', _NONE);
-      stdout.write('\n');
+      _print();
       success = true;
     } else {
       _progressLine(_start, _pass, _fail, 'Some tests failed.', _RED);
-      stdout.write('\n');
+      _print();
       if (uncaughtError != null) {
-        print('Top-level uncaught error: $uncaughtError');
+        _print('Top-level uncaught error: $uncaughtError');
       }
-      print('$passed PASSED, $failed FAILED, $errors ERRORS');
+      _print('$passed PASSED, $failed FAILED, $errors ERRORS');
     }
   }
 
   int _lastLength = 0;
 
   final int _nonVisiblePrefix = 1 + _GREEN.length + _NONE.length;
 
   void _progressLine(DateTime startTime, int passed, int failed, String message,
       [String color = _NONE]) {
     var duration = (new DateTime.now()).difference(startTime);
@@ skipping 75 matching lines @@
     // the beginning of a word.
     var res = text.substring(text.length - maxLength + 4);
     var firstSpace = res.indexOf(' ');
     if (firstSpace > 0) {
       res = res.substring(firstSpace);
     }
     return '...$res';
   }
 }
 
+// TODO(sigmund): delete when dartbug.com/17269 is fixed (use `print` instead).
+_print([value = '']) => stdout.write('$value\n');
+
 void useCompactVMConfiguration() {
   // If the test is running on the Dart buildbots, we don't want to use this
   // config since it's output may not be what the bots expect.
   if (Platform.environment['LOGNAME'] == 'chrome-bot') {
     return;
   }
 
   unittestConfiguration = _singleton;
 }
 
 final _singleton = new CompactVMConfiguration();