Chromium Code Reviews| Index: tools/testing/dart/test_suite.dart |
| diff --git a/tools/testing/dart/test_suite.dart b/tools/testing/dart/test_suite.dart |
| index 8147f612cd51dcc123bac94128c1af1d79c9e6d5..b247c77f7860087f418a784825bd86155bf93bd2 100644 |
| --- a/tools/testing/dart/test_suite.dart |
| +++ b/tools/testing/dart/test_suite.dart |
| @@ -254,13 +254,14 @@ class StandardTestSuite implements TestSuite { |
| // rest. They use the .dart suffix in the status files. They |
| // find tests in weird ways (testing that they contain "#"). |
| // They need to be redone. |
| - // directoryPath may start with '../'. |
| // TODO(1058): This does not work on Windows. |
|
Mads Ager (google)
2012/02/01 12:57:46
Is this TODO obsolete?
|
| - String sanitizedDirectoryPath = (directoryPath.startsWith('../')) ? |
| - directoryPath.substring(3) : directoryPath; |
| - start = filename.indexOf(sanitizedDirectoryPath); |
| - testName = filename.substring(start + sanitizedDirectoryPath.length + 1, |
| - filename.length); |
| + start = filename.indexOf(directoryPath); |
| + if (start != -1) { |
| + testName = filename.substring(start + directoryPath.length + 1); |
| + } else { |
| + testName = filename; |
| + } |
| + |
| if (configuration['component'] != 'dartc') { |
| if (testName.endsWith('.dart')) { |
| testName = testName.substring(0, testName.length - 5); |