respect default value

src/diff/index.js

@@ -443,6 +443,7 @@
 			} else if (i == 'dangerouslySetInnerHTML') {
 				oldHtml = value;
 			} else if (i !== 'key' && !(i in newProps)) {
+				if (i == 'value' && 'defaultValue' in newProps) continue;
 				setProperty(dom, i, null, value, isSvg);
 			}
 		}
@@ -524,6 +525,7 @@
 					// again, which triggers IE11 to re-evaluate the select value
 					(nodeType === 'option' && inputValue !== oldProps[i]))
 			) {
+				console.log('setting', i);
 				setProperty(dom, i, inputValue, oldProps[i], false);
 			}
 

test/browser/hydrate.test.js

@@ -57,6 +57,12 @@ describe('hydrate()', () => {
 		teardown(scratch);
 	});
 
+	it('should respect defaultValue in hydrate', () => {
+		scratch.innerHTML = '<input value="foo">';
+		hydrate(<input defaultValue="foo" />, scratch);
+		expect(scratch.firstChild.value).to.equal('foo');
+	});
+
 	it('should reuse existing DOM', () => {
 		const onClickSpy = sinon.spy();
 		const html = ul([li('1'), li('2'), li('3')]);

test/browser/render.test.js

@@ -1,5 +1,5 @@
 import { setupRerender } from 'preact/test-utils';
-import { createElement, render, Component, options } from 'preact';
+import { createElement, render, Component, options, hydrate } from 'preact';
 import {
 	setupScratch,
 	teardown,
@@ -472,6 +472,13 @@ describe('render()', () => {
 		expect(scratch.firstChild.spellcheck).to.equal(false);
 	});
 
+	// Test for preactjs/preact#651
+	it('should respect defaultValue in render', () => {
+		scratch.innerHTML = '<input value="foo">';
+		render(<input defaultValue="foo" />, scratch);
+		expect(scratch.firstChild.value).to.equal('foo');
+	});
+
 	it('should render download attribute', () => {
 		render(<a download="" />, scratch);
 		expect(scratch.firstChild.getAttribute('download')).to.equal('');