Reduce test.dart memory usage

tools/testing/dart/test_progress.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.
 
 library test_progress;
 
 import "dart:async";
 import "dart:io";
 import "dart:io" as io;
 import "dart:convert" show JSON;
@@ skipping 61 matching lines @@
   output.add('');
   output.add(formatter.failed('FAILED: ${test.configurationString}'
                               ' ${test.displayName}'));
   StringBuffer expected = new StringBuffer();
   expected.write('Expected: ');
   for (var expectation in test.expectedOutcomes) {
     expected.write('$expectation ');
   }
   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.');
     } else {
       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.');
       }
     }
   }
   for (var i = 0; i < test.commands.length; i++) {
     var command = test.commands[i];
     var commandOutput = test.commandOutputs[command];
     if (commandOutput != null) {
       output.add("CommandOutput[${command.displayName}]:");
       if (!commandOutput.diagnostics.isEmpty) {
@@ skipping 557 matching lines @@
       return new VerboseProgressIndicator(startTime);
     case 'status':
       return new ProgressIndicator(startTime);
     case 'buildbot':
       return new BuildbotProgressIndicator(startTime);
     default:
       assert(false);
       break;
   }
 }