Fix logic in processSinglePathBinding

If the delegate returns an observer, it still must use the ObserverTransform if the binding is compound (has non-empty string tokens outside the mustache)

R=arv
BUG=

Review URL: https://codereview.appspot.com/43060044

Author: rafaelw

src/TemplateBinding.js

@@ -653,10 +653,9 @@
   function processSinglePathBinding(name, tokens, node, model) {
     var delegateFn = tokens[2];
     var delegateValue = delegateFn && delegateFn(model, node);
-    if (Observer.isObservable(delegateValue))
-      return delegateValue;
+    var observer = Observer.isObservable(delegateValue) ? delegateValue :
+        new PathObserver(model, tokens[1]);
 
-    var observer = new PathObserver(model, tokens[1]);
     return tokens.isSimplePath ? observer :
         new ObserverTransform(observer, tokens.combinator);
   }

tests/tests.js

@@ -2166,6 +2166,34 @@ suite('Binding Delegate API', function() {
 
     assert.equal('barValue', div.lastChild.getAttribute('bar'));
   });
+
+  test('issue-18', function() {
+
+    var delegate = {
+      prepareBinding: function(path, name, node) {
+        if (name != 'class')
+          return;
+
+        return function(model) {
+          return new PathObserver(model, path);
+        }
+      }
+    };
+
+    var div = createTestHtml(
+        '<template bind>' +
+          '<div class="foo: {{ bar }}"></div>' +
+        '</template>');
+
+    var model = {
+      bar: 2
+    };
+
+    recursivelySetTemplateModel(div, model, delegate);
+    Platform.performMicrotaskCheckpoint();
+
+    assert.equal('foo: 2', div.lastChild.getAttribute('class'));
+  });
 });
 
 suite('Compat', function() {