[BUGFIX beta] Undefined should not set an attribute.

packages/ember-htmlbars/tests/helpers/bind_attr_test.js

@@ -584,7 +584,7 @@ QUnit.test("src attribute bound to undefined is not present", function() {
 
   runAppend(view);
 
-  ok(!view.element.hasAttribute('src'), "src attribute not present");
+  ok(!view.element.firstChild.hasAttribute('src'), "src attribute not present");
 });
 
 QUnit.test("src attribute bound to null is not present", function() {
@@ -597,7 +597,7 @@ QUnit.test("src attribute bound to null is not present", function() {
 
   runAppend(view);
 
-  ok(!view.element.hasAttribute('src'), "src attribute not present");
+  ok(!view.element.firstChild.hasAttribute('src'), "src attribute not present");
 });
 
 QUnit.test('specifying `<div {{bind-attr style=userValue}}></div>` is [DEPRECATED]', function() {

packages/ember-htmlbars/tests/helpers/input_test.js

@@ -350,3 +350,41 @@ QUnit.test("checkbox name is updated", function() {
 QUnit.test("checkbox checked property is updated", function() {
   equal(view.$('input').prop('checked'), false, "the checkbox isn't checked yet");
 });
+
+QUnit.module("{{input type='text'}} - null/undefined values", {
+  teardown() {
+    runDestroy(view);
+  }
+});
+
+QUnit.test("placeholder attribute bound to undefined is not present", function() {
+  view = View.extend({
+    controller: {},
+    template: compile('{{input placeholder=someThingNotThere}}')
+  }).create();
+
+  runAppend(view);
+
+  ok(!view.element.childNodes[1].hasAttribute('placeholder'), "attribute not present");
+
+  run(null, set, view, 'controller.someThingNotThere', 'foo');
+
+  equal(view.element.childNodes[1].placeholder, 'foo', "attribute is present");
+});
+
+QUnit.test("placeholder attribute bound to null is not present", function() {
+  view = View.extend({
+    controller: {
+      someNullProperty: null
+    },
+    template: compile('{{input placeholder=someNullProperty}}')
+  }).create();
+
+  runAppend(view);
+
+  ok(!view.element.childNodes[1].hasAttribute('placeholder'), "attribute not present");
+
+  run(null, set, view, 'controller.someNullProperty', 'foo');
+
+  equal(view.element.childNodes[1].placeholder, 'foo', "attribute is present");
+});

packages/ember-views/lib/attr_nodes/attr_node.js

@@ -49,12 +49,18 @@ AttrNode.prototype.render = function render(buffer) {
   if (this.isDestroying) {
     return;
   }
+
   var value = read(this.attrValue);
 
   if (this.attrName === 'value' && (value === null || value === undefined)) {
     value = '';
   }
 
+  if (value === undefined) {
+    value = null;
+  }
+
+
   // If user is typing in a value we don't want to rerender and loose cursor position.
   if (this.hasRenderedInitially && this.attrName === 'value' && this._morph.element.value === value) {
     this.lastValue = value;