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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: pkg/scheduled_test/lib/scheduled_process.dart

Issue 14070010: Refactor Future constructors. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 years, 8 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 | « pkg/oauth2/test/utils.dart ('k') | pkg/scheduled_test/lib/scheduled_server.dart » ('j') | sdk/lib/async/future.dart » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pkg/scheduled_test/lib/scheduled_process.dart
diff --git a/pkg/scheduled_test/lib/scheduled_process.dart b/pkg/scheduled_test/lib/scheduled_process.dart
index cf37211e21c2a9bf2004b2ad314082ef64985064..45e50616dbce1f8b41a03809a47fc610a15c6460 100644
--- a/pkg/scheduled_test/lib/scheduled_process.dart
+++ b/pkg/scheduled_test/lib/scheduled_process.dart
@@ -134,9 +134,9 @@ class ScheduledProcess {
_handleExit(exitCodeCompleter);
return Future.wait([
- new Future.of(() => executable),
+ new Future.sync(() => executable),
awaitObject(arguments),
- new Future.of(() => options)
+ new Future.sync(() => options)
]).then((results) {
executable = results[0];
arguments = results[1];
« no previous file with comments | « pkg/oauth2/test/utils.dart ('k') | pkg/scheduled_test/lib/scheduled_server.dart » ('j') | sdk/lib/async/future.dart » ('J')

Powered by Google App Engine
This is Rietveld 408576698