Improve how we handle packages/ HTML imports. This resolves better any valid

tools/testing/dart/test_suite.dart

Index: tools/testing/dart/test_suite.dart
diff --git a/tools/testing/dart/test_suite.dart b/tools/testing/dart/test_suite.dart
index 1df99ad1f4f660ac5557cad2f632e6ef0381ef30..1c69d0e552bba231ca29a5a8967ddf4ec6c6b730 100644
--- a/tools/testing/dart/test_suite.dart
+++ b/tools/testing/dart/test_suite.dart
@@ -1121,7 +1121,20 @@ class StandardTestSuite extends TestSuite {
             commands.add(_polymerDeployCommand(
                 customHtmlPath, tempDir, optionsFromFile));
 
-            htmlPath = '$tempDir/test/$nameNoExt.html';
+            // Find package's home dir, where pubspec.yaml is located. The
+            // relative directory within the package will be part of the
+            // entrypoint path.
+            var homeDir = filePath;
+            var pubspec = '';
+            do {
+              homeDir = homeDir.directoryPath;
+              if (homeDir.toString() == '/') {
+                homeDir = dir;
+                break;
+              }
+              pubspec = homeDir.append('pubspec.yaml').toNativePath();
+            } while (!new File(pubspec).existsSync());

[kustermann, 2014/01/29 20:48:40]

There is already logic to do something similar. Please update your local working
copy. 
See here:
https://code.google.com/p/dart/source/browse/branches/bleeding_edge/dart/tool...

[Siggi Cherem (dart-lang), 2014/01/29 22:15:07]

Thanks. All set.

+            htmlPath = '$tempDir/${dir.relativeTo(homeDir)}/$nameNoExt.html';

[kustermann, 2014/01/29 20:48:40]

I don't understand what the purpose of this is. The enqueuing phase will look
recursively for tests (i.e. not  just the files inside pkg/X/test/).

The only thing you change here is "htmlPath". Does the polymer build/deploy
command put the files in a different directory now?

Could you explain why this is necessary?

[Siggi Cherem (dart-lang), 2014/01/29 22:15:07]

Yes and no - it always supported putting the output file in different
directories, but we haven't written tests that exposed this until now.
The polymer-build command works similar to pub-build: for a given file it finds
where the package directory lives (finding the pubspec file), and it generates
output that maintains the layout of original package.

There are 2 packages in the repo containing polymer tests. Most tests are in the
polymer package under "pkg/polymer/test/", but we have some other tests under a
test-only package called 'canonicalization'. This package is currently under
"pkg/polymer/example/canonicalization/" (it is practically a nested package, we
could also move that somewhere else if it makes things cleaner).

Up until now, all of tests were placed directly under the 'test/' folder of
their respective packages, so that made every test be OK with the output
described as: "$tmpDir/test/"

In this CL I had to add a test that lives deeper inside the test folder of the
canonicalization package: "test/dir/dev_test", so the changes in this file are
needed to be able to find the generated output (in this case under
"$tmpDir/test/dir/").

             dartWrapperFilename = '${htmlPath}_bootstrap.dart';
             compiledDartWrapperFilename = '$dartWrapperFilename.js';
           } else {