Ship with a reasonable MirrorUsed for angularDart

[mhevery]

No description provided.

[sethladd]

/sub

[jbdeboer]

Work involved:
-> add our "best known" list of mirrors near ngBootstrap.
-> verify that new directives can be added without needed to update the list
-> document how and when apps need to provide additional MirrorsUsed.
-> verify that apps which provide their own list of mirrors don't break

[sethladd]

See also https://code.google.com/p/dart/issues/detail?id=14686

[shailen]

Kasper Lund landed a commit (https://codereview.chromium.org//137953002) that added the MirrorsUsed declaration to the todo sample in the dart repo. See:

https://code.google.com/p/dart/source/browse/branches/bleeding_edge/dart/samples/third_party/angular_todo/main.dart

This can serve as a useful guide for what Angular should automatically ship with so there is no need for every app to add Angular-specific MirrorsUsed metadata.

[mhevery]

very nice

[mhevery]

@jbdeboer do you have numbers as to how big hello world app is now?

[ufoscout]

@mhevery the current size of angular hello world app is around 800kb. It is not so small :(
Are there additional optimisation of this aspect planned? Am I wrong or I read somewhere that there are plans to not use mirrors at all in a future version?

[TedSander]

There are Transformers now which remove the need for mirrors used. It creates static code for all the locations that used Mirrors. See: https://github.com/angular/angular.dart/blob/master/lib/transformer.dart for the transformers.

This lowers the code size, and speeds up the application.

[mhevery]

@ufoscout The current hello world size after gzip is around 200kb. I don't think we can get it much smaller.

https://github.com/angular/angular.dart/blob/master/scripts/travis/build.sh#L47