generate an html report of the compilation results

generate an html report of the compilation results
cleanup css

BUG=
R=vsm@google.com

Committed: https://github.com/dart-lang/dev_compiler/commit/17e1ebe546a9c7c05edf39c3e76c3bc5bebd7486

[devoncarew, 2015-10-27 18:29:44]

devoncarew@google.com changed reviewers:
+ jmesserly@google.com, leafp@google.com, vsm@google.com

[devoncarew, 2015-10-27 18:29:44]

Here's the patch to generate an html report of the ddc errors, warnings (and any
infos it may print out). Some sample output:
http://dart-atom.github.io/dartlang/entry_dart_results.html

I've found it useful when strong-mode-ifying apps. It's really nice to be able
to see the issues sorted both by severity and by package.

[vsm, 2015-10-27 20:04:55]

lgtm!

https://codereview.chromium.org/1419973003/diff/1/lib/src/compiler.dart
File lib/src/compiler.dart (right):

https://codereview.chromium.org/1419973003/diff/1/lib/src/compiler.dart#newco...
lib/src/compiler.dart:72: (reporter as HtmlReporter).finish(options);
You might not need the cast here if type promotion is working - I think it
should here.

https://codereview.chromium.org/1419973003/diff/1/lib/src/report/html_gen.dart
File lib/src/report/html_gen.dart (right):

https://codereview.chromium.org/1419973003/diff/1/lib/src/report/html_gen.dar...
lib/src/report/html_gen.dart:9: final StringBuffer _buffer = new StringBuffer();
We've generally been omitting the type when there is an initializer expression. 
You'd still need to keep any generics though ... e.g., final _tags = <String>[];

https://codereview.chromium.org/1419973003/diff/1/lib/src/report/html_gen.dar...
lib/src/report/html_gen.dart:38: '<meta name="viewport"
content="width=device-width, initial-scale=1.0">');
nit: line len

https://codereview.chromium.org/1419973003/diff/1/lib/src/report/html_reporte...
File lib/src/report/html_reporter.dart (right):

https://codereview.chromium.org/1419973003/diff/1/lib/src/report/html_reporte...
lib/src/report/html_reporter.dart:37: // TOOD: This can fail when extracting
context spans.
s/TOOD/TODO/

[devoncarew, 2015-10-27 20:32:35]

Thanks! Updated.

https://codereview.chromium.org/1419973003/diff/1/lib/src/compiler.dart
File lib/src/compiler.dart (right):

https://codereview.chromium.org/1419973003/diff/1/lib/src/compiler.dart#newco...
lib/src/compiler.dart:72: (reporter as HtmlReporter).finish(options);
On 2015/10/27 20:04:55, vsm wrote:
> You might not need the cast here if type promotion is working - I think it
> should here.

Done.

https://codereview.chromium.org/1419973003/diff/1/lib/src/report/html_gen.dart
File lib/src/report/html_gen.dart (right):

https://codereview.chromium.org/1419973003/diff/1/lib/src/report/html_gen.dar...
lib/src/report/html_gen.dart:9: final StringBuffer _buffer = new StringBuffer();
On 2015/10/27 20:04:55, vsm wrote:
> We've generally been omitting the type when there is an initializer
expression. 
> You'd still need to keep any generics though ... e.g., final _tags =
<String>[];

Done.

https://codereview.chromium.org/1419973003/diff/1/lib/src/report/html_gen.dar...
lib/src/report/html_gen.dart:38: '<meta name="viewport"
content="width=device-width, initial-scale=1.0">');
On 2015/10/27 20:04:55, vsm wrote:
> nit: line len

Done.

https://codereview.chromium.org/1419973003/diff/1/lib/src/report/html_reporte...
File lib/src/report/html_reporter.dart (right):

https://codereview.chromium.org/1419973003/diff/1/lib/src/report/html_reporte...
lib/src/report/html_reporter.dart:37: // TOOD: This can fail when extracting
context spans.
On 2015/10/27 20:04:55, vsm wrote:
> s/TOOD/TODO/

Done.

[vsm, 2015-10-28 12:41:38]

lgtm

[devoncarew, 2015-10-28 16:57:42]

Committed patchset #2 (id:20001) manually as
17e1ebe546a9c7c05edf39c3e76c3bc5bebd7486 (presubmit successful).