From d9dabac48d0a86a94d2ad9936a81105bd129ca37 Mon Sep 17 00:00:00 2001 From: Ben Alpert Date: Wed, 23 Dec 2015 15:08:43 -0800 Subject: [PATCH] Remove now-unused instancesByReactID in DOMInput --- src/renderers/dom/client/wrappers/ReactDOMInput.js | 11 ----------- src/renderers/dom/shared/ReactDOMComponent.js | 14 +------------- 2 files changed, 1 insertion(+), 24 deletions(-) diff --git a/src/renderers/dom/client/wrappers/ReactDOMInput.js b/src/renderers/dom/client/wrappers/ReactDOMInput.js index 682a34961a..c99b6d828d 100644 --- a/src/renderers/dom/client/wrappers/ReactDOMInput.js +++ b/src/renderers/dom/client/wrappers/ReactDOMInput.js @@ -20,8 +20,6 @@ var assign = require('Object.assign'); var invariant = require('invariant'); var warning = require('warning'); -var instancesByReactID = {}; - var didWarnValueLink = false; var didWarnCheckedLink = false; var didWarnValueNull = false; @@ -112,15 +110,6 @@ var ReactDOMInput = { }; }, - mountReadyWrapper: function(inst) { - // Can't be in mountWrapper or else server rendering leaks. - instancesByReactID[inst._rootNodeID] = inst; - }, - - unmountWrapper: function(inst) { - delete instancesByReactID[inst._rootNodeID]; - }, - updateWrapper: function(inst) { var props = inst._currentElement.props; diff --git a/src/renderers/dom/shared/ReactDOMComponent.js b/src/renderers/dom/shared/ReactDOMComponent.js index 65b76b892b..e0ee675a5e 100644 --- a/src/renderers/dom/shared/ReactDOMComponent.js +++ b/src/renderers/dom/shared/ReactDOMComponent.js @@ -333,10 +333,6 @@ function trapBubbledEventsLocal() { } } -function mountReadyInputWrapper() { - ReactDOMInput.mountReadyWrapper(this); -} - function postUpdateSelectWrapper() { ReactDOMSelect.postUpdateWrapper(this); } @@ -582,13 +578,8 @@ ReactDOMComponent.Mixin = { } switch (this._tag) { - case 'input': - transaction.getReactMountReady().enqueue( - mountReadyInputWrapper, - this - ); - // falls through case 'button': + case 'input': case 'select': case 'textarea': if (props.autoFocus) { @@ -1020,9 +1011,6 @@ ReactDOMComponent.Mixin = { } } break; - case 'input': - ReactDOMInput.unmountWrapper(this); - break; case 'html': case 'head': case 'body':