Call deliver() from attachedCallback() rather than createdCallback().

This is more efficient for programatically created widgets because it avoids
calling computeProperties() and refreshRendering() twice on initialization:
once with the default property values and once after user specified values have
been processed.

Refs #385, works around ibm-js/deliteful#535 and therefore fixes ibm-js/deliteful#532.

CustomElement.js

@@ -215,10 +215,6 @@ define([
 						attributes: true
 					});
 				}
-
-				if (this.deliver) {
-					this.deliver();
-				}
 			}
 		}),
 
@@ -241,6 +237,11 @@ define([
 		 */
 		attachedCallback: dcl.advise({
 			before: function () {
+				// Call computeProperties() and refreshRendering() for declaratively set properties.
+				// Do this in attachedCallback() rather than createdCallback() to avoid calling refreshRendering() etc.
+				// prematurely in the programmatic case (i.e. calling it before user parameters have been applied).
+				this.deliver();
+
 				this.attached = true;
 
 				// Protect against repeated calls.

tests/unit/handlebars.js

@@ -731,19 +731,19 @@ define([
 			var cwpc = new ClassConstWidget({
 				className: "userDefinedClass"
 			});
-			assert.strictEqual(cwpc.className, "constClass myBaseClass userDefinedClass",
+			assert.strictEqual(cwpc.className, "constClass userDefinedClass myBaseClass",
 				"programmatic const");
 
 			// Test programmatic with class={{foo}}
 			var cwpv = new ClassVarWidget({
 				className: "userDefinedClass"
 			});
-			assert.strictEqual(cwpv.className, "myTemplateClass1 myBaseClass userDefinedClass",
+			assert.strictEqual(cwpv.className, "userDefinedClass myTemplateClass1 myBaseClass",
 				"programmatic var, before update");
 
 			cwpv.templateClass = "myTemplateClass2";
 			cwpv.deliver();
-			assert.strictEqual(cwpv.className, "myBaseClass userDefinedClass myTemplateClass2",
+			assert.strictEqual(cwpv.className, "userDefinedClass myBaseClass myTemplateClass2",
 				"programmatic var, after update");
 		},