Fix crash in typechecker

Fix crash in typechecker

BUG=http://dartbug.com/21793
R=floitsch@google.com

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

[Johnni Winther, 2014-12-09 13:39:14]

johnniwinther@google.com changed reviewers:
+ floitsch@google.com

[floitsch, 2014-12-09 14:00:15]

LGTM.

https://codereview.chromium.org/787753005/diff/20001/tests/language/regress_2...
File tests/language/regress_21793_test.dart (right):

https://codereview.chromium.org/787753005/diff/20001/tests/language/regress_2...
tests/language/regress_21793_test.dart:17: test();  /// 01: continued
Nit:
I prefer to be more minimalistic in what we remove.
If there was a typo in the "print" line, that it might not test the right thing.

/*   /// 01: static type warning, runtime error
class A { call(x) => x; }
*/   /// 01: continued

main() {
  print(new A()(499));
}

[Johnni Winther, 2014-12-10 08:30:02]

Committed patchset #3 (id:40001) manually as 42236 (presubmit successful).

[Johnni Winther, 2014-12-10 08:37:19]

https://codereview.chromium.org/787753005/diff/20001/tests/language/regress_2...
File tests/language/regress_21793_test.dart (right):

https://codereview.chromium.org/787753005/diff/20001/tests/language/regress_2...
tests/language/regress_21793_test.dart:17: test();  /// 01: continued
On 2014/12/09 14:00:15, floitsch wrote:
> Nit:
> I prefer to be more minimalistic in what we remove.
> If there was a typo in the "print" line, that it might not test the right
thing.
> 
> /*   /// 01: static type warning, runtime error
> class A { call(x) => x; }
> */   /// 01: continued
> 
> main() {
>   print(new A()(499));
> }

Done.