From dcefa708c94707a83c820601569617f8418b5a3f Mon Sep 17 00:00:00 2001
From: Scott Miles <sjmiles@google.com>
Date: Mon, 5 Nov 2012 12:52:20 -0800
Subject: [PATCH 1/3] in takeAttributes, trap attributes containing mustache
 markup and treat them as uninitialized

---
 src/g-component.html | 16 ++++++++++++++--
 1 file changed, 14 insertions(+), 2 deletions(-)

diff --git a/src/g-component.html b/src/g-component.html
index 6744258673..1407266c8a 100644
--- a/src/g-component.html
+++ b/src/g-component.html
@@ -296,17 +296,29 @@
           squelchSideEffects = true;
           this.boundAttributes.forEach(function(a) {
             if (this.hasAttribute(a)) {
-              var value = deserializeValue(this.getAttribute(a));
+              // get raw attribute value
+              var value = this.getAttribute(a);
+              // filter out 'mustached' values, these are to be
+              // replaced with bound-data and are not runtime
+              // values themselves
+              if (value.indexOf(bindModel.mustache) >= 0) {
+                return;
+              }
+              // deserialize Boolean or Number values from attribute
+              value = deserializeValue(value);
               if (this[a] !== value) {
+                // track values that differ from property values
                 changed.push({name: a, old: this[a]});
+                // install new value (side-effects squelched)
                 this[a] = value;
               }
-              this[a] = deserializeValue(value);
             }
           }, this);
         } finally {
           squelchSideEffects = false;
         }
+        // invoke side-effects in a batch after initializing
+        // all values
         changed.forEach(function(c) {
           propertyChanged.call(this, c.name, c.old);
         }, this);

From 98cce180cf0ffe4afd1a90caceeefab6375ea1a6 Mon Sep 17 00:00:00 2001
From: Scott Miles <sjmiles@google.com>
Date: Mon, 5 Nov 2012 18:34:50 -0800
Subject: [PATCH 2/3] s/inUber/inLifecycle, other non-semantic changes; add
 optional 3rd parameter for timeout duration to asyncMethod

---
 src/g-component.html | 28 ++++++++++++++++------------
 1 file changed, 16 insertions(+), 12 deletions(-)

diff --git a/src/g-component.html b/src/g-component.html
index 1407266c8a..67e937f111 100644
--- a/src/g-component.html
+++ b/src/g-component.html
@@ -72,7 +72,7 @@
 
       // event bindings
       //
-      // <element events="onclick: click, onkeypress: keypress, ..."
+      // <element handlers="onclick: click, onkeypress: keypress, ..."
       //
       // create listeners on instances that map named events to
       // methods on the instance
@@ -379,37 +379,41 @@
 
       // asyncMethod impl
       
-      var asyncMethod = function(inMethod, inArgs) {
+      var asyncMethod = function(inMethod, inArgs, inTimeout) {
         window.setTimeout(function() {
           this[inMethod].apply(this, inArgs);
-        }.bind(this), 0);
+        }.bind(this), inTimeout || 0);
       };
          
       // decorate HTMLElementElement with toolkit API
 
-      HTMLElementElement.prototype.component = function(inUber) {
+      HTMLElementElement.prototype.component = function(inLifecycle) {
         var attributes = this.element.attributes;
         this.lifecycle({
           shadowRootCreated: function(inRoot) {
-            initialize.call(this, inRoot, inUber);
+            initialize.call(this, inRoot, inLifecycle);
           },
           created: function() {
-            //console.log("created", this);
+            // TODO(sjmiles): notional event delegate attached to DOM tree
+            //   to explore declarative event mapping as platform feature
             this.controller = this;
+            // automate basic tasks, so applications never have to
             automate.call(this, 
               attributes, 
-              // TODO(sjmiles): remove backward-compatibility expressions
-              inUber[conventions.OBSERVE_DIRECTIVE] || inUber.published,  
-              inUber[conventions.DELEGATES_DIRECTIVE] || inUber.bindings || inUber.delegates
+              // TODO(sjmiles): remove deprecated expressions
+              inLifecycle[conventions.OBSERVE_DIRECTIVE] 
+                || inLifecycle.published,  
+              inLifecycle[conventions.DELEGATES_DIRECTIVE] 
+                 || inLifecycle.bindings || inLifecycle.delegates
             );
             takeAttributes.call(this);
-            if (inUber.created) {
-              inUber.created.call(this);
+            if (inLifecycle.created) {
+              inLifecycle.created.call(this);
             }
             this.attrObserver = new AttrObserver(this);
           }
         });
-        var p = inUber.prototype || {};
+        var p = inLifecycle.prototype || {};
         // attach some API
         // TODO(sjmiles): this is probably not the best way to do this;
         // probably better to insert another link in the prototype chain

From 8013d1a1f4a90b51a45be790f7870a841f9110b6 Mon Sep 17 00:00:00 2001
From: Scott Miles <sjmiles@google.com>
Date: Mon, 5 Nov 2012 18:40:17 -0800
Subject: [PATCH 3/3] make asyncMethod return the setTimeout handle

---
 src/g-component.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/g-component.html b/src/g-component.html
index acae8ef77e..99dd0a462b 100644
--- a/src/g-component.html
+++ b/src/g-component.html
@@ -379,7 +379,7 @@
       // asyncMethod impl
       
       var asyncMethod = function(inMethod, inArgs, inTimeout) {
-        window.setTimeout(function() {
+        return window.setTimeout(function() {
           this[inMethod].apply(this, inArgs);
         }.bind(this), inTimeout || 0);
       };