Warn if a falsy value prop is provided without an onChange

scripts/fiber/tests-passing.txt

@@ -1839,6 +1839,9 @@ src/renderers/dom/shared/wrappers/__tests__/ReactDOMInput-test.js
 * should control radio buttons
 * should control radio buttons if the tree updates during render
 * should warn with value and no onChange handler and readOnly specified
+* should warn with an empty value and no onChange handler and readOnly specified
+* should warn with a value of number `0` and no onChange handler and readOnly specified
+* should warn with a value of `false` and no onChange handler and readOnly specified
 * should have a this value of undefined if bind is not used
 * should warn with checked and no onChange handler with readOnly specified
 * should update defaultValue to empty string

src/renderers/dom/shared/utils/ReactControlledValuePropTypes.js

@@ -35,7 +35,7 @@ if (__DEV__) {
   var propTypes = {
     value: function(props, propName, componentName) {
       if (
-        !props[propName] ||
+        props[propName] == null ||
         hasReadOnlyValue[props.type] ||
         props.onChange ||
         props.readOnly ||
@@ -52,7 +52,7 @@ if (__DEV__) {
     },
     checked: function(props, propName, componentName) {
       if (
-        !props[propName] ||
+        props[propName] == null ||
         props.onChange ||
         props.readOnly ||
         props.disabled

src/renderers/dom/shared/wrappers/__tests__/ReactDOMInput-test.js

@@ -780,6 +780,63 @@ describe('ReactDOMInput', () => {
     );
   });
 
+  it('should warn with an empty value and no onChange handler and readOnly specified', () => {
+    ReactTestUtils.renderIntoDocument(
+      <input type="text" value="" readOnly={true} />,
+    );
+    expectDev(console.error.calls.count()).toBe(0);
+
+    ReactTestUtils.renderIntoDocument(
+      <input type="text" value="" readOnly={false} />,
+    );
+    expectDev(console.error.calls.count()).toBe(1);
+    expectDev(normalizeCodeLocInfo(console.error.calls.argsFor(0)[0])).toBe(
+      'Warning: Failed form propType: You provided a `value` prop to a form ' +
+        'field without an `onChange` handler. This will render a read-only ' +
+        'field. If the field should be mutable use `defaultValue`. ' +
+        'Otherwise, set either `onChange` or `readOnly`.\n' +
+        '    in input (at **)',
+    );
+  });
+
+  it('should warn with a value of number `0` and no onChange handler and readOnly specified', () => {
+    ReactTestUtils.renderIntoDocument(
+      <input type="text" value={0} readOnly={true} />,
+    );
+    expectDev(console.error.calls.count()).toBe(0);
+
+    ReactTestUtils.renderIntoDocument(
+      <input type="text" value={0} readOnly={false} />,
+    );
+    expectDev(console.error.calls.count()).toBe(1);
+    expectDev(normalizeCodeLocInfo(console.error.calls.argsFor(0)[0])).toBe(
+      'Warning: Failed form propType: You provided a `value` prop to a form ' +
+        'field without an `onChange` handler. This will render a read-only ' +
+        'field. If the field should be mutable use `defaultValue`. ' +
+        'Otherwise, set either `onChange` or `readOnly`.\n' +
+        '    in input (at **)',
+    );
+  });
+
+  it('should warn with a value of `false` and no onChange handler and readOnly specified', () => {
+    ReactTestUtils.renderIntoDocument(
+      <input type="text" value={false} readOnly={true} />,
+    );
+    expectDev(console.error.calls.count()).toBe(0);
+
+    ReactTestUtils.renderIntoDocument(
+      <input type="text" value={false} readOnly={false} />,
+    );
+    expectDev(console.error.calls.count()).toBe(1);
+    expectDev(normalizeCodeLocInfo(console.error.calls.argsFor(0)[0])).toBe(
+      'Warning: Failed form propType: You provided a `value` prop to a form ' +
+        'field without an `onChange` handler. This will render a read-only ' +
+        'field. If the field should be mutable use `defaultValue`. ' +
+        'Otherwise, set either `onChange` or `readOnly`.\n' +
+        '    in input (at **)',
+    );
+  });
+
   it('should have a this value of undefined if bind is not used', () => {
     var unboundInputOnChange = function() {
       expect(this).toBe(undefined);
@@ -793,12 +850,12 @@ describe('ReactDOMInput', () => {
 
   it('should warn with checked and no onChange handler with readOnly specified', () => {
     ReactTestUtils.renderIntoDocument(
-      <input type="checkbox" checked="false" readOnly={true} />,
+      <input type="checkbox" checked={false} readOnly={true} />,
     );
     expectDev(console.error.calls.count()).toBe(0);
 
     ReactTestUtils.renderIntoDocument(
-      <input type="checkbox" checked="false" readOnly={false} />,
+      <input type="checkbox" checked={false} readOnly={false} />,
     );
     expectDev(console.error.calls.count()).toBe(1);
   });

src/renderers/dom/shared/wrappers/__tests__/ReactDOMTextarea-test.js

@@ -99,7 +99,7 @@ describe('ReactDOMTextarea', () => {
   });
 
   it('should display `value` of number 0', () => {
-    var stub = <textarea value={0} />;
+    var stub = <textarea value={0} onChange={emptyFunction} />;
     var node = renderTextarea(stub);
 
     expect(node.value).toBe('0');