Do not infer types of arguments of closures and methods named "call".

Do not infer types of arguments of closures and methods named "call".

R=ahe@google.com

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

[ahe, 2013-04-29 10:07:41]

I'll try patching it in locally.

https://codereview.chromium.org/14260012/diff/2001/sdk/lib/_internal/compiler...
File sdk/lib/_internal/compiler/implementation/resolution/members.dart (right):

https://codereview.chromium.org/14260012/diff/2001/sdk/lib/_internal/compiler...
sdk/lib/_internal/compiler/implementation/resolution/members.dart:2174: if
(element != null && element.isConstructor()) {
I don't understand what this is for.  Is this from a different CL?

[ngeoffray, 2013-04-29 10:12:09]

https://codereview.chromium.org/14260012/diff/2001/sdk/lib/_internal/compiler...
File sdk/lib/_internal/compiler/implementation/resolution/members.dart (right):

https://codereview.chromium.org/14260012/diff/2001/sdk/lib/_internal/compiler...
sdk/lib/_internal/compiler/implementation/resolution/members.dart:2174: if
(element != null && element.isConstructor()) {
On 2013/04/29 10:07:41, ahe wrote:
> I don't understand what this is for.  Is this from a different CL?

No, it's related to that CL. If I don't do that check, we're creating a closure
selector (line 2180), which is not correct.

[ahe, 2013-04-29 10:19:16]

I patched this in, and dart2js now self-hosts and the resulting JavaScript can
compile everything I throw at it.

[ahe, 2013-04-29 10:25:05]

LGTM

[ngeoffray, 2013-04-29 10:38:53]

Committed patchset #1 manually as r22109 (presubmit successful).