port TemplateBinding to ed3266266e751b5ab1f75f8e0509d0d5f0ef35d8

pkg/polymer/lib/src/instance.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.
 
 part of polymer;
 
 /**
  * Use this annotation to publish a field as an attribute. For example:
  *
  *     class MyPlaybackElement extends PolymerElement {
@@ skipping 419 matching lines @@
       // to ensure binding is not left on attribute if property
       // does not update due to not changing.
       // Dart note: we include this patch:
       // https://github.com/Polymer/polymer/pull/319
       reflectPropertyToAttribute(property.simpleName);
       return bindings[name] = observer;
     }
   }
 
   Map<String, NodeBinding> get bindings => nodeBindFallback(this).bindings;
+  TemplateInstance get templateInstance =>
+      nodeBindFallback(this).templateInstance;
 
   void unbind(String name) => nodeBindFallback(this).unbind(name);
 
   void asyncUnbindAll() {
     if (_unbound == true) return;
     _unbindLog.fine('[$localName] asyncUnbindAll');
     _unbindAllJob = _runJob(_unbindAllJob, unbindAll, Duration.ZERO);
   }
 
   void unbindAll() {
@@ skipping 281 matching lines @@
    * custom element in which the node exists. Adding a '@' in the path directs
    * the event binding to use the model path as the event listener.  In both
    * cases, the actual listener is attached to a generic method which evaluates
    * the bound path at event execution time.
    */
   // from src/instance/event.js#prepareBinding
   // Dart note: template_binding doesn't have the notion of prepareBinding, so
   // we implement this by wrapping/overriding getBinding instead.
   // TODO(sorvell): we're patching the syntax while evaluating
   // event bindings. we'll move this to a better spot when that's done
-  static getBindingWithEvents(
-      model, String path, name, node, originalGetBinding) {
+  static PrepareBindingFunction prepareBinding(String path, String name, node,
+      originalPrepareBinding) {
+
     // if lhs an event prefix,
-    if (name is! String || !_hasEventPrefix(name)) {
-      return originalGetBinding(model, path, name, node);
-    }
+    if (!_hasEventPrefix(name)) return originalPrepareBinding(path, name, node);
 
-    // provide an event-binding callback.
-    // return (model, name, node) {
-    if (_eventsLog.isLoggable(Level.FINE)) {
-      _eventsLog.fine('event: [${node.localName}].$name => '
-          '[${model.localName}].$path())');
-    }
-    var eventName = _removeEventPrefix(name);
-    // TODO(sigmund): polymer.js dropped event translations. reconcile?
-    var translated = _eventTranslations[eventName];
-    eventName = translated != null ? translated : eventName;
-    return node.on[eventName].listen((event) {
-      var ctrlr = _findController(node);
-      if (ctrlr is! Polymer) return;
-      var obj = ctrlr;
-      var method = path;
-      if (path[0] == '@') {
-        obj = model;
-        // Dart note: using getBinding gets us the result of evaluating the
-        // original path (without the @) as a normal expression.
-        method = originalGetBinding(model, path.substring(1), name, node).value;
+    return (model, node) {
+      // provide an event-binding callback.

[Siggi Cherem (dart-lang), 2013/10/29 21:00:07]

move this line above the return (to match polymer.js's code)

[Jennifer Messerly, 2013/10/29 22:35:07]

Done.

+      // return (model, name, node) {

[Siggi Cherem (dart-lang), 2013/10/29 21:00:07]

remove this line (it was there just to make it easier to map to the original
code)

[Jennifer Messerly, 2013/10/29 22:35:07]

Done.

+      if (_eventsLog.isLoggable(Level.FINE)) {
+        _eventsLog.fine('event: [${node}].$name => [${model}].$path())');
       }
-      var detail = event is CustomEvent ?
-          (event as CustomEvent).detail : null;
-      ctrlr.dispatchMethod(obj, method, [event, detail, node]);
-    });
+      var eventName = _removeEventPrefix(name);
+      // TODO(sigmund): polymer.js dropped event translations. reconcile?
+      var translated = _eventTranslations[eventName];
+      eventName = translated != null ? translated : eventName;
+
+      // TODO(jmesserly): returning a StreamSubscription as the model is quite
+      // strange. package:template_binding doesn't have any cleanup logic to
+      // handle that.
+      return node.on[eventName].listen((event) {
+        var ctrlr = _findController(node);
+        if (ctrlr is! Polymer) return;
+        var obj = ctrlr;
+        var method = path;
+        if (path[0] == '@') {
+          obj = model;
+          // Dart note: using getBinding gets us the result of evaluating the
+          // original path (without the @) as a normal expression.

[Siggi Cherem (dart-lang), 2013/10/29 21:00:07]

we might also want to add a note here about what we discussed earlier today
(that we are using polymer-expr here instead of a path observer.) 

I think it's reasonable to do it this way because this is basically hooking into
the existing syntax, but we might have to follow up with polymer-dev on this.

[Jennifer Messerly, 2013/10/29 22:35:07]

Switched it to use PathObserver. but yeah follow up sounds good.

+          method = originalPrepareBinding(path.substring(1), name, node)
+              (model, node).value;
+        }
+        var detail = event is CustomEvent ?
+            (event as CustomEvent).detail : null;
+        ctrlr.dispatchMethod(obj, method, [event, detail, node]);
+      });
+    };
   }
 
   // TODO(jmesserly): this won't find the correct host unless the ShadowRoot
   // was created on a PolymerElement.
   static Polymer _findController(Node node) {
     while (node.parentNode != null) {
       node = node.parentNode;
     }
     return _shadowHost[node];
   }
@@ skipping 284 matching lines @@
     polymerCreated();
   }
 }
 
 class _PropertyValue {
   Object oldValue, newValue;
   _PropertyValue(this.oldValue);
 }
 
 class _PolymerExpressionsWithEventDelegate extends PolymerExpressions {
-  getBinding(model, String path, name, node) {
-    return Polymer.getBindingWithEvents(
-        model, path, name, node, super.getBinding);
+  prepareBinding(String path, name, node) {

[Siggi Cherem (dart-lang), 2013/10/29 21:00:07]

nit, it was like this before, but consider using => here

[Jennifer Messerly, 2013/10/29 22:35:07]

Done.

+    return Polymer.prepareBinding(path, name, node, super.prepareBinding);
   }
 }