Removed note explaining why the logic is there, since the justification is now different.
jim committed
Jan 5, 2016 at 15:24 UTC
4e3d38ed2794906bb7acceb97bdd8f6bc9a5bf5f
1 file changed
-6
src/renderers/shared/reconciler/ReactCompositeComponent.js
-6
@@ -806,13 +806,7 @@ var ReactCompositeComponentMixin = {
806
this._processChildContext(context)
807
);
808
} else {
809
- // TODO: This is currently necessary due to the unfortunate caching
810
- // that ReactMount does which makes it exceedingly difficult to unmount
811
- // a set of siblings without accidentally repopulating the node cache (see
812
- // #5151). Once ReactMount no longer stores the nodes by ID, this method
813
- // can go away.
809
var oldNativeNode = ReactReconciler.getNativeNode(prevComponentInstance);
815
-
810
ReactReconciler.unmountComponent(prevComponentInstance);
811
812
this._renderedNodeType = ReactNodeTypes.getType(nextRenderedElement);