Bugfix in multitest.dart: Make sure we copy imported files to the generated_tests directory

Bugfix in multitest.dart: Make sure we copy imported files to the generated_tests directory

The regular expression was no longer valid after import statements have changed.
Furthermore the guard against cyclic imports was not working because of missing
hashCode/operator== on the Path class.

R=ricow@google.com

Committed: https://code.google.com/p/dart/source/detail?r=25155

[ricow1, 2013-07-18 17:07:44]

LGTM,
I would consider just calling it foundImports as before

https://codereview.chromium.org/19774003/diff/1/tools/testing/dart/multitest....
File tools/testing/dart/multitest.dart (right):

https://codereview.chromium.org/19774003/diff/1/tools/testing/dart/multitest....
tools/testing/dart/multitest.dart:155: Set<String> foundImportsAsString = new
Set<String>();
any reason to call this AsString other than to make clear what is going on in
this specific cl?

[kustermann, 2013-07-18 17:11:27]

https://codereview.chromium.org/19774003/diff/1/tools/testing/dart/multitest....
File tools/testing/dart/multitest.dart (right):

https://codereview.chromium.org/19774003/diff/1/tools/testing/dart/multitest....
tools/testing/dart/multitest.dart:155: Set<String> foundImportsAsString = new
Set<String>();
On 2013/07/18 17:07:44, ricow1 wrote:
> any reason to call this AsString other than to make clear what is going on in
> this specific cl?

No. My first version had two sets. Then I removed the old one. 

Done.

[kustermann, 2013-07-18 17:11:50]

Committed patchset #2 manually as r25155 (presubmit successful).