add LibraryIdentifier

add LibraryIdentifier

R=sigmund@google.com

Committed: https://github.com/dart-lang/static_init/commit/126674209b13b463555acaf1960998f7b8251dea

[jakemac, 2015-01-26 23:55:10]

jakemac@google.com changed reviewers:
+ sigmund@google.com

[jakemac, 2015-01-26 23:55:10]

This is to support @HtmlImport path normalization in both mirror and static
modes

[jakemac, 2015-01-27 17:42:27]

Note: An alternative to this would be to force absolute package urls for
everything. This also has the side effect of enforcing that only entry point
files live in web, which is actually a good thing IMO.

[jakemac, 2015-01-27 17:47:55]

On 2015/01/27 17:42:27, jakemac wrote:
> Note: An alternative to this would be to force absolute package urls for
> everything. This also has the side effect of enforcing that only entry point
> files live in web, which is actually a good thing IMO.

Sorry not only one entry point, but only entry points in general.

[Siggi Cherem (dart-lang), 2015-01-27 20:26:29]

lgtm, just minor nits below

https://codereview.chromium.org/880713002/diff/1/initialize_test_fixtures/.gi...
File initialize_test_fixtures/.gitignore (right):

https://codereview.chromium.org/880713002/diff/1/initialize_test_fixtures/.gi...
initialize_test_fixtures/.gitignore:1: # Pub files.
for this contained package we can delete the .gitignore, .status, CHANGELOG.md,
LICENSE, and possibly README.md

https://codereview.chromium.org/880713002/diff/1/initialize_test_fixtures/pub...
File initialize_test_fixtures/pubspec.yaml (right):

https://codereview.chromium.org/880713002/diff/1/initialize_test_fixtures/pub...
initialize_test_fixtures/pubspec.yaml:1: name: initialize_test_fixtures
after our chat, let's rename this to `test_package`

https://codereview.chromium.org/880713002/diff/1/initialize_test_fixtures/pub...
initialize_test_fixtures/pubspec.yaml:2: version: 0.1.0
if it happens to work without a version, that would be nice to prevent us from
publishing it :)

https://codereview.chromium.org/880713002/diff/1/initialize_test_fixtures/pub...
initialize_test_fixtures/pubspec.yaml:3: author: Polymer.dart Authors
<web@dartlang.org>
I think it's ok to remove all entries except the dependencies here and below :)

[jakemac, 2015-01-28 17:52:31]

https://codereview.chromium.org/880713002/diff/1/initialize_test_fixtures/.gi...
File initialize_test_fixtures/.gitignore (right):

https://codereview.chromium.org/880713002/diff/1/initialize_test_fixtures/.gi...
initialize_test_fixtures/.gitignore:1: # Pub files.
On 2015/01/27 20:26:28, Siggi Cherem (dart-lang) wrote:
> for this contained package we can delete the .gitignore, .status,
CHANGELOG.md,
> LICENSE, and possibly README.md

Done.

https://codereview.chromium.org/880713002/diff/1/initialize_test_fixtures/pub...
File initialize_test_fixtures/pubspec.yaml (right):

https://codereview.chromium.org/880713002/diff/1/initialize_test_fixtures/pub...
initialize_test_fixtures/pubspec.yaml:1: name: initialize_test_fixtures
On 2015/01/27 20:26:29, Siggi Cherem (dart-lang) wrote:
> after our chat, let's rename this to `test_package`

Done.

https://codereview.chromium.org/880713002/diff/1/initialize_test_fixtures/pub...
initialize_test_fixtures/pubspec.yaml:2: version: 0.1.0
On 2015/01/27 20:26:29, Siggi Cherem (dart-lang) wrote:
> if it happens to work without a version, that would be nice to prevent us from
> publishing it :)

Done.

https://codereview.chromium.org/880713002/diff/1/initialize_test_fixtures/pub...
initialize_test_fixtures/pubspec.yaml:3: author: Polymer.dart Authors
<web@dartlang.org>
On 2015/01/27 20:26:29, Siggi Cherem (dart-lang) wrote:
> I think it's ok to remove all entries except the dependencies here and below
:)

Done.

[jakemac, 2015-01-28 17:53:05]

Committed patchset #2 (id:20001) manually as
126674209b13b463555acaf1960998f7b8251dea (presubmit successful).