Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(19)

Unified Diff: tools/testing/dart/multitest.dart

Issue 9559007: Update test.dart for detection output of machine formatted errors (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Added diagnostic for @static-clean conflicts Created 8 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: tools/testing/dart/multitest.dart
diff --git a/tools/testing/dart/multitest.dart b/tools/testing/dart/multitest.dart
index a637c987e23872ae08b9ca0b87ac5a08127bd96f..c1f992a5e68a5d8445236acec32b7843fc90f646 100644
--- a/tools/testing/dart/multitest.dart
+++ b/tools/testing/dart/multitest.dart
@@ -127,7 +127,8 @@ void DoMultitest(String filename,
bool isNegative,
[bool isNegativeIfChecked,
bool hasFatalTypeErrors,
- bool hasRuntimeErrors]),
+ bool hasRuntimeErrors,
+ String multitestOutcome]),
Function multitestDone) {
// Each new test is a single String value in the Map tests.
Map<String, String> tests = new Map<String, String>();
@@ -158,7 +159,8 @@ void DoMultitest(String filename,
isNegative,
isNegativeIfChecked,
enableFatalTypeErrors,
Bill Hesse 2012/03/02 08:55:18 Can't you remove all the boolean flag arguments fr
zundel 2012/03/02 23:51:00 I think you are right, I added TODO.
- hasRuntimeErrors);
+ hasRuntimeErrors,
+ outcome);
}
multitestDone();
}

Powered by Google App Engine
This is Rietveld 408576698