Don't use object pool for initialOldData, and properly recycle tempObject

Author: mrxz

src/core/a-entity.js

@@ -278,7 +278,7 @@ class AEntity extends ANode {
   /**
    * Initialize component.
    *
-   * @param {string} attrName - Attribute name asociated to the component.
+   * @param {string} attrName - Attribute name associated to the component.
    * @param {object} data - Component data
    * @param {boolean} isDependency - True if the component is a dependency.
    */

src/core/component.js

@@ -19,6 +19,7 @@ var upperCaseRegExp = new RegExp('[A-Z]+');
 
 // Object pools by component, created upon registration.
 var objectPools = {};
+var emptyInitialOldData = Object.freeze({});
 
 /**
  * Component class definition.
@@ -203,6 +204,7 @@ Component.prototype = {
       // If value is an object, copy it to our pooled newAttrValue object to use to update
       // the attrValue.
       tempObject = this.objectPool.use();
+      utils.objectPool.clearObject(tempObject);
       newAttrValue = utils.extend(tempObject, value);
     } else {
       newAttrValue = this.parseAttrValueForCache(value);
@@ -223,7 +225,7 @@ Component.prototype = {
     }
     utils.objectPool.clearObject(this.attrValue);
     this.attrValue = extendProperties(this.attrValue, newAttrValue, this.isObjectBased);
-    utils.objectPool.clearObject(tempObject);
+    this.objectPool.recycle(tempObject);
   },
 
   /**
@@ -271,7 +273,7 @@ Component.prototype = {
    *
    * @param {string} attrValue - HTML attribute value.
    *        If undefined, use the cached attribute value and continue updating properties.
-   * @param {boolean} clobber - The previous component data is overwritten by the atrrValue.
+   * @param {boolean} clobber - The previous component data is overwritten by the attrValue.
    */
   updateProperties: function (attrValue, clobber) {
     var el = this.el;
@@ -324,9 +326,8 @@ Component.prototype = {
 
     // For oldData, pass empty object to multiple-prop schemas or object single-prop schema.
     // Pass undefined to rest of types.
-    initialOldData = this.isObjectBased ? this.objectPool.use() : undefined;
+    initialOldData = this.isObjectBased ? emptyInitialOldData : undefined;
     this.update(initialOldData);
-    if (this.isObjectBased) { this.objectPool.recycle(initialOldData); }
 
     // Play the component if the entity is playing.
     if (el.isPlaying) { this.play(); }