Reduce test.dart memory usage

tools/testing/dart/test_progress.dart

Index: tools/testing/dart/test_progress.dart
diff --git a/tools/testing/dart/test_progress.dart b/tools/testing/dart/test_progress.dart
index 6bfa8283abd772d82e166cf0631576b8ab6dac09..e315c3465c42ed5ffef3ff0193e0fefbac5148c9 100644
--- a/tools/testing/dart/test_progress.dart
+++ b/tools/testing/dart/test_progress.dart
@@ -79,7 +79,7 @@ List<String> _buildFailureOutput(TestCase test,
   }
   output.add(expected.toString());
   output.add('Actual: ${test.result}');
-  if (!test.lastCommandOutput.hasTimedOut && test.info != null) {
+  if (!test.lastCommandOutput.hasTimedOut) {
     if (test.commandOutputs.length != test.commands.length
         && !test.expectCompileError) {

[Bill Hesse, 2014/04/23 17:38:44]

This is the one case who's semantics is changed - the cctests and package
compile tests, with no test.info, will output this if the rest of the condition
holds, since expectCompileError will be false.  This seems correct.

       output.add('Unexpected compile-time error.');
@@ -87,10 +87,10 @@ List<String> _buildFailureOutput(TestCase test,
       if (test.expectCompileError) {
         output.add('Compile-time error expected.');
       }
-      if (test.info.hasRuntimeError) {
+      if (test.hasRuntimeError) {
         output.add('Runtime error expected.');
       }
-      if (test.configuration['checked'] && test.info.isNegativeIfChecked) {
+      if (test.configuration['checked'] && test.isNegativeIfChecked) {
         output.add('Dynamic type error expected.');
       }
     }