Fix LibraryMirror.invoke to call through the contents of an implicit getter.

Fix LibraryMirror.invoke to call through the contents of an implicit getter.

BUG=http://dartbug.com/17452
R=asiva@google.com, gbracha@google.com

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

[floitsch, 2014-03-13 12:06:42]

I would prefer to split the multitest.
I know, it duplicates code, but dart2js only fails on some of the 01-tests, and
adding "02: ok" lines (so we can mark them in the status file) is then harder.

https://codereview.chromium.org/196953007/diff/1/tests/lib/mirrors/invoke_cal...
File tests/lib/mirrors/invoke_call_through_getter_test.dart (right):

https://codereview.chromium.org/196953007/diff/1/tests/lib/mirrors/invoke_cal...
tests/lib/mirrors/invoke_call_through_getter_test.dart:133: // Multitest so we
exerice the paths where are and are not previously accessed
exercise

https://codereview.chromium.org/196953007/diff/1/tests/lib/mirrors/invoke_cal...
File tests/lib/mirrors/invoke_call_through_implicit_getter_test.dart (right):

https://codereview.chromium.org/196953007/diff/1/tests/lib/mirrors/invoke_cal...
tests/lib/mirrors/invoke_call_through_implicit_getter_test.dart:139: //
Multitest so we exerice the paths where are and are not previously accessed
exercise

[siva, 2014-03-13 17:32:05]

The VM changes LGTM.

[rmacnak, 2014-03-13 17:41:36]

On 2014/03/13 12:06:42, floitsch wrote:
> I would prefer to split the multitest.
> I know, it duplicates code, but dart2js only fails on some of the 01-tests,
and
> adding "02: ok" lines (so we can mark them in the status file) is then harder.

Done.

[gbracha, 2014-03-14 01:03:44]

tests lgtm

[rmacnak, 2014-03-14 01:14:14]

Committed patchset #2 manually as r33682 (presubmit successful).