add @MirrorsUsed to observe/polymer/polymer_exprs pkgs

pkg/observe/lib/transform.dart

 // Copyright (c) 2013, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 /**
  * Code transform for @observable. The core transformation is relatively
  * straightforward, and essentially like an editor refactoring.
  */
 library observe.transform;
 
@@ skipping 86 matching lines @@
 }
 
 class _ErrorCollector extends AnalysisErrorListener {
   final errors = <AnalysisError>[];
   onError(error) => errors.add(error);
 }
 
 _getSpan(SourceFile file, ASTNode node) => file.span(node.offset, node.end);
 
 /** True if the node has the `@observable` or `@published` annotation. */
-// TODO(jmesserly): it is not good to be hard coding these. We should do a
-// resolve and do a proper ObservableProperty subtype check. However resolve
-// is very expensive in analyzer_experimental, so it isn't feasible yet.
+// TODO(jmesserly): it is not good to be hard coding Polymer support here.
 bool _hasObservable(AnnotatedNode node) =>
-    _hasAnnotation(node, 'observable') || _hasAnnotation(node, 'published');
+    node.metadata.any(_isObservableAnnotation);
 
-bool _hasAnnotation(AnnotatedNode node, String name) {
-  // TODO(jmesserly): this isn't correct if the annotation has been imported
-  // with a prefix, or cases like that. We should technically be resolving, but
-  // that is expensive.
-  return node.metadata.any((m) => m.name.name == name &&
-      m.constructorName == null && m.arguments == null);
-}
+// TODO(jmesserly): this isn't correct if the annotation has been imported
+// with a prefix, or cases like that. We should technically be resolving, but
+// that is expensive in analyzer_experimental, so it isn't feasible yet.
+bool _isObservableAnnotation(Annotation node) =>
+    _isAnnotationContant(node, 'observable') ||
+    _isAnnotationContant(node, 'published') ||
+    _isAnnotationType(node, 'ObservableProperty') ||
+    _isAnnotationType(node, 'PublishedProperty');
+
+bool _isAnnotationContant(Annotation m, String name) =>
+    m.name.name == name && m.constructorName == null && m.arguments == null;
+
+bool _isAnnotationType(Annotation m, String name) => m.name == name;
 
 void _transformClass(ClassDeclaration cls, TextEditTransaction code,
     SourceFile file, TransformLogger logger) {
 
   if (_hasObservable(cls)) {
     logger.warning('@observable on a class no longer has any effect. '
         'It should be placed on individual fields.',
         span: _getSpan(file, cls));
   }
 
@@ skipping 200 matching lines @@
   //     @otherMetaData
   //         Foo
   //             foo = 1, bar = 2,
   //             baz;
   //
   // Will be transformed into something like:
   //
   //     @observable
   //     @OtherMetaData()
   //         Foo
-  //             get foo => __foo; Foo __foo = 1; set foo ...; ... bar ...
+  //             get foo => __foo; Foo __foo = 1; @reflectable set foo ...; ...
   //             @observable @OtherMetaData() Foo get baz => __baz; Foo baz; ...

[Siggi Cherem (dart-lang), 2013/10/15 21:52:18]

should we include @reflectable on the 'get' part of the example too?

[Jennifer Messerly, 2013/10/15 22:03:29]

good catch.

   //
   // Metadata is moved to the getter.
 
   String metadata = '';
   if (fields.variables.length > 1) {
     metadata = member.metadata
       .map((m) => _getOriginalCode(code, m))
       .join(' ');
+    metadata = '@reflectable $metadata';
   }
 
   for (int i = 0; i < fields.variables.length; i++) {
     final field = fields.variables[i];
     final name = field.name.name;
 
     var beforeInit = 'get $name => __\$$name; $type __\$$name';
 
     // The first field is expanded differently from subsequent fields, because
     // we can reuse the metadata and type annotation.
-    if (i > 0) beforeInit = '$metadata $type $beforeInit';
+    if (i == 0) {
+      final begin = member.metadata.first.offset;
+      code.edit(begin, begin, '@reflectable ');
+    } else {
+      beforeInit = '$metadata $type $beforeInit';
+    }
 
     code.edit(field.name.offset, field.name.end, beforeInit);
 
     // Replace comma with semicolon
     final end = _findFieldSeperator(field.endToken.next);
     if (end.type == TokenType.COMMA) code.edit(end.offset, end.end, ';');
 
-    code.edit(end.end, end.end, ' set $name($type value) { '
+    code.edit(end.end, end.end, ' @reflectable set $name($type value) { '
         '__\$$name = notifyPropertyChange(#$name, __\$$name, value); }');
   }
 }
 
 Token _findFieldSeperator(Token token) {
   while (token != null) {
     if (token.type == TokenType.COMMA || token.type == TokenType.SEMICOLON) {
       break;
     }
     token = token.next;
   }
   return token;
 }