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

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

Issue 91473002: Checked-in binary update to v1.0.0.3 (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 7 years, 1 month 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 | « dart/tools/testing/dart/browser_controller.dart ('k') | dart/tools/testing/dart/launch_browser.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: dart/tools/testing/dart/http_server.dart
diff --git a/dart/tools/testing/dart/http_server.dart b/dart/tools/testing/dart/http_server.dart
index 93487632a58e1cc893704d24058799c3009d4dcd..5a55119238771586c36cfe59837a5bc6036eea7c 100644
--- a/dart/tools/testing/dart/http_server.dart
+++ b/dart/tools/testing/dart/http_server.dart
@@ -6,7 +6,6 @@ library http_server;
import 'dart:async';
import 'dart:io';
-import 'dart:isolate';
import 'test_suite.dart'; // For TestUtils.
// TODO(efortuna): Rewrite to not use the args library and simply take an
// expected number of arguments, so test.dart doesn't rely on the args library?
@@ -38,7 +37,7 @@ const PREFIX_DARTDIR = 'root_dart';
// http://host:port/root_packages/X -> $BuildDir/packages/X
// Issue: 8368
-main() {
+main(List<String> arguments) {
/** Convenience method for local testing. */
var parser = new ArgParser();
parser.addOption('port', abbr: 'p',
@@ -57,14 +56,14 @@ main() {
parser.addOption('runtime', help: 'The runtime we are using (for csp flags).',
defaultsTo: 'none');
- var args = parser.parse(new Options().arguments);
+ var args = parser.parse(arguments);
if (args['help']) {
print(parser.getUsage());
} else {
// Pretend we're running test.dart so that TestUtils doesn't get confused
// about the "current directory." This is only used if we're trying to run
// this file independently for local testing.
- TestUtils.testScriptPath = new Path(new Options().script)
+ TestUtils.testScriptPath = new Path(Platform.script.path)
.directoryPath
.join(new Path('../../test.dart'))
.canonicalize()
« no previous file with comments | « dart/tools/testing/dart/browser_controller.dart ('k') | dart/tools/testing/dart/launch_browser.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698