Merge pull request #5728 from spicyj/input-gc

Remove now-unused instancesByReactID in DOMInput

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;
@@ -148,15 +146,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;
 

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) {
@@ -1033,9 +1024,6 @@ ReactDOMComponent.Mixin = {
           }
         }
         break;
-      case 'input':
-        ReactDOMInput.unmountWrapper(this);
-        break;
       case 'html':
       case 'head':
       case 'body':