Separate compile-time and runtime errors.

dart/tools/testing/dart/multitest.dart

Index: dart/tools/testing/dart/multitest.dart
diff --git a/dart/tools/testing/dart/multitest.dart b/dart/tools/testing/dart/multitest.dart
index 418bd3803088d28b2a1aa22438699eef53e4a13c..93a9f0af6568bb10c8d4f9a59568372ea141f25e 100644
--- a/dart/tools/testing/dart/multitest.dart
+++ b/dart/tools/testing/dart/multitest.dart
@@ -191,8 +191,6 @@ Set<Path> _findAllRelativeImports(Path topLibrary) {
 void DoMultitest(Path filePath,
                  String outputDir,
                  Path suiteDir,
-                 // TODO(zundel): Are the boolean flags now redundant
-                 // with the 'multitestOutcome' field?
                  CreateTest doTest,
                  VoidFunction multitestDone) {
   // Each new test is a single String value in the Map tests.
@@ -234,14 +232,13 @@ void DoMultitest(Path filePath,
       Set<String> outcome = outcomes[key];
       bool enableFatalTypeErrors = outcome.contains('static type warning');
       bool hasRuntimeErrors = outcome.contains('runtime error');
-      bool isNegative = hasRuntimeErrors
-          || outcome.contains('compile-time error');
+      bool hasCompileError = outcome.contains('compile-time error');
       bool isNegativeIfChecked = outcome.contains('dynamic type error');
       doTest(multitestFilename,
-             isNegative,
+             hasCompileError,
+             hasRuntimeErrors,
              isNegativeIfChecked: isNegativeIfChecked,
              hasFatalTypeErrors: enableFatalTypeErrors,
-             hasRuntimeErrors: hasRuntimeErrors,
              multitestOutcome: outcome);
     }
     multitestDone();