Fix test.dart on dartc client compilation tests.

tools/testing/dart/test_suite.dart

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #library("test_suite");
 
 #import("dart:io");
 #import("status_file_parser.dart");
 #import("test_runner.dart");
 #import("multitest.dart");
@@ skipping 236 matching lines @@
       start = filename.lastIndexOf('/');
       int middle = filename.lastIndexOf('_');
       testName = filename.substring(start + 1, middle) + '/' +
           filename.substring(middle + 1, filename.length - 5);
     } else {
       // This case is hit by the dartc client compilation
       // tests. These tests are pretty broken compared to the
       // 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);
         }
       }
     }
     Set<String> expectations = testExpectations.expectations(testName);
     if (configuration['report']) {
       // Tests with multiple VMOptions are counted more than once.
       for (var dummy in optionsFromFile["vmOptions"]) {
@@ skipping 782 matching lines @@
  * $noCrash tests are expected to be flaky but not crash
  * $pass tests are expected to pass
  * $failOk tests are expected to fail that we won't fix
  * $fail tests are expected to fail that we should fix
  * $crash tests are expected to crash that we should fix
  * $timeout tests are allowed to timeout
 """;
     print(report);
    }
 }