From 4c1e77910e23610e93660d5b9de4ce252f3efe7c Mon Sep 17 00:00:00 2001
From: Rafael Weinstein <rafaelw@chromium.org>
Date: Fri, 11 Apr 2014 10:55:00 -0700
Subject: [PATCH] cr comments

---
 src/TemplateBinding.js | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/TemplateBinding.js b/src/TemplateBinding.js
index ae69271..efae0ed 100644
--- a/src/TemplateBinding.js
+++ b/src/TemplateBinding.js
@@ -903,26 +903,26 @@
     return map;
   }
 
-  var contentIdCounter = 1;
+  var contentUidCounter = 1;
 
   // TODO(rafaelw): Setup a MutationObserver on content which clears the id
   // so that bindingMaps regenerate when the template.content changes.
-  function getContentId(content) {
+  function getContentUid(content) {
     var id = content.id_;
     if (!id)
-      id = content.id_ = contentIdCounter++;
+      id = content.id_ = contentUidCounter++;
     return id;
   }
 
   // Each delegate is associated with a set of bindingMaps, one for each
   // content which may be used by a template. The intent is that each binding
-  // delegate gets the opportunity to prepare* the template content once
-  // across all uses.
+  // delegate gets the opportunity to prepare the instance (via the prepare*
+  // delegate calls) once across all uses.
   // TODO(rafaelw): Separate out the parse map from the binding map. In the
   // current implementation, if two delegates need a binding map for the same
   // content, the second will have to reparse.
   function getInstanceBindingMap(content, delegate_) {
-    var contentId = getContentId(content);
+    var contentId = getContentUid(content);
     if (delegate_) {
       var map = delegate_.bindingMaps[contentId];
       if (!map) {