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

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

Issue 9610002: remove node.js dependency by running the frog compiler in (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: fix frog html Created 8 years, 9 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/test_runner.dart
diff --git a/tools/testing/dart/test_runner.dart b/tools/testing/dart/test_runner.dart
index d3c97e120ff8a83775c6a784639b05e0695b601d..a3232e357f2853332bd9beac844f793548290205 100644
--- a/tools/testing/dart/test_runner.dart
+++ b/tools/testing/dart/test_runner.dart
@@ -511,7 +511,6 @@ class RunningProcess {
if (currentStep == totalSteps) { // done with test command
testComplete(exitCode);
} else if (exitCode != 0) {
- stderr.add('test.dart: Compilation failed$suffix, exit code $exitCode\n');
Bill Hesse 2012/03/08 10:34:27 This is deleted because the absence of the "Compil
mattsh 2012/03/08 16:43:28 Good catch. I didn't intend to delete this, put b
testComplete(exitCode);
} else {
stderr.add('test.dart: Compilion finished $suffix\n');
Bill Hesse 2012/03/08 10:34:27 Compilation, not Compilion.
mattsh 2012/03/08 16:43:28 This spelling typo is unrelated to this CL, I'd pr

Powered by Google App Engine
This is Rietveld 408576698