Fixes issues in string interpolation tests

Fixes issues in string interpolation tests
This started as my attempt to add a test for http://codereview.chromium.org/8591010/, but it snowballed into fixing other issues when I discovered that StringInterpolateTest was failing already.

Committed: https://code.google.com/p/dart/source/detail?r=1634

[Jennifer Messerly, 2011-11-17 21:48:29]

tbr

I know we had some questions about what $a$b$c *should* do in string
interpolation, but it's pretty easy to implement the current spec'd version and
get the tests passing. We can always revert that part when/if spec changes.

[jimhug, 2011-11-18 17:05:29]

LGTM!

Any time a fix snowballs into fixing more bugs it's fine with me.  I'm a little
sad that you disrupted my non-violent protest of the $ confusions - but I guess
that following the spec is more important than my feelings <smile>.

http://codereview.chromium.org/8585044/diff/4001/frog/frogsh
File frog/frogsh (right):

http://codereview.chromium.org/8585044/diff/4001/frog/frogsh#newcode14577
frog/frogsh:14577: function lang_Token(kind, source, start, end) {
TODO: There seems to be some sort of non-determinism in which of the
Token/Node/Parser/etc. types gets the prefix.  We should try to get this
deterministic in order to better take advantage of these diffs.

http://codereview.chromium.org/8585044/diff/4001/frog/tokenizer.g.dart
File frog/tokenizer.g.dart (right):

http://codereview.chromium.org/8585044/diff/4001/frog/tokenizer.g.dart#newcod...
frog/tokenizer.g.dart:376: static bool isIdentifierStart(int c) {
This definition is now incorrect - if you want to follow the spec. I believe
that '$' is a legal identifier start character.  In the tokenizer, you handle
this in the '$' case so I don't think there's a real but in the code, but this
method name should be changed or its behavior should be changed to be correct.

http://codereview.chromium.org/8585044/diff/4001/frog/tokenizer.g.dart#newcod...
frog/tokenizer.g.dart:396: static bool isInterpIdentifierPart(int c) {
FYI - I really hate this -but I can't fault you for just following the spec...

http://codereview.chromium.org/8585044/diff/4001/frog/value.dart
File frog/value.dart (right):

http://codereview.chromium.org/8585044/diff/4001/frog/value.dart#newcode137
frog/value.dart:137: var m = type.resolveMember('noSuchMethod');
Funny, I have this same change sitting in my current CL <smile>.