Tests for instanceMembers, staticMembers, toplevelMembers.

sdk/lib/mirrors/mirrors.dart

Index: sdk/lib/mirrors/mirrors.dart
diff --git a/sdk/lib/mirrors/mirrors.dart b/sdk/lib/mirrors/mirrors.dart
index 109f83226d7cf16e0c7527c11b8288d4fc2797f6..4349675ea5434e9f592018e39435ed6e927c9ae8 100644
--- a/sdk/lib/mirrors/mirrors.dart
+++ b/sdk/lib/mirrors/mirrors.dart
@@ -684,6 +684,8 @@ abstract class LibraryMirror implements DeclarationMirror, ObjectMirror {
    */
   Map<Symbol, DeclarationMirror> get declarations;
 
+  Map<Symbol, MethodMirror> get toplevelMembers;

[gbracha, 2013/10/26 18:37:48]

So is the agreed name for api? It's not great, but I can live with it. There's
no spec for it though. It needs to be added. 

+
   /**
    * An immutable map from from names to mirrors for all members in
    * this library.
@@ -839,6 +841,23 @@ abstract class ClassMirror implements TypeMirror, ObjectMirror {
   Map<Symbol, DeclarationMirror> get declarations;
 
   /**
+   * Returns a map of the methods, getters and setters of an 
+   * instance of the class.
+   * The intent is to capture those members that can are
+   * part of the API of an instance. Hence fields are not
+   * included. Getters that are implicitly introduced 
+   * by fields are included. Setters that are implicitly introduced
+   * by fields are included iff the field is non-final. 

[gbracha, 2013/10/26 18:37:48]

Since finals no longer imply a setter, we can drop the "iff the field is
non-final".

+   * The map includes methods, getters and setters that
+   * are inherited as well as those introduced by the class itself.
+   * The map is keyed by the simple names of the members.
+   */
+  Map<Symbol, MethodMirror> get instanceMembers;
+
+  Map<Symbol, MethodMirror> get staticMembers;

[gbracha, 2013/10/26 18:37:48]

Add analogous spec for static members.

+

[ahe, 2013/10/29 10:14:38]

Extra line.

+
+  /**
    * The mixin of this class.
    * If this class is the result of a mixin application of the
    * form S with M, returns a class mirror on M.