Chromium Code Reviews| Index: tools/testing/dart/runtime_configuration.dart |
| diff --git a/tools/testing/dart/runtime_configuration.dart b/tools/testing/dart/runtime_configuration.dart |
| index d0b90f031a47e409f34b527ceaca7a55df16b1c1..1b8b9092a7a3ca1cd01134dece3023b443dbc983 100644 |
| --- a/tools/testing/dart/runtime_configuration.dart |
| +++ b/tools/testing/dart/runtime_configuration.dart |
| @@ -237,9 +237,17 @@ class StandaloneDartRuntimeConfiguration extends DartVmRuntimeConfiguration { |
| throw "Dart VM cannot run files of type '$type'."; |
| } |
| String executable = suite.dartVmBinaryFileName; |
| - return <Command>[ |
| - commandBuilder.getVmCommand(executable, arguments, environmentOverrides, needsDFERunner: needsDFERunner) |
| - ]; |
| + List commands = new List(); |
|
Bill Hesse
2017/02/01 02:07:44
var commands = new List<Command>();
or
var command
rmacnak
2017/02/01 17:21:11
Done.
|
| + commands.add(commandBuilder.getVmCommand(executable, |
| + arguments, |
| + environmentOverrides, |
| + needsDFERunner: needsDFERunner)); |
| + if (type == 'application/dart-snapshot') { |
| + // Delete snapshots as we go to reduce the space required to run the test |
| + // suite. |
| + commands.add(commandBuilder.getDeleteCommand(script)); |
| + } |
| + return commands; |
| } |
| } |
| @@ -282,10 +290,13 @@ class DartPrecompiledRuntimeConfiguration extends DartVmRuntimeConfiguration { |
| throw "dart_precompiled cannot run files of type '$type'."; |
| } |
| - return <Command>[ |
| - commandBuilder.getVmCommand(suite.dartPrecompiledBinaryFileName, |
| - arguments, environmentOverrides) |
| - ]; |
| + List commands = new List(); |
| + commands.add(commandBuilder.getVmCommand(suite.dartPrecompiledBinaryFileName, |
| + arguments, environmentOverrides)); |
| + // Delete snapshots (dylibs or blobs) as we go to reduce the space required |
| + // to run the test suite. |
| + commands.add(commandBuilder.getDeleteCommand(script)); |
| + return commands; |
| } |
| } |