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

Side by Side Diff: tools/test.dart

Issue 22420002: Add support for ie in the browser controller (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: Created 7 years, 4 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
OLDNEW
1 #!/usr/bin/env dart 1 #!/usr/bin/env dart
2 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file 2 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file
3 // for details. All rights reserved. Use of this source code is governed by a 3 // for details. All rights reserved. Use of this source code is governed by a
4 // BSD-style license that can be found in the LICENSE file. 4 // BSD-style license that can be found in the LICENSE file.
5 5
6 /** 6 /**
7 * This file is the entrypoint of the dart test suite. This suite is used 7 * This file is the entrypoint of the dart test suite. This suite is used
8 * to test: 8 * to test:
9 * 9 *
10 * 1. the dart vm 10 * 1. the dart vm
(...skipping 134 matching lines...) Expand 10 before | Expand all | Expand 10 after
145 var servers = new TestingServers(new Path(TestUtils.buildDir(conf)), 145 var servers = new TestingServers(new Path(TestUtils.buildDir(conf)),
146 useContentSecurityPolicy, 146 useContentSecurityPolicy,
147 conf['runtime']); 147 conf['runtime']);
148 serverFutures.add(servers.startServers(conf['local_ip'])); 148 serverFutures.add(servers.startServers(conf['local_ip']));
149 conf['_servers_'] = servers; 149 conf['_servers_'] = servers;
150 } 150 }
151 151
152 // There should not be more than one InternetExplorerDriver instance 152 // There should not be more than one InternetExplorerDriver instance
153 // running at a time. For details, see 153 // running at a time. For details, see
154 // http://code.google.com/p/selenium/wiki/InternetExplorerDriver. 154 // http://code.google.com/p/selenium/wiki/InternetExplorerDriver.
155 if (conf['runtime'].startsWith('ie')) { 155 if (conf['runtime'].startsWith('ie') && !conf["use_browser_controller"]) {
156 maxBrowserProcesses = 1; 156 maxBrowserProcesses = 1;
kustermann 2013/08/06 13:58:45 Does that mean we can now use multiple versions of
ricow1 2013/08/13 06:42:43 Yes :-)
157 } else if (conf['runtime'].startsWith('safari') && 157 } else if (conf['runtime'].startsWith('safari') &&
158 conf['use_browser_controller']) { 158 conf['use_browser_controller']) {
159 // Safari does not allow us to run from a fresh profile, so we can only 159 // Safari does not allow us to run from a fresh profile, so we can only
160 // use one browser. 160 // use one browser.
161 maxBrowserProcesses = 1; 161 maxBrowserProcesses = 1;
162 } 162 }
163 163
164 for (String key in selectors.keys) { 164 for (String key in selectors.keys) {
165 if (key == 'co19') { 165 if (key == 'co19') {
166 testSuites.add(new Co19TestSuite(conf)); 166 testSuites.add(new Co19TestSuite(conf));
(...skipping 127 matching lines...) Expand 10 before | Expand all | Expand 10 after
294 void main() { 294 void main() {
295 deleteTemporaryDartDirectories().then((_) { 295 deleteTemporaryDartDirectories().then((_) {
296 var optionsParser = new TestOptionsParser(); 296 var optionsParser = new TestOptionsParser();
297 var configurations = optionsParser.parse(new Options().arguments); 297 var configurations = optionsParser.parse(new Options().arguments);
298 if (configurations != null && configurations.length > 0) { 298 if (configurations != null && configurations.length > 0) {
299 testConfigurations(configurations); 299 testConfigurations(configurations);
300 } 300 }
301 }); 301 });
302 } 302 }
303 303
OLDNEW
« no previous file with comments | « no previous file | tools/testing/dart/browser_controller.dart » ('j') | tools/testing/dart/browser_controller.dart » ('J')

Powered by Google App Engine
This is Rietveld 408576698