Fixes two type warnings in test_runner.dart

tools/testing/dart/test_runner.dart

Index: tools/testing/dart/test_runner.dart
diff --git a/tools/testing/dart/test_runner.dart b/tools/testing/dart/test_runner.dart
index 34d193dda321534ecf0e4ff69eb8f2070262606e..9dd9e0295c25dcc097b1fc68b8e9910c5063bc1e 100644
--- a/tools/testing/dart/test_runner.dart
+++ b/tools/testing/dart/test_runner.dart
@@ -331,12 +331,12 @@ class AnalysisTestOutputImpl extends TestOutputImpl {
       if (line.length == 0) continue;
       List<String> fields = splitMachineError(line);
       if (fields[0] == 'ERROR') {
-        errors.add(fields);
+        errors.add(fields[7]);

[Bill Hesse, 2012/03/23 22:16:08]

Perhaps add a comment that the relevant data is in field 8?  Or add constants
for the field positions?

Why did this not cause failures?  Do we only ever use errors.length and
staticWarnings.length?

[zundel, 2012/03/24 01:06:39]

Added constants.

In the case of multitests, it looks like only the lengths are checked.  Still, I
want to put something meaningful in there.

       } else if (fields[0] == 'WARNING') {
         // We only care about testing Static type warnings
         // ignore all others
         if (fields[1] == 'STATIC_TYPE') {
-          staticWarnings.add(fields);
+          staticWarnings.add(fields[7]);
         }
       }
       // OK to Skip error output that doesn't match the machine format