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

Side by Side Diff: tools/testing/dart/drt_updater.dart

Issue 11884029: Upload new Dart binaries and re-enable unit tests for test scripts. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Change Futures.wait to Future.wait in two test script files. 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « tools/testing/dart/co19_test.dart ('k') | tools/testing/dart/http_server.dart » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 library drt_updater; 5 library drt_updater;
6 6
7 import "dart:async"; 7 import "dart:async";
8 import "dart:io"; 8 import "dart:io";
9 9
10 import "test_suite.dart"; 10 import "test_suite.dart";
(...skipping 18 matching lines...) Expand all
29 onUpdated = [() {updated = true;} ]; 29 onUpdated = [() {updated = true;} ];
30 _updatingProcess = Process.run('python', _getUpdateCommand); 30 _updatingProcess = Process.run('python', _getUpdateCommand);
31 _updatingProcess.then(_onUpdatedHandler).catchError((e) { 31 _updatingProcess.then(_onUpdatedHandler).catchError((e) {
32 print("Error starting $script process: $e"); 32 print("Error starting $script process: $e");
33 return false; 33 return false;
34 }); 34 });
35 } 35 }
36 } 36 }
37 37
38 List<String> get _getUpdateCommand { 38 List<String> get _getUpdateCommand {
39 Path testScriptPath = new Path.fromNative(TestUtils.testScriptPath); 39 Path testScriptPath = new Path(TestUtils.testScriptPath);
40 Path updateScriptPath = testScriptPath.directoryPath.append(script); 40 Path updateScriptPath = testScriptPath.directoryPath.append(script);
41 List<String> command = [updateScriptPath.toNativePath()]; 41 List<String> command = [updateScriptPath.toNativePath()];
42 if (null != option) { 42 if (null != option) {
43 command.add(option); 43 command.add(option);
44 } 44 }
45 return command; 45 return command;
46 } 46 }
47 47
48 void _onUpdatedHandler(ProcessResult result) { 48 void _onUpdatedHandler(ProcessResult result) {
49 if (result.exitCode == 0) { 49 if (result.exitCode == 0) {
(...skipping 25 matching lines...) Expand all
75 // Download the default Dartium from Google Storage. 75 // Download the default Dartium from Google Storage.
76 if (_dartiumUpdater == null) { 76 if (_dartiumUpdater == null) {
77 _dartiumUpdater = new _DartiumUpdater('Dartium Chrome', 'get_archive.py', 77 _dartiumUpdater = new _DartiumUpdater('Dartium Chrome', 'get_archive.py',
78 'dartium'); 78 'dartium');
79 } 79 }
80 return _dartiumUpdater; 80 return _dartiumUpdater;
81 } else { 81 } else {
82 return null; 82 return null;
83 } 83 }
84 } 84 }
OLDNEW
« no previous file with comments | « tools/testing/dart/co19_test.dart ('k') | tools/testing/dart/http_server.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698