Reapply "reflectType() dynamic type arguments support (#26012)"

Reapply "reflectType() dynamic type arguments support (#26012)"

This was a pull request: 8a8033a4176a2f26e8ed035497631d539dc1ede5

MirrorsUsed doesn't transitively include reflective information. However, it must still be able to create TypeMirrors for types that are used as return- or parameter types.

Initially, the patch checked that TypeMirrors had the correct number of arguments for generic types. This is now disabled.

A better approach would be to know if a class has full reflective information, or not. But this would require much bigger changes to the system.

R=sigmund@google.com

Committed: https://github.com/dart-lang/sdk/commit/2547caab258bec7b8d9742df2cce3fdfc2680786

[floitsch, 2017-01-06 18:56:42]

Description was changed from

==========
Reapply "reflectType() dynamic type arguments support (#26012)"
==========

to

==========
Reapply "reflectType() dynamic type arguments support (#26012)"

This was a pull request: 8a8033a4176a2f26e8ed035497631d539dc1ede5
==========

[floitsch, 2017-01-06 22:33:39]

Description was changed from

==========
Reapply "reflectType() dynamic type arguments support (#26012)"

This was a pull request: 8a8033a4176a2f26e8ed035497631d539dc1ede5
==========

to

==========
Reapply "reflectType() dynamic type arguments support (#26012)"

This was a pull request: 8a8033a4176a2f26e8ed035497631d539dc1ede5

MirrorsUsed doesn't transitively include reflective information. However, it
must still be able to create TypeMirrors for types that are used as return- or
parameter types.

Initially, the patch checked that TypeMirrors had the correct number of
arguments for generic types. This is now disabled.

A better approach would be to know if a class has full reflective information,
or not. But this would require much bigger changes to the system.
==========

[floitsch, 2017-01-06 22:34:05]

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

[floitsch, 2017-01-06 22:34:06]

I think the easiest is, if I just commit the CL (once I have your ok).

[Siggi Cherem (dart-lang), 2017-01-09 18:05:45]

lgtm

[floitsch, 2017-01-10 14:53:28]

Description was changed from

==========
Reapply "reflectType() dynamic type arguments support (#26012)"

This was a pull request: 8a8033a4176a2f26e8ed035497631d539dc1ede5

MirrorsUsed doesn't transitively include reflective information. However, it
must still be able to create TypeMirrors for types that are used as return- or
parameter types.

Initially, the patch checked that TypeMirrors had the correct number of
arguments for generic types. This is now disabled.

A better approach would be to know if a class has full reflective information,
or not. But this would require much bigger changes to the system.
==========

to

==========
Reapply "reflectType() dynamic type arguments support (#26012)"

This was a pull request: 8a8033a4176a2f26e8ed035497631d539dc1ede5

MirrorsUsed doesn't transitively include reflective information. However, it
must still be able to create TypeMirrors for types that are used as return- or
parameter types.

Initially, the patch checked that TypeMirrors had the correct number of
arguments for generic types. This is now disabled.

A better approach would be to know if a class has full reflective information,
or not. But this would require much bigger changes to the system.

R=sigmund@google.com

Committed:
https://github.com/dart-lang/sdk/commit/2547caab258bec7b8d9742df2cce3fdfc2680786
==========

[floitsch, 2017-01-10 14:53:29]

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