More burndown list work

A huge amount of polish and refactoring of the viewer code.
Please ignore the .json files.

See
http://jacobr-latest.dartlang-api.appspot.com/

for an example of the running app.
My apologies for the large size of this CL.

[sigmund, 2013-02-13 22:08:35]

passing by comments...

https://codereview.appspot.com/7307109/diff/1/client/pubspec.yaml
File client/pubspec.yaml (right):

https://codereview.appspot.com/7307109/diff/1/client/pubspec.yaml#newcode14
client/pubspec.yaml:14: widget: any
do you need any of these 3 packages? I didn't find where you use them.

https://codereview.appspot.com/7307109/diff/1/client/web/details.html
File client/web/details.html (right):

https://codereview.appspot.com/7307109/diff/1/client/web/details.html#newcode24
client/web/details.html:24: // TODO(jacobr): we shouldn't have to specify a
class definition for no-op
this shoudn't be needed, if you remove the script tag entirely, we should be
generating this class for you

[jacobr, 2013-02-14 00:56:23]

Fixed. Uploaded new version to the alternate version of the CL with an easier to
read diff of ast.dart, etc.

https://codereview.appspot.com/7307109/diff/1/client/pubspec.yaml
File client/pubspec.yaml (right):

https://codereview.appspot.com/7307109/diff/1/client/pubspec.yaml#newcode14
client/pubspec.yaml:14: widget: any
On 2013/02/13 22:08:35, sigmund wrote:
> do you need any of these 3 packages? I didn't find where you use them.

I use poppy.  Removed the packages I don't use.

https://codereview.appspot.com/7307109/diff/1/client/web/details.html
File client/web/details.html (right):

https://codereview.appspot.com/7307109/diff/1/client/web/details.html#newcode24
client/web/details.html:24: // TODO(jacobr): we shouldn't have to specify a
class definition for no-op
On 2013/02/13 22:08:35, sigmund wrote:
> this shoudn't be needed, if you remove the script tag entirely, we should be
> generating this class for you
Removed

[John Messerly, 2013-02-14 00:58:42]

hmmm I still only see patch set #1

[sigmund, 2013-02-14 01:01:17]

I think Jacob is referring to this other CL:

https://codereview.appspot.com/7301094

the comment below was a comment I had before I switched to start reading on the
other location.

https://codereview.appspot.com/7307109/diff/1/client/lib/markdown.dart
File client/lib/markdown.dart (right):

https://codereview.appspot.com/7307109/diff/1/client/lib/markdown.dart#newcod...
client/lib/markdown.dart:119: List<MarkdownNode> parseInline(String text) => new
InlineParser(text, this).parse();
long line

[jacobr, 2013-02-14 01:12:19]

https://codereview.appspot.com/7307109/diff/1/client/lib/markdown.dart
File client/lib/markdown.dart (right):

https://codereview.appspot.com/7307109/diff/1/client/lib/markdown.dart#newcod...
client/lib/markdown.dart:119: List<MarkdownNode> parseInline(String text) => new
InlineParser(text, this).parse();
On 2013/02/14 01:01:18, sigmund wrote:
> long line

Done.

[sigmund, 2013-02-14 04:28:45]

seems I had written some comments here, they might be duplicate with the
comments I wrote in the other CL

https://codereview.appspot.com/7307109/diff/1/client/lib/model.dart
File client/lib/model.dart (right):

https://codereview.appspot.com/7307109/diff/1/client/lib/model.dart#newcode370
client/lib/model.dart:370: /** Score of the search result match. Higher is
better. */
+ empty line before comment

https://codereview.appspot.com/7307109/diff/1/client/lib/model.dart#newcode406
client/lib/model.dart:406: if ((code >= 65 && code <= 90) || (code >= 48 && code
<= 57)) {
please introduce constants for these.

Consider doing:
final _A = 'A'.codeUnitAt(0);

or do regexp matching?

'[A-Z0-9]'

https://codereview.appspot.com/7307109/diff/1/client/lib/model.dart#newcode441
client/lib/model.dart:441: // TODO(jacobr): this formula is primitive. We should
order by popularity
long line

https://codereview.appspot.com/7307109/diff/1/client/lib/model.dart#newcode444
client/lib/model.dart:444: num score = -name.indexOf(query) -
element.name.length * 0.001;
some parethesis might help to make the it clear what precedence you want (I
assume * applies only to the length)?

https://codereview.appspot.com/7307109/diff/1/client/lib/model.dart#newcode455
client/lib/model.dart:455: // TODO(jacobr): ugly way to do this.
Could you do?
  return results.take(maxResults).toList();

[jacobr, 2013-02-14 05:51:44]

https://codereview.appspot.com/7307109/diff/1/client/lib/model.dart
File client/lib/model.dart (right):

https://codereview.appspot.com/7307109/diff/1/client/lib/model.dart#newcode370
client/lib/model.dart:370: /** Score of the search result match. Higher is
better. */
On 2013/02/14 04:28:46, sigmund wrote:
> + empty line before comment

Done.

https://codereview.appspot.com/7307109/diff/1/client/lib/model.dart#newcode441
client/lib/model.dart:441: // TODO(jacobr): this formula is primitive. We should
order by popularity
On 2013/02/14 04:28:46, sigmund wrote:
> long line 

Done.

https://codereview.appspot.com/7307109/diff/1/client/lib/model.dart#newcode444
client/lib/model.dart:444: num score = -name.indexOf(query) -
element.name.length * 0.001;
On 2013/02/14 04:28:46, sigmund wrote:
> some parethesis might help to make the it clear what precedence you want (I
> assume * applies only to the length)?
Done

https://codereview.appspot.com/7307109/diff/1/client/lib/model.dart#newcode455
client/lib/model.dart:455: // TODO(jacobr): ugly way to do this.
On 2013/02/14 04:28:46, sigmund wrote:
> Could you do?
>   return results.take(maxResults).toList();
Done