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

Unified Diff: utils/testrunner/options.dart

Issue 11783009: Big merge from experimental to bleeding edge. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 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
« no previous file with comments | « utils/template/utils.dart ('k') | utils/testrunner/run_pipeline.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: utils/testrunner/options.dart
diff --git a/utils/testrunner/options.dart b/utils/testrunner/options.dart
index c05773de79cf9d9592223573f59fadcdb5cd4346..5d21bd43524ba97a63bcbdafb47889e833c90282 100644
--- a/utils/testrunner/options.dart
+++ b/utils/testrunner/options.dart
@@ -200,7 +200,7 @@ ArgResults loadConfiguration(optionsParser) {
var options = new List();
// We first load options from a test.config file in the working directory.
options.addAll(getFileContents('test.config', false).
- filter((e) => e.trim().length > 0 && e[0] != '#'));
+ where((e) => e.trim().length > 0 && e[0] != '#'));
// Next we look to see if the command line included a -testconfig argument,
// and if so, load options from that file too; where these are not
// multi-valued they will take precedence over the ones in test.config.
@@ -213,11 +213,11 @@ ArgResults loadConfiguration(optionsParser) {
throw new Exception('Missing argument to $cfgarg');
}
options.addAll(getFileContents(commandLineArgs[++i], true).
- filter((e) => e.trim().length > 0 && e[0] != '#'));
+ where((e) => e.trim().length > 0 && e[0] != '#'));
} else if (commandLineArgs[i].startsWith('$cfgarg=')) {
options.addAll(
getFileContents(commandLineArgs[i].substring(cfgarg.length), true).
- filter((e) => e.trim().length > 0 && e[0] != '#'));
+ where((e) => e.trim().length > 0 && e[0] != '#'));
} else {
throw new Exception('Missing argument to $cfgarg');
}
« no previous file with comments | « utils/template/utils.dart ('k') | utils/testrunner/run_pipeline.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698