Remove Stack-only code branches

Author: gaearon

src/isomorphic/classic/element/ReactCurrentOwner.js

@@ -10,7 +10,6 @@
 
 'use strict';
 
-import type {ReactInstance} from 'ReactInstanceType';
 import type {Fiber} from 'ReactFiber';
 
 /**
@@ -24,7 +23,7 @@ var ReactCurrentOwner = {
    * @internal
    * @type {ReactComponent}
    */
-  current: (null: null | ReactInstance | Fiber),
+  current: (null: null | Fiber),
 };
 
 module.exports = ReactCurrentOwner;

src/renderers/dom/fiber/ReactDOMFiberEntry.js

@@ -229,7 +229,7 @@ var DOMRenderer = ReactFiberReconciler({
     if (__DEV__) {
       // TODO: take namespace into account when validating.
       const hostContextDev = ((hostContext: any): HostContextDev);
-      validateDOMNesting(type, null, null, hostContextDev.ancestorInfo);
+      validateDOMNesting(type, null, hostContextDev.ancestorInfo);
       if (
         typeof props.children === 'string' ||
         typeof props.children === 'number'
@@ -240,7 +240,7 @@ var DOMRenderer = ReactFiberReconciler({
           type,
           null,
         );
-        validateDOMNesting(null, string, null, ownAncestorInfo);
+        validateDOMNesting(null, string, ownAncestorInfo);
       }
       parentNamespace = hostContextDev.namespace;
     } else {
@@ -295,7 +295,7 @@ var DOMRenderer = ReactFiberReconciler({
           type,
           null,
         );
-        validateDOMNesting(null, string, null, ownAncestorInfo);
+        validateDOMNesting(null, string, ownAncestorInfo);
       }
     }
     return diffProperties(
@@ -362,7 +362,7 @@ var DOMRenderer = ReactFiberReconciler({
   ): TextInstance {
     if (__DEV__) {
       const hostContextDev = ((hostContext: any): HostContextDev);
-      validateDOMNesting(null, text, null, hostContextDev.ancestorInfo);
+      validateDOMNesting(null, text, hostContextDev.ancestorInfo);
     }
     var textNode: TextInstance = createTextNode(text, rootContainerInstance);
     precacheFiberNode(internalInstanceHandle, textNode);

src/renderers/dom/shared/ReactDOMComponentFlags.js

@@ -9,116 +9,18 @@
 
 'use strict';
 
-var DOMProperty = require('DOMProperty');
-var ReactDOMComponentFlags = require('ReactDOMComponentFlags');
 var {HostComponent, HostText} = require('ReactTypeOfWork');
-var {ELEMENT_NODE, COMMENT_NODE} = require('HTMLNodeType');
 
 var invariant = require('fbjs/lib/invariant');
 
-var ATTR_NAME = DOMProperty.ID_ATTRIBUTE_NAME;
-var Flags = ReactDOMComponentFlags;
-
 var randomKey = Math.random().toString(36).slice(2);
-
 var internalInstanceKey = '__reactInternalInstance$' + randomKey;
-
 var internalEventHandlersKey = '__reactEventHandlers$' + randomKey;
 
-/**
- * Check if a given node should be cached.
- */
-function shouldPrecacheNode(node, nodeID) {
-  return (
-    (node.nodeType === ELEMENT_NODE &&
-      node.getAttribute(ATTR_NAME) === '' + nodeID) ||
-    (node.nodeType === COMMENT_NODE &&
-      node.nodeValue === ' react-text: ' + nodeID + ' ') ||
-    (node.nodeType === COMMENT_NODE &&
-      node.nodeValue === ' react-empty: ' + nodeID + ' ')
-  );
-}
-
-/**
- * Drill down (through composites and empty components) until we get a host or
- * host text component.
- *
- * This is pretty polymorphic but unavoidable with the current structure we have
- * for `_renderedChildren`.
- */
-function getRenderedHostOrTextFromComponent(component) {
-  var rendered;
-  while ((rendered = component._renderedComponent)) {
-    component = rendered;
-  }
-  return component;
-}
-
-/**
- * Populate `_hostNode` on the rendered host/text component with the given
- * DOM node. The passed `inst` can be a composite.
- */
-function precacheNode(inst, node) {
-  var hostInst = getRenderedHostOrTextFromComponent(inst);
-  hostInst._hostNode = node;
-  node[internalInstanceKey] = hostInst;
-}
-
 function precacheFiberNode(hostInst, node) {
   node[internalInstanceKey] = hostInst;
 }
 
-function uncacheNode(inst) {
-  var node = inst._hostNode;
-  if (node) {
-    delete node[internalInstanceKey];
-    inst._hostNode = null;
-  }
-}
-
-/**
- * Populate `_hostNode` on each child of `inst`, assuming that the children
- * match up with the DOM (element) children of `node`.
- *
- * We cache entire levels at once to avoid an n^2 problem where we access the
- * children of a node sequentially and have to walk from the start to our target
- * node every time.
- *
- * Since we update `_renderedChildren` and the actual DOM at (slightly)
- * different times, we could race here and see a newer `_renderedChildren` than
- * the DOM nodes we see. To avoid this, ReactMultiChild calls
- * `prepareToManageChildren` before we change `_renderedChildren`, at which
- * time the container's child nodes are always cached (until it unmounts).
- */
-function precacheChildNodes(inst, node) {
-  if (inst._flags & Flags.hasCachedChildNodes) {
-    return;
-  }
-  var children = inst._renderedChildren;
-  var childNode = node.firstChild;
-  outer: for (var name in children) {
-    if (!children.hasOwnProperty(name)) {
-      continue;
-    }
-    var childInst = children[name];
-    var childID = getRenderedHostOrTextFromComponent(childInst)._domID;
-    if (childID === 0) {
-      // We're currently unmounting this child in ReactMultiChild; skip it.
-      continue;
-    }
-    // We assume the child nodes are in the same order as the child instances.
-    for (; childNode !== null; childNode = childNode.nextSibling) {
-      if (shouldPrecacheNode(childNode, childID)) {
-        precacheNode(childInst, childNode);
-        continue outer;
-      }
-    }
-    // We reached the end of the DOM children without finding an ID match.
-    invariant(false, 'Unable to find element with ID %s.', childID);
-  }
-  inst._flags |= Flags.hasCachedChildNodes;
-}
-
 /**
  * Given a DOM node, return the closest ReactDOMComponent or
  * ReactDOMTextComponent instance ancestor.
@@ -149,9 +51,6 @@ function getClosestInstanceFromNode(node) {
   }
   for (; node && (inst = node[internalInstanceKey]); node = parents.pop()) {
     closest = inst;
-    if (parents.length) {
-      precacheChildNodes(inst, node);
-    }
   }
 
   return closest;
@@ -166,18 +65,11 @@ function getInstanceFromNode(node) {
   if (inst) {
     if (inst.tag === HostComponent || inst.tag === HostText) {
       return inst;
-    } else if (inst._hostNode === node) {
-      return inst;
     } else {
       return null;
     }
   }
-  inst = getClosestInstanceFromNode(node);
-  if (inst != null && inst._hostNode === node) {
-    return inst;
-  } else {
-    return null;
-  }
+  return null;
 }
 
 /**
@@ -193,33 +85,7 @@ function getNodeFromInstance(inst) {
 
   // Without this first invariant, passing a non-DOM-component triggers the next
   // invariant for a missing parent, which is super confusing.
-  invariant(
-    inst._hostNode !== undefined,
-    'getNodeFromInstance: Invalid argument.',
-  );
-
-  if (inst._hostNode) {
-    return inst._hostNode;
-  }
-
-  // Walk up the tree until we find an ancestor whose DOM node we have cached.
-  var parents = [];
-  while (!inst._hostNode) {
-    parents.push(inst);
-    invariant(
-      inst._hostParent,
-      'React DOM tree root should always have a node reference.',
-    );
-    inst = inst._hostParent;
-  }
-
-  // Now parents contains each ancestor that does *not* have a cached native
-  // node, and `inst` is the deepest ancestor that does.
-  for (; parents.length; inst = parents.pop()) {
-    precacheChildNodes(inst, inst._hostNode);
-  }
-
-  return inst._hostNode;
+  invariant(false, 'getNodeFromInstance: Invalid argument.');
 }
 
 function getFiberCurrentPropsFromNode(node) {
@@ -234,9 +100,6 @@ var ReactDOMComponentTree = {
   getClosestInstanceFromNode: getClosestInstanceFromNode,
   getInstanceFromNode: getInstanceFromNode,
   getNodeFromInstance: getNodeFromInstance,
-  precacheChildNodes: precacheChildNodes,
-  precacheNode: precacheNode,
-  uncacheNode: uncacheNode,
   precacheFiberNode: precacheFiberNode,
   getFiberCurrentPropsFromNode,
   updateFiberProps,

src/renderers/dom/shared/ReactDOMComponentTree.js

@@ -31,22 +31,14 @@ function findRootContainerNode(inst) {
   // TODO: It may be a good idea to cache this to prevent unnecessary DOM
   // traversal, but caching is difficult to do correctly without using a
   // mutation observer to listen for all DOM changes.
-  if (typeof inst.tag === 'number') {
-    while (inst.return) {
-      inst = inst.return;
-    }
-    if (inst.tag !== HostRoot) {
-      // This can happen if we're in a detached tree.
-      return null;
-    }
-    return inst.stateNode.containerInfo;
-  } else {
-    while (inst._hostParent) {
-      inst = inst._hostParent;
-    }
-    var rootNode = ReactDOMComponentTree.getNodeFromInstance(inst);
-    return rootNode.parentNode;
+  while (inst.return) {
+    inst = inst.return;
+  }
+  if (inst.tag !== HostRoot) {
+    // This can happen if we're in a detached tree.
+    return null;
   }
+  return inst.stateNode.containerInfo;
 }
 
 // Used to store ancestor hierarchy in top level callback

src/renderers/dom/shared/ReactDOMEventListener.js

@@ -11,7 +11,6 @@
 
 var ReactDOMFeatureFlags = {
   fiberAsyncScheduling: false,
-  useFiber: true,
 };
 
 module.exports = ReactDOMFeatureFlags;

src/renderers/dom/shared/ReactDOMFeatureFlags.js

@@ -23,17 +23,11 @@ describe('ReactDOMComponentTree', () => {
   }
 
   function getTypeOf(instance) {
-    if (typeof instance.tag === 'number') {
-      return instance.type;
-    }
-    return instance._currentElement.type;
+    return instance.type;
   }
 
   function getTextOf(instance) {
-    if (typeof instance.tag === 'number') {
-      return instance.memoizedProps;
-    }
-    return instance._stringText;
+    return instance.memoizedProps;
   }
 
   beforeEach(() => {

src/renderers/dom/shared/__tests__/ReactDOMComponentTree-test.js

@@ -34,7 +34,7 @@ import type {
   DispatchConfig,
   ReactSyntheticEvent,
 } from 'ReactSyntheticEventType';
-import type {ReactInstance} from 'ReactInstanceType';
+import type {Fiber} from 'ReactFiber';
 import type {EventTypes, PluginModule} from 'PluginModuleType';
 
 /**
@@ -179,7 +179,7 @@ var SimpleEventPlugin: PluginModule<MouseEvent> = {
 
   extractEvents: function(
     topLevelType: TopLevelTypes,
-    targetInst: ReactInstance,
+    targetInst: Fiber,
     nativeEvent: MouseEvent,
     nativeEventTarget: EventTarget,
   ): null | ReactSyntheticEvent {