dart2js: mirrors. Support Typedef.reflectedType.

dart2js: mirrors. Support Typedef.reflectedType.

BUG= http://dartbug.com/18938
R=karlklose@google.com

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

[floitsch, 2014-09-19 10:05:31]

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

[karlklose, 2014-09-19 10:39:51]

Change to reflectType LGTM.

https://codereview.chromium.org/579303003/diff/40001/sdk/lib/_internal/compil...
File sdk/lib/_internal/compiler/js_lib/js_mirrors.dart (left):

https://codereview.chromium.org/579303003/diff/40001/sdk/lib/_internal/compil...
sdk/lib/_internal/compiler/js_lib/js_mirrors.dart:2535: // TODO(ahe): Implement
this method.
Restore TODO (with 'ahe' -> 'floitsch').

https://codereview.chromium.org/579303003/diff/40001/sdk/lib/_internal/compil...
File sdk/lib/_internal/compiler/js_lib/js_mirrors.dart (right):

https://codereview.chromium.org/579303003/diff/40001/sdk/lib/_internal/compil...
sdk/lib/_internal/compiler/js_lib/js_mirrors.dart:2526: List<TypeMirror> get
typeArguments => const <TypeMirror>[];
Do you plan to create an InstantiatedJsTypedefMirror for typedefs with type
arguments?

Otherwise these three methods should probably stay unimplemented.

[floitsch, 2014-09-19 11:42:09]

https://codereview.chromium.org/579303003/diff/40001/sdk/lib/_internal/compil...
File sdk/lib/_internal/compiler/js_lib/js_mirrors.dart (left):

https://codereview.chromium.org/579303003/diff/40001/sdk/lib/_internal/compil...
sdk/lib/_internal/compiler/js_lib/js_mirrors.dart:2535: // TODO(ahe): Implement
this method.
On 2014/09/19 10:39:51, karlklose wrote:
> Restore TODO (with 'ahe' -> 'floitsch').

Done.

https://codereview.chromium.org/579303003/diff/40001/sdk/lib/_internal/compil...
File sdk/lib/_internal/compiler/js_lib/js_mirrors.dart (right):

https://codereview.chromium.org/579303003/diff/40001/sdk/lib/_internal/compil...
sdk/lib/_internal/compiler/js_lib/js_mirrors.dart:2526: List<TypeMirror> get
typeArguments => const <TypeMirror>[];
On 2014/09/19 10:39:51, karlklose wrote:
> Do you plan to create an InstantiatedJsTypedefMirror for typedefs with type
> arguments?
> 
> Otherwise these three methods should probably stay unimplemented.

They are needed by the createRuntimeType.
Might be thet JsTypeBoundTypedefMirror is what we want, but we can revisit when
we look at it.

[floitsch, 2014-09-19 11:42:48]

Committed patchset #4 (id:60001) manually as 40494 (presubmit successful).