assigning bindingDelegate while template is operating throws

R=arv BUG= Review URL: https://codereview.appspot.com/59490049
googlearchive · Feb 6, 2014 · b78a841 · jmesserly · May 21, 2014 · rafaelw
1 parent e66083f
commit b78a841
Show file tree

Hide file tree

Showing 2 changed files with 153 additions and 126 deletions.
diff --git a/src/TemplateBinding.js b/src/TemplateBinding.js
@@ -557,6 +557,11 @@
     // make sense to issue a warning or even throw if the template is already
     // "activated", since this would be a strange thing to do.
     set bindingDelegate(bindingDelegate) {
+      if (this.delegate_) {
+        throw Error('Template must be cleared before a new bindingDelegate ' +
+                    'can be assigned');
+      }
+
       this.setDelegate_(this.newDelegate_(bindingDelegate));
     },