Test running changes.

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 1cae1d7773c8e79629eb85a3e4fb0b962accf879..2a26836c5642a7f68a3433fd28ae0333bd03b915 100644
--- a/tools/testing/dart/test_runner.dart
+++ b/tools/testing/dart/test_runner.dart
@@ -850,7 +850,9 @@ class AnalysisCommandOutputImpl extends CommandOutputImpl {
       } else if (fields[ERROR_LEVEL] == 'WARNING') {
         // We only care about testing Static type warnings
         // ignore all others

[kustermann, 2013/07/01 06:24:17]

This comment seems to be wrong now.

Is there a reason why you switch on the 'ERROR_TYPE'. If you want all warnings
to be reported as warnings, you should do:
  if (fields[ERROR_LEVEL] == 'ERROR') {
    errors.add(fields[FORMATTED_ERROR]);
  } else if (fields[ERROR_LEVEL] == 'WARNING') {
    warnings.add(fields[FORMATTED_ERROR]);
  }

Are there other kinds of warnings we want to ignore?

[scheglov, 2013/07/01 07:04:30]

  Thank you, fixed.

-        if (fields[ERROR_TYPE] == 'STATIC_TYPE' || fields[ERROR_TYPE] == 'STATIC_TYPE_WARNING') {
+        if (fields[ERROR_TYPE] == 'STATIC_TYPE' ||
+            fields[ERROR_TYPE] == 'STATIC_WARNING' ||
+            fields[ERROR_TYPE] == 'STATIC_TYPE_WARNING') {
           staticWarnings.add(fields[FORMATTED_ERROR]);
         }
       }