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

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

Issue 2768493002: Update name of content-shell flag --run-layout-test in test scripts (Closed)
Patch Set: Created 3 years, 9 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
« no previous file with comments | « no previous file | no next file » | 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 /** 5 /**
6 * Classes and methods for enumerating and preparing tests. 6 * Classes and methods for enumerating and preparing tests.
7 * 7 *
8 * This library includes: 8 * This library includes:
9 * 9 *
10 * - Creating tests by listing all the Dart files in certain directories, 10 * - Creating tests by listing all the Dart files in certain directories,
(...skipping 1264 matching lines...) Expand 10 before | Expand all | Expand 10 after
1275 1275
1276 var htmlPath_subtest = _createUrlPathFromFile(new Path(htmlPath)); 1276 var htmlPath_subtest = _createUrlPathFromFile(new Path(htmlPath));
1277 var fullHtmlPath = 1277 var fullHtmlPath =
1278 _getUriForBrowserTest(htmlPath_subtest, subtestName).toString(); 1278 _getUriForBrowserTest(htmlPath_subtest, subtestName).toString();
1279 1279
1280 if (runtime == "drt") { 1280 if (runtime == "drt") {
1281 var dartFlags = []; 1281 var dartFlags = [];
1282 var contentShellOptions = []; 1282 var contentShellOptions = [];
1283 1283
1284 contentShellOptions.add('--no-timeout'); 1284 contentShellOptions.add('--no-timeout');
1285 contentShellOptions.add('--dump-render-tree'); 1285 contentShellOptions.add('--run-layout-test');
1286 1286
1287 // Disable the GPU under Linux and Dartium. If the GPU is enabled, 1287 // Disable the GPU under Linux and Dartium. If the GPU is enabled,
1288 // Chrome may send a termination signal to a test. The test will be 1288 // Chrome may send a termination signal to a test. The test will be
1289 // terminated if a machine (bot) doesn't have a GPU or if a test is 1289 // terminated if a machine (bot) doesn't have a GPU or if a test is
1290 // still running after a certain period of time. 1290 // still running after a certain period of time.
1291 if (configuration['system'] == 'linux' && 1291 if (configuration['system'] == 'linux' &&
1292 configuration['runtime'] == 'drt') { 1292 configuration['runtime'] == 'drt') {
1293 contentShellOptions.add('--disable-gpu'); 1293 contentShellOptions.add('--disable-gpu');
1294 } 1294 }
1295 if (compiler == 'none') { 1295 if (compiler == 'none') {
(...skipping 1124 matching lines...) Expand 10 before | Expand all | Expand 10 after
2420 } 2420 }
2421 if (path.length > WINDOWS_SHORTEN_PATH_LIMIT) { 2421 if (path.length > WINDOWS_SHORTEN_PATH_LIMIT) {
2422 ++shortNameCounter; 2422 ++shortNameCounter;
2423 var pathEnd = path.substring(path.length - WINDOWS_PATH_END_LENGTH); 2423 var pathEnd = path.substring(path.length - WINDOWS_PATH_END_LENGTH);
2424 path = "short${shortNameCounter}_$pathEnd"; 2424 path = "short${shortNameCounter}_$pathEnd";
2425 } 2425 }
2426 } 2426 }
2427 return path; 2427 return path;
2428 } 2428 }
2429 } 2429 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698