Fix event handling when input name='nodeName'

facebook · Mar 21, 2016 · 026b0a0 · 026b0a0
1 parent 40c4635
commit 026b0a0
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 8 deletions.
diff --git a/src/renderers/dom/client/eventPlugins/ChangeEventPlugin.js b/src/renderers/dom/client/eventPlugins/ChangeEventPlugin.js
@@ -57,10 +57,9 @@ var activeElementValueProp = null;
  * SECTION: handle `change` event
  */
 function shouldUseChangeEvent(elem) {
-  var nodeName = elem.nodeName && elem.nodeName.toLowerCase();
   return (
-    nodeName === 'select' ||
-    (nodeName === 'input' && elem.type === 'file')
+    elem instanceof window.HTMLSelectElement ||
+    (elem instanceof window.HTMLInputElement && elem.type === 'file')
   );
 }
 
@@ -307,7 +306,7 @@ function shouldUseClickEvent(elem) {
   // This approach works across all browsers, whereas `change` does not fire
   // until `blur` in IE8.
   return (
-    (elem.nodeName && elem.nodeName.toLowerCase() === 'input') &&
+    (elem instanceof window.HTMLInputElement) &&
     (elem.type === 'checkbox' || elem.type === 'radio')
   );
 }

diff --git a/src/shared/utils/isTextInputElement.js b/src/shared/utils/isTextInputElement.js
@@ -33,10 +33,9 @@ var supportedInputTypes = {
 };
 
 function isTextInputElement(elem) {
-  var nodeName = elem && elem.nodeName && elem.nodeName.toLowerCase();
-  return nodeName && (
-    (nodeName === 'input' && supportedInputTypes[elem.type]) ||
-    nodeName === 'textarea'
+  return (
+    elem instanceof window.HTMLTextAreaElement ||
+    (elem instanceof window.HTMLInputElement && supportedInputTypes[elem.type])
   );
 }