Turn compile time errors related to missing getters and setters into invocation

Turn compile time errors related to missing getters and setters into invocation
of noSuchMethod or into throwing of NoSuchMethodError according to spec.
Return result of InvocationMirror.invokeOn to caller.
Fix various tests.
Committed: https://code.google.com/p/dart/source/detail?r=16463

[hausner, 2012-12-22 00:35:20]

LGTM

[regis, 2012-12-22 00:41:52]

Thanks!

[srdjan, 2012-12-26 17:36:50]

lgtm

[Lasse Reichstein Nielsen, 2013-01-02 09:59:29]

https://codereview.chromium.org/11669015/diff/6001/tests/language/invocation_...
File tests/language/invocation_mirror_test.dart (right):

https://codereview.chromium.org/11669015/diff/6001/tests/language/invocation_...
tests/language/invocation_mirror_test.dart:45: Expect.equals(0,
im.namedArguments.length, "$name:#named");
This change is incorrect.
The API specification for InvocationMirror states that if isGetter is true, then
both positional and named arguments must be null.
Now, I'm quite open to changing that if you have reasons to prefer using empty
unmodifiable collections instead - it's actually quite reasonable, and if you
have the (reusable) objects anyway, it should be no more expensive than passing
null.

[regis, 2013-01-02 17:37:21]

https://codereview.chromium.org/11669015/diff/6001/tests/language/invocation_...
File tests/language/invocation_mirror_test.dart (right):

https://codereview.chromium.org/11669015/diff/6001/tests/language/invocation_...
tests/language/invocation_mirror_test.dart:45: Expect.equals(0,
im.namedArguments.length, "$name:#named");
On 2013/01/02 09:59:29, Lasse Reichstein Nielsen wrote:
> This change is incorrect.
> The API specification for InvocationMirror states that if isGetter is true,
then
> both positional and named arguments must be null.
> Now, I'm quite open to changing that if you have reasons to prefer using empty
> unmodifiable collections instead - it's actually quite reasonable, and if you
> have the (reusable) objects anyway, it should be no more expensive than
passing
> null.

The Language Spec has refined the description of the InvocationMirror instance
passed to noSuchMethod(). In the case of a getter, positionalArguments is an
empty array and namedArguments is an empty map. Please, update the API spec
accordingly. Thanks.