- Properly return null if an unknown package is resolved.

- Properly return null if an unknown package is resolved.
- Allow short form package:foo imports.

R=asiva@google.com

Committed: https://github.com/dart-lang/sdk/commit/3b2a695d71057f3a538f79f48a96f6bdca2474bb

[Ivan Posva, 2016-03-21 23:09:22]

iposva@google.com changed reviewers:
+ asiva@google.com

[siva, 2016-03-21 23:24:26]

lgtm

https://codereview.chromium.org/1822863002/diff/1/tests/isolate/scenarios/bad...
File tests/isolate/scenarios/bad_resolve_package/bad_resolve_package_test.dart
(right):

https://codereview.chromium.org/1822863002/diff/1/tests/isolate/scenarios/bad...
tests/isolate/scenarios/bad_resolve_package/bad_resolve_package_test.dart:23: //
Excpecting a null resolution for inexistent package mapping.
Expecting

[Ivan Posva, 2016-03-22 16:02:09]

Description was changed from

==========
- Properly return null if an unknown package is resolved.
- Allow short form package:foo imports.
==========

to

==========
- Properly return null if an unknown package is resolved.
- Allow short form package:foo imports.

R=asiva@google.com

Committed:
https://github.com/dart-lang/sdk/commit/3b2a695d71057f3a538f79f48a96f6bdca2474bb
==========

[Ivan Posva, 2016-03-22 16:02:10]

Committed patchset #2 (id:20001) manually as
3b2a695d71057f3a538f79f48a96f6bdca2474bb (presubmit successful).

[Lasse Reichstein Nielsen, 2016-03-23 11:18:18]

lrn@google.com changed reviewers:
+ lrn@google.com

[Lasse Reichstein Nielsen, 2016-03-23 11:18:18]

https://codereview.chromium.org/1822863002/diff/20001/runtime/bin/builtin.dart
File runtime/bin/builtin.dart (right):

https://codereview.chromium.org/1822863002/diff/20001/runtime/bin/builtin.dar...
runtime/bin/builtin.dart:305: path = "$packageName.dart";
While I like this change in general, it should be supported by all other
implementations as well, otherwise you can have code that works in the VM but
not in dart2js. Is that planned as well?

Otherwise, how about just resolving the path to "/"?