fix packages for latest SDK changes

pkg/mdv/test/custom_element_bindings_test.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 custom_element_bindings_test;
 
 import 'dart:async';
 import 'dart:html';
 import 'package:mdv/mdv.dart' as mdv;
 import 'package:observe/observe.dart';
@@ skipping 108 matching lines @@
 
   observeTest('template bind uses overridden custom element bind', () {
 
     var model = toSymbolMap({'a': new Point(123, 444), 'b': new Monster(100)});
 
     var div = createTestHtml('<template bind>'
           '<my-custom-element my-point="{{a}}" scary-monster="{{b}}">'
           '</my-custom-element>'
         '</template>');
 
-    mdv.instanceCreated.listen((fragment) {
+    callback(fragment) {

[Siggi Cherem (dart-lang), 2013/08/05 20:20:52]

interesting, I didn't know this was equivalent to

var callback = (fragment) { ...};

I was afraid that maybe, like methods, each use of 'callback' as a variable
could have created a new closure for it. But, I did some testing and it seems to
work OK.

       for (var e in fragment.queryAll('my-custom-element')) {
         new MyCustomElement.attach(e);
       }
-    });
+    }
+    mdv.instanceCreated.add(callback);
 
     div.query('template').model = model;
     performMicrotaskCheckpoint();
 
     var element = div.nodes[1];
 
     expect(element.xtag is MyCustomElement, true,
         reason: '${element.xtag} should be a MyCustomElement');
 
     expect(element.xtag.myPoint, model[sym('a')]);
@@ skipping 10 matching lines @@
     performMicrotaskCheckpoint();
 
     expect(element.parentNode, null, reason: 'element was detached');
 
     model[sym('a')] = new Point(1, 2);
     model[sym('b')] = new Monster(200);
     performMicrotaskCheckpoint();
 
     expect(element.xtag.myPoint, null, reason: 'model was unbound');
     expect(element.xtag.scaryMonster.health, 100, reason: 'model was unbound');
+
+    mdv.instanceCreated.remove(callback);
   });
 
 }
 
 class Monster {
   int health;
   Monster(this.health);
 }
 
 /** Demonstrates a custom element overriding bind/unbind/unbindAll. */
 class MyCustomElement implements Element {
   final Element real;
 
   Point myPoint;
   Monster scaryMonster;
 
-  StreamSubscription _sub1, _sub2;
-
   MyCustomElement() : this.attach(new Element.tag('my-custom-element'));
 
   MyCustomElement.attach(this.real) {
     real.xtag = this;
   }
 
+  get attributes => real.attributes;
+  get bindings => real.bindings;
 
-  get attributes => real.attributes;
-
-  void bind(String name, model, String path) {
+  NodeBinding createBinding(String name, model, String path) {
     switch (name) {
       case 'my-point':
-        unbind('my-point');
-        attributes.remove('my-point');
-
-        _sub1 = new PathObserver(model, path).bindSync((v) {
-          myPoint = v;
-        });
-        return;
       case 'scary-monster':
-        unbind('scary-monster');
-        attributes.remove('scary-monster');
-
-        _sub2 = new PathObserver(model, path).bindSync((v) {
-          scaryMonster = v;
-        });
-        return;
+        return new _MyCustomBinding(this, name, model, path);
     }
-    real.bind(name, model, path);
+    return real.createBinding(name, model, path);
   }
 
-  void unbind(String name) {
-    switch (name) {
-      case 'my-point':
-        if (_sub1 != null) {
-          _sub1.cancel();
-          _sub1 = null;
-        }
-        return;
-      case 'scary-monster':
-        if (_sub2 != null) {
-          _sub2.cancel();
-          _sub2 = null;
-        }
-        return;
-    }
-    real.unbind(name);
+  bind(String name, model, String path) => real.bind(name, model, path);
+  void unbind(String name) => real.unbind(name);
+  void unbindAll() => real.unbindAll();
+}
+
+class _MyCustomBinding extends mdv.NodeBinding {
+  _MyCustomBinding(MyCustomElement node, property, model, path)
+      : super(node, property, model, path) {
+
+    node.attributes.remove(property);
   }
 
-  void unbindAll() {
-    unbind('my-point');
-    unbind('scary-monster');
-    real.unbindAll();
+  MyCustomElement get node => super.node;
+
+  void boundValueChanged(newValue) {
+    if (property == 'my-point') node.myPoint = newValue;
+    if (property == 'scary-monster') node.scaryMonster = newValue;
   }
 }
 
+
 /**
  * Demonstrates a custom element can override attributes []= and remove.
  * and see changes that the data binding system is making to the attributes.
  */
 class WithAttrsCustomElement implements Element {
   final Element real;
   final AttributeMapWrapper<String, String> attributes;
 
   factory WithAttrsCustomElement() {
     var real = new Element.tag('with-attrs-custom-element');
     var attributes = new AttributeMapWrapper(real.attributes);
     return new WithAttrsCustomElement._(real, attributes);
   }
 
   WithAttrsCustomElement._(this.real, this.attributes) {
     real.xtag = this;
   }
 
-  void bind(String name, model, String path) => real.bind(name, model, path);
+  createBinding(String name, model, String path) =>
+      real.createBinding(name, model, path);
+  bind(String name, model, String path) => real.bind(name, model, path);
   void unbind(String name) => real.unbind(name);
   void unbindAll() => real.unbindAll();
 }
 
 // TODO(jmesserly): would be nice to use mocks when mirrors work on dart2js.
 class AttributeMapWrapper<K, V> implements Map<K, V> {
   final List log = [];
   Map<K, V> _map;
 
   AttributeMapWrapper(this._map);
@@ skipping 15 matching lines @@
   }
 
   void clear() => _map.clear();
   void forEach(void f(K key, V value)) => _map.forEach(f);
   Iterable<K> get keys => _map.keys;
   Iterable<V> get values => _map.values;
   int get length => _map.length;
   bool get isEmpty => _map.isEmpty;
   bool get isNotEmpty => _map.isNotEmpty;
 }