From 4e3d38ed2794906bb7acceb97bdd8f6bc9a5bf5f Mon Sep 17 00:00:00 2001 From: jim Date: Tue, 5 Jan 2016 15:24:09 -0800 Subject: [PATCH] Removed note explaining why the logic is there, since the justification is now different. --- src/renderers/shared/reconciler/ReactCompositeComponent.js | 6 ------ 1 file changed, 6 deletions(-) diff --git a/src/renderers/shared/reconciler/ReactCompositeComponent.js b/src/renderers/shared/reconciler/ReactCompositeComponent.js index 3c5da9815744c..612bf5fd34035 100644 --- a/src/renderers/shared/reconciler/ReactCompositeComponent.js +++ b/src/renderers/shared/reconciler/ReactCompositeComponent.js @@ -806,13 +806,7 @@ var ReactCompositeComponentMixin = { this._processChildContext(context) ); } else { - // TODO: This is currently necessary due to the unfortunate caching - // that ReactMount does which makes it exceedingly difficult to unmount - // a set of siblings without accidentally repopulating the node cache (see - // #5151). Once ReactMount no longer stores the nodes by ID, this method - // can go away. var oldNativeNode = ReactReconciler.getNativeNode(prevComponentInstance); - ReactReconciler.unmountComponent(prevComponentInstance); this._renderedNodeType = ReactNodeTypes.getType(nextRenderedElement);