dart2js: add an example script that shows the distribution of code among

dart2js: add an example script that shows the distribution of code among
libraries.

For example, running this on the dump-info of angular2 'hello
world' would produce:

  path             8807   1.21%
  stack_trace     11501   1.59%
  intl            14135   1.95%
  Loose files    142311  19.62%
  angular2       224171  30.90%
  Core libs      225120  31.03%
  All packages   258614  35.65%
  TOTAL          626045  86.30%

Note that the grouping of libraries, say by package, and summary information is
configurable.

R=het@google.com

Committed: https://github.com/dart-lang/sdk/commit/571c1d5b6e8990d07c0f93aad924454ef8e4bd4b

[Siggi Cherem (dart-lang), 2015-08-11 01:10:15]

Patchset #1 (id:1) has been deleted

[Siggi Cherem (dart-lang), 2015-08-11 01:11:18]

sigmund@google.com changed reviewers:
+ het@google.com, johnniwinther@google.com, sra@google.com

[Harry Terkelsen, 2015-08-11 01:43:40]

lgtm

https://codereview.chromium.org/1287543002/diff/20001/pkg/compiler/tool/libra...
File pkg/compiler/tool/library_size_split.dart (right):

https://codereview.chromium.org/1287543002/diff/20001/pkg/compiler/tool/libra...
pkg/compiler/tool/library_size_split.dart:33: var name = group['name'];
so you can only omit the name for one group?

https://codereview.chromium.org/1287543002/diff/20001/pkg/compiler/tool/libra...
pkg/compiler/tool/library_size_split.dart:70: /// name/regexp pairs. If the name
is omitted, it is assume to be group(1) of
or group(0) if there is only 1 group

[Siggi Cherem (dart-lang), 2015-08-11 21:01:10]

Patchset #2 (id:40001) has been deleted

[Siggi Cherem (dart-lang), 2015-08-11 21:08:00]

thanks!

https://codereview.chromium.org/1287543002/diff/20001/pkg/compiler/tool/libra...
File pkg/compiler/tool/library_size_split.dart (right):

https://codereview.chromium.org/1287543002/diff/20001/pkg/compiler/tool/libra...
pkg/compiler/tool/library_size_split.dart:33: var name = group['name'];
On 2015/08/11 01:43:39, Harry Terkelsen wrote:
> so you can only omit the name for one group?

Good catch, that was not intentional. I changed this a bit to create a
specification of a group and allowing multiple unnamed groups.

The change also made it easy to add support for clustering groups (something I
wanted to add anyways).

Now that I have that I changed the default grouping to also include individual
libraries. Here is a portion of what the new output for the defaultGrouping in
angular2 helloworld now looks like:



 ...
 package:angular2/src/render/dom/dom_renderer.ng_deps.dart         307   0.06%
 package:angular2/src/core/compiler/view_resolver.dart             310   0.06%
 ...
 dart:html_common                                                 3176   0.57%
 package:angular2/src/core/compiler/view.dart                     3183   0.57%
 package:angular2/src/render/dom/compiler/directive_parser.dart   3417   0.61%
 package:path/src/context.dart                                    3662   0.66%
 ...
 dart:js                                                          5048   0.91%
 ...
 package:intl/intl.dart                                          13504   2.42%
 ...
 dart:async                                                      36312   6.52%
 ----------------------------------------------------------------------------
 path                                                             8800   1.58%
 stack_trace                                                     11481   2.06%
 intl                                                            14126   2.54%
 angular2                                                       220724  39.62%
 -----------------------------------------------------------------------------
 Loose files                                                      2266   0.41%
 Core libs                                                      217634  39.07%
 All packages                                                   255131  45.80%
 -----------------------------------------------------------------------------
 Total (excludes preambles, statics & consts)                   475031  85.27%
 Program Size                                                   557099    100%

https://codereview.chromium.org/1287543002/diff/20001/pkg/compiler/tool/libra...
pkg/compiler/tool/library_size_split.dart:70: /// name/regexp pairs. If the name
is omitted, it is assume to be group(1) of
On 2015/08/11 01:43:39, Harry Terkelsen wrote:
> or group(0) if there is only 1 group

Done.

[Siggi Cherem (dart-lang), 2015-08-11 21:41:13]

PTAL - I forgot to say that since I added a bit of code, you might want to take
another look before I submit.

[Siggi Cherem (dart-lang), 2015-08-11 21:41:24]

Patchset #2 (id:60001) has been deleted

[Harry Terkelsen, 2015-08-11 21:54:20]

lgtm

[Siggi Cherem (dart-lang), 2015-08-11 22:00:04]

Committed patchset #2 (id:80001) manually as
571c1d5b6e8990d07c0f93aad924454ef8e4bd4b (presubmit successful).