The Dart VM should provide reflective access to metadata

[peter-ahe-google]

The Dart VM should support this getter on DeclarationMirror:

/**
 * A list of the metadata associated with this declaration.
 */
List<InstanceMirror> get metadata;

See issue #6614.

[iposva-google]

Removed Priority-Medium label.
Added Priority-Unassigned label.

[iposva-google]

Set owner to @mhausner.
Removed Priority-Unassigned label.
Added Priority-High, Accepted labels.

[gbracha]

One should now be able to obtain metadata for top level functions, variables and classes as wella s for class members. Libraries and type variables still don't work, but this is progress.

[DartBot]

This comment was originally written by @mhausner

---

Reducing priority since the most useful functionality is implemented.

---

Removed Priority-High label.
Added Priority-Medium label.

[sethladd]

Hi Matthias,

What is expected to work at this point? I see this simple case works:

import 'dart:mirrors';

const awesome = const Awesome('way cool');

class Awesome {
  final String msg;
  const Awesome(this.msg);
  
  String toString() => msg;
}

@awesome('it works!')
class Cool {
  
}

void main() {
  var classMirror = reflectClass(Cool);
  var metadata = classMirror.metadata;
  var obj = metadata.first.reflectee;
  print(metadata);
  print(obj);
}

Thanks!

---

Removed Type-Defect label.
Added Type-Enhancement label.

[peter-ahe-google]

Both dart2js and the VM pass this test:
https://code.google.com/p/dart/source/browse/branches/bleeding_edge/dart/tests/lib/mirrors/metadata_test.dart

[DartBot]

This comment was originally written by @mhausner

---

Supported at this time is metadata on:

• top-level variables, functions, getters and setters
• classes
• class members

[rmacnak-google]

Support for metadata on libraries added in r25721.

[gbracha]

Marked this as blocking #6614.

[iposva-google]

Ryan, can you please list what is still needs to be implemented before we can close this bug?

---

Set owner to @rmacnak-google.

[rmacnak-google]

I believe metadata for parameters is the only one still missing.

https://chromiumcodereview.appspot.com/23224016/

---

Added Started label.

[rmacnak-google]

Actually we're also missing metadata on mixin and function typedefs.

[rmacnak-google]

Metadata for mixin and function typedefs added in r26680.

[rmacnak-google]

Metadata for parameters added in r27082.

[rmacnak-google]

Gah, type parameters can have metadata too.

[DartBot]

This comment was originally written by [email protected]

---

Metadata for type parameters added in r27523.

That should be it.

---

Added Fixed label.