Add support for getters, setters and constructors in declarations of classMirrors.

reflectable/lib/src/mirrors_unimpl.dart

Index: reflectable/lib/src/mirrors_unimpl.dart
diff --git a/reflectable/lib/src/mirrors_unimpl.dart b/reflectable/lib/src/mirrors_unimpl.dart
index a07b6318eb0e571498eda6d1fbafb1e7b5d8a3e6..c9be684613b27fca2098ba734cc5880fcdab7019 100644
--- a/reflectable/lib/src/mirrors_unimpl.dart
+++ b/reflectable/lib/src/mirrors_unimpl.dart
@@ -146,7 +146,8 @@ abstract class MethodMirrorUnimpl extends DeclarationMirrorUnimpl
   String get source => _unsupported();
   List<ParameterMirror> get parameters => _unsupported();
   bool get isStatic => _unsupported();
-  bool get isAbstract => _unsupported();
+  // We do not support reflection on abstract methods.

[eernst, 2015/06/18 10:07:18]

Let's include abstract methods even though dart:mirrors doesn't. Then we'll keep
track of the cases that don't work in pre-transform form by putting them in
.status (as opposed to spending time on fixing them in any way), and as an bonus
document various little glitches in dart:mirrors that may or may not be known.

[sigurdm, 2015/06/18 11:04:03]

Done.

+  bool get isAbstract => false;
   bool get isSynthetic => _unsupported();
   bool get isRegularMethod => _unsupported();
   bool get isOperator => _unsupported();
@@ -175,16 +176,15 @@ class MethodMirrorImpl implements MethodMirror {
   @override
   String get constructorName => name;
 
+  // We do not support reflection on abstract methods.

[eernst, 2015/06/18 10:07:18]

We may then support them after all.

[sigurdm, 2015/06/18 11:04:03]

Done.

   @override
-  bool get isAbstract => 0 != descriptor & constants.abstractAttribute;
+  bool get isAbstract => false;

[eernst, 2015/06/18 10:07:18]

In case we support abstract methods, reverting this line should be sufficient.

[sigurdm, 2015/06/18 11:04:03]

Done.

 
   @override
   bool get isConstConstructor => 0 != descriptor & constants.constAttribute;
 
   @override
-  bool get isConstructor => isGenerativeConstructor ||
-      isFactoryConstructor ||
-      isRedirectingConstructor;
+  bool get isConstructor => isFactoryConstructor || isGenerativeConstructor;
 
   @override
   bool get isFactoryConstructor => kind == constants.factoryConstructor;
@@ -203,7 +203,8 @@ class MethodMirrorImpl implements MethodMirror {
   bool get isPrivate => 0 != descriptor & constants.privateAttribute;
 
   @override
-  bool get isRedirectingConstructor => kind == constants.redirectingConstructor;
+  bool get isRedirectingConstructor =>
+      0 != descriptor & constants.redirectingConstructor;
 
   @override
   bool get isRegularMethod => kind == constants.method;
@@ -241,7 +242,7 @@ class MethodMirrorImpl implements MethodMirror {
 
   @override
   String get simpleName => isConstructor
-      ? (name == '' ? "$owner.simpleName" : "${owner.simpleName}.$name")
+      ? (name == '' ? "${owner.simpleName}" : "${owner.simpleName}.$name")
       : name;
 
   @override