package:observe fix various api issues

pkg/polymer_expressions/lib/eval.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.
 
 library polymer_expressions.eval;
 
 import 'dart:async';
 import 'dart:collection';
 
 @MirrorsUsed(metaTargets: const [Reflectable, ObservableProperty],
@@ skipping 402 matching lines @@
       : super(value);
 
   accept(Visitor v) => v.visitMapLiteralEntry(this);
 }
 
 class IdentifierObserver extends ExpressionObserver<Identifier>
     implements Identifier {
 
   IdentifierObserver(Identifier value) : super(value);
 
-  dynamic get value => _expr.value;
+  String get value => (_expr as Identifier).value;
 
   _updateSelf(Scope scope) {
-    _value = scope[_expr.value];
+    _value = scope[value];
 
-    var owner = scope.ownerOf(_expr.value);
+    var owner = scope.ownerOf(value);
     if (owner is Observable) {
-      _subscription = (owner as Observable).changes.listen(
-          (List<ChangeRecord> changes) {
-            var symbol = new Symbol(_expr.value);
-            if (changes.any((c) => c.changes(symbol))) {
-              _invalidate(scope);
-            }
-          });
+      var symbol = new Symbol(value);
+      _subscription = (owner as Observable).changes.listen((changes) {
+        if (changes.any(
+            (c) => c is PropertyChangeRecord && c.name == symbol)) {
+          _invalidate(scope);
+        }
+      });
     }
   }
 
   accept(Visitor v) => v.visitIdentifier(this);
 }
 
 class ParenthesizedObserver extends ExpressionObserver<ParenthesizedExpression>
     implements ParenthesizedExpression {
   final ExpressionObserver child;
 
@@ skipping 91 matching lines @@
       // special case [] because we don't need mirrors
       if (_expr.method == '[]') {
         assert(args.length == 1);
         var key = args[0];
         _value = receiverValue[key];
 
         if (receiverValue is Observable) {
           _subscription = (receiverValue as Observable).changes.listen(
               (List<ChangeRecord> changes) {
                 if (changes.any((c) =>
-                    c is MapChangeRecord && c.changes(key))) {
+                    c is MapChangeRecord && c.key == key)) {
                   _invalidate(scope);
                 }
               });
         }
       } else {
         var mirror = reflect(receiverValue);
         var symbol = new Symbol(_expr.method);
         _value = (_expr.isGetter)
             ? mirror.getField(symbol).reflectee
             : mirror.invoke(symbol, args, null).reflectee;
 
         if (receiverValue is Observable) {
           _subscription = (receiverValue as Observable).changes.listen(
               (List<ChangeRecord> changes) {
-                if (changes.any((c) => c.changes(symbol))) {
+                if (changes.any(
+                    (c) => c is PropertyChangeRecord && c.name == symbol)) {
                   _invalidate(scope);
                 }
               });
         }
       }
     }
   }
 
   accept(Visitor v) => v.visitInvoke(this);
 }
@@ skipping 70 matching lines @@
    * This does not work for calls that need to pass more than one argument.
    */
   call(arg0) => mirror.invoke(symbol, [arg0], null).reflectee;
 }
 
 class EvalException implements Exception {
   final String message;
   EvalException(this.message);
   String toString() => "EvalException: $message";
 }