Add support for importing using "package:" to the standalone VM.

Add support for importing using "package:" to the standalone VM.

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

[Bob Nystrom, 2012-04-09 23:32:42]

First pass at this. Let me know if there's anything you'd like me to do
differently.

[nweiz, 2012-04-10 00:22:28]

lgtm

https://chromiumcodereview.appspot.com/10034001/diff/1/tests/standalone/src/p...
File tests/standalone/src/package/PackageTest.dart (right):

https://chromiumcodereview.appspot.com/10034001/diff/1/tests/standalone/src/p...
tests/standalone/src/package/PackageTest.dart:4: #library('package_test');
Nit: Empty line between license comment and #library.

[Bob Nystrom, 2012-04-10 00:57:47]

Changed to just use "packages" and not "entrypoint.packages" as the directory
name as per our discussion. PTAL!

https://chromiumcodereview.appspot.com/10034001/diff/1/tests/standalone/src/p...
File tests/standalone/src/package/PackageTest.dart (right):

https://chromiumcodereview.appspot.com/10034001/diff/1/tests/standalone/src/p...
tests/standalone/src/package/PackageTest.dart:4: #library('package_test');
On 2012/04/10 00:22:28, nweiz wrote:
> Nit: Empty line between license comment and #library.

Done.

[nweiz, 2012-04-10 01:00:12]

lgtm

[Ivan Posva, 2012-04-11 12:31:24]

LGTM -ip

https://chromiumcodereview.appspot.com/10034001/diff/7005/runtime/bin/builtin...
File runtime/bin/builtin.dart (right):

https://chromiumcodereview.appspot.com/10034001/diff/7005/runtime/bin/builtin...
runtime/bin/builtin.dart:105: var path = uri.path;
Maybe we should check that uri.domain is null in this case.

https://chromiumcodereview.appspot.com/10034001/diff/7005/runtime/bin/builtin...
runtime/bin/builtin.dart:116: var path =
_entrypoint.resolve('packages/${uri.path}').path;
ditto: assert(uri.domain == null)

[Bob Nystrom, 2012-04-11 19:39:02]

Thanks! Added some testing for incorrect URLs that have domains (and a negative
test to ensure that works right, at least for package:).

https://chromiumcodereview.appspot.com/10034001/diff/7005/runtime/bin/builtin...
File runtime/bin/builtin.dart (right):

https://chromiumcodereview.appspot.com/10034001/diff/7005/runtime/bin/builtin...
runtime/bin/builtin.dart:105: var path = uri.path;
On 2012/04/11 12:31:24, Ivan Posva wrote:
> Maybe we should check that uri.domain is null in this case.

Done.

https://chromiumcodereview.appspot.com/10034001/diff/7005/runtime/bin/builtin...
runtime/bin/builtin.dart:116: var path =
_entrypoint.resolve('packages/${uri.path}').path;
On 2012/04/11 12:31:24, Ivan Posva wrote:
> ditto: assert(uri.domain == null)

Done.

[srdjan, 2012-04-11 19:48:12]

DBC

https://chromiumcodereview.appspot.com/10034001/diff/11009/runtime/bin/builti...
File runtime/bin/builtin.dart (right):

https://chromiumcodereview.appspot.com/10034001/diff/11009/runtime/bin/builti...
runtime/bin/builtin.dart:129: var path =
_entrypoint.resolve('packages/${uri.path}').path;
This probably does not work on windows....

[Bob Nystrom, 2012-04-11 20:28:04]

https://chromiumcodereview.appspot.com/10034001/diff/11009/runtime/bin/builti...
File runtime/bin/builtin.dart (right):

https://chromiumcodereview.appspot.com/10034001/diff/11009/runtime/bin/builti...
runtime/bin/builtin.dart:129: var path =
_entrypoint.resolve('packages/${uri.path}').path;
On 2012/04/11 19:48:13, srdjan wrote:
> This probably does not work on windows....

I think that's OK, but I'm not sure. I don't see any of the existing code
avoiding "/" on Windows here, so I assume that is handled further down.

[Ivan Posva, 2012-04-11 20:47:58]

https://chromiumcodereview.appspot.com/10034001/diff/11009/runtime/bin/builti...
File runtime/bin/builtin.dart (right):

https://chromiumcodereview.appspot.com/10034001/diff/11009/runtime/bin/builti...
runtime/bin/builtin.dart:129: var path =
_entrypoint.resolve('packages/${uri.path}').path;
On 2012/04/11 20:28:04, Bob Nystrom wrote:
> On 2012/04/11 19:48:13, srdjan wrote:
> > This probably does not work on windows....
> 
> I think that's OK, but I'm not sure. I don't see any of the existing code
> avoiding "/" on Windows here, so I assume that is handled further down.

As far as I can tell URIs contain forward slashes even on windows.