From d0d4280640967eca2faad0d96be6fca08ac846a6 Mon Sep 17 00:00:00 2001 From: Nathan Hunzaker Date: Mon, 11 Jun 2018 09:16:50 -0400 Subject: [PATCH] Remove old reference to inst._wrapperState (#12987) This commit removes a reference to inst._wrapperState, which was the old way of tracking input state in the stack renderer. This means we no longer need to pass the instance into the associated function, allowing us to eliminate an exception for IE (and a TODO). --- packages/react-dom/src/events/ChangeEventPlugin.js | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/packages/react-dom/src/events/ChangeEventPlugin.js b/packages/react-dom/src/events/ChangeEventPlugin.js index ce0d328d876d5..2d04e5dd43f07 100644 --- a/packages/react-dom/src/events/ChangeEventPlugin.js +++ b/packages/react-dom/src/events/ChangeEventPlugin.js @@ -231,14 +231,8 @@ function getTargetInstForInputOrChangeEvent(topLevelType, targetInst) { } } -function handleControlledInputBlur(inst, node) { - // TODO: In IE, inst is occasionally null. Why? - if (inst == null) { - return; - } - - // Fiber and ReactDOM keep wrapper state in separate places - let state = inst._wrapperState || node._wrapperState; +function handleControlledInputBlur(node) { + let state = node._wrapperState; if (!state || !state.controlled || node.type !== 'number') { return; @@ -303,7 +297,7 @@ const ChangeEventPlugin = { // When blurring, set the value attribute for number inputs if (topLevelType === TOP_BLUR) { - handleControlledInputBlur(targetInst, targetNode); + handleControlledInputBlur(targetNode); } }, };