Support Package Resolution Configuration files.

Support Package Resolution Configuration files.

BUG=http://dartbug.com/23371
R=johnniwinther@google.com

Committed: https://github.com/dart-lang/sdk/commit/36c29d05bcc1dfe43a2f95698df5b41857f93221

Committed: https://github.com/dart-lang/sdk/commit/63681d5b59a26d00233ee582e8f9b9fa63c0bd80

[Harry Terkelsen, 2015-06-02 21:43:58]

het@google.com changed reviewers:
+ floitsch@google.com, johnniwinther@google.com, sra@chromium.org

[Harry Terkelsen, 2015-06-03 00:08:27]

Some tests are failing because dart2js now depends on package_config, which
depends on http, which depends on mirrors. Do you think we should change
package_config to avoid the http dependency, or should we just whitelist the
warning since removing the mirrors dependency is already a goal for http?

[Johnni Winther, 2015-06-03 11:36:39]

LGTM but let's discuss dart:mirror dependency tomorrow over VC.

https://codereview.chromium.org/1162363004/diff/40001/pkg/compiler/lib/src/ap...
File pkg/compiler/lib/src/apiimpl.dart (right):

https://codereview.chromium.org/1162363004/diff/40001/pkg/compiler/lib/src/ap...
pkg/compiler/lib/src/apiimpl.dart:351: Future setPackages(Uri uri) async {
Rename to 'setupPackages'.

https://codereview.chromium.org/1162363004/diff/40001/pkg/compiler/lib/src/ap...
pkg/compiler/lib/src/apiimpl.dart:362: new
MapPackages(pkgs.parse(packageConfigContents, packageConfig));
Indent by 4.

[Harry Terkelsen, 2015-06-04 22:05:30]

https://codereview.chromium.org/1162363004/diff/40001/pkg/compiler/lib/src/ap...
File pkg/compiler/lib/src/apiimpl.dart (right):

https://codereview.chromium.org/1162363004/diff/40001/pkg/compiler/lib/src/ap...
pkg/compiler/lib/src/apiimpl.dart:351: Future setPackages(Uri uri) async {
On 2015/06/03 11:36:39, Johnni Winther wrote:
> Rename to 'setupPackages'.

Done.

https://codereview.chromium.org/1162363004/diff/40001/pkg/compiler/lib/src/ap...
pkg/compiler/lib/src/apiimpl.dart:362: new
MapPackages(pkgs.parse(packageConfigContents, packageConfig));
On 2015/06/03 11:36:39, Johnni Winther wrote:
> Indent by 4.

Done.

[Harry Terkelsen, 2015-06-04 22:15:15]

Committed patchset #6 (id:100001) manually as
36c29d05bcc1dfe43a2f95698df5b41857f93221 (presubmit successful).

[Harry Terkelsen, 2015-06-08 22:16:55]

Had to roll back because of broken tests. PTAL

[Johnni Winther, 2015-06-09 07:47:43]

lgtm

[Harry Terkelsen, 2015-06-09 18:07:16]

Committed patchset #10 (id:180001) manually as
63681d5b59a26d00233ee582e8f9b9fa63c0bd80 (presubmit successful).