@samitouri / QOS-React / commits / d9dabac48d

Remove now-unused instancesByReactID in DOMInput

Ben Alpert committed Dec 23, 2015 at 15:08 UTC d9dabac48d0a86a94d2ad9936a81105bd129ca37
2 files changed +1 -24
src/renderers/dom/client/wrappers/ReactDOMInput.js
-11
@@ -20,8 +20,6 @@ var assign = require('Object.assign');
20 var invariant = require('invariant');
21 var warning = require('warning');
22
23 -var instancesByReactID = {};
24 -
23 var didWarnValueLink = false;
24 var didWarnCheckedLink = false;
25 var didWarnValueNull = false;
@@ -112,15 +110,6 @@ var ReactDOMInput = {
110 };
111 },
112
115 - mountReadyWrapper: function(inst) {
116 - // Can't be in mountWrapper or else server rendering leaks.
117 - instancesByReactID[inst._rootNodeID] = inst;
118 - },
119 -
120 - unmountWrapper: function(inst) {
121 - delete instancesByReactID[inst._rootNodeID];
122 - },
123 -
113 updateWrapper: function(inst) {
114 var props = inst._currentElement.props;
115
src/renderers/dom/shared/ReactDOMComponent.js
+1 -13
@@ -333,10 +333,6 @@ function trapBubbledEventsLocal() {
333 }
334 }
335
336 -function mountReadyInputWrapper() {
337 - ReactDOMInput.mountReadyWrapper(this);
338 -}
339 -
336 function postUpdateSelectWrapper() {
337 ReactDOMSelect.postUpdateWrapper(this);
338 }
@@ -582,13 +578,8 @@ ReactDOMComponent.Mixin = {
578 }
579
580 switch (this._tag) {
585 - case 'input':
586 - transaction.getReactMountReady().enqueue(
587 - mountReadyInputWrapper,
588 - this
589 - );
590 - // falls through
581 case 'button':
582 + case 'input':
583 case 'select':
584 case 'textarea':
585 if (props.autoFocus) {
@@ -1020,9 +1011,6 @@ ReactDOMComponent.Mixin = {
1011 }
1012 }
1013 break;
1023 - case 'input':
1024 - ReactDOMInput.unmountWrapper(this);
1025 - break;
1014 case 'html':
1015 case 'head':
1016 case 'body':