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

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

Issue 1663863002: Add product mode: (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: Created 4 years, 11 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_suite.dart
diff --git a/tools/testing/dart/test_suite.dart b/tools/testing/dart/test_suite.dart
index fd67ebf12285386a679b0a18f4565398e71d86de..60a7ab5f61521e18b275664b2b96d9fce5ba0c7f 100644
--- a/tools/testing/dart/test_suite.dart
+++ b/tools/testing/dart/test_suite.dart
@@ -2325,7 +2325,19 @@ class TestUtils {
// is an X in front of the arch. We don't allow both a cross compiled
// and a normal version to be present (except if you specifically pass
// in the build_directory).
- var mode = (configuration['mode'] == 'debug') ? 'Debug' : 'Release';
+ var mode;
+ switch (configuration['mode']) {
+ case 'debug': mode = 'Debug';
Cutch 2016/02/03 17:09:01 newline
Ivan Posva 2016/02/03 22:11:08 Done.
+ break;
+ case 'release':
+ mode = 'Release';
+ break;
+ case 'product':
+ mode = 'Product';
+ break;
+ default:
+ throw 'Unrecognized mode configuration: ${configuration['mode']}';
+ }
var arch = configuration['arch'].toUpperCase();
var normal = '$mode$arch';
var cross = '${mode}X$arch';

Powered by Google App Engine
This is Rietveld 408576698