Add Flow type to new host config method

Author: acdlite

packages/react-art/src/ReactARTHostConfig.js

@@ -472,6 +472,6 @@ export function preparePortalMount(portalInstance: any): void {
   // noop
 }
 
-export function unmountNode(node: any): void {
+export function detachDeletedInstance(node: Instance): void {
   // noop
 }

packages/react-dom/src/client/ReactDOMComponentTree.js

@@ -43,12 +43,11 @@ const internalEventHandlersKey = '__reactEvents$' + randomKey;
 const internalEventHandlerListenersKey = '__reactListeners$' + randomKey;
 const internalEventHandlesSetKey = '__reactHandles$' + randomKey;
 
-export function unmountNode(
-  node: Instance | TextInstance | SuspenseInstance | ReactScopeInstance,
-): void {
+export function detachDeletedInstance(node: Instance): void {
+  // TODO: This function is only called on host components. I don't think all of
+  // these fields are relevant.
   delete (node: any)[internalInstanceKey];
   delete (node: any)[internalPropsKey];
-  delete (node: any)[internalContainerInstanceKey];
   delete (node: any)[internalEventHandlersKey];
   delete (node: any)[internalEventHandlerListenersKey];
   delete (node: any)[internalEventHandlesSetKey];

packages/react-dom/src/client/ReactDOMHostConfig.js

@@ -24,8 +24,8 @@ import {
   getFiberFromScopeInstance,
   getInstanceFromNode as getInstanceFromNodeDOMTree,
   isContainerMarkedAsRoot,
-  unmountNode as unmountNodeFromDOMTree,
 } from './ReactDOMComponentTree';
+export {detachDeletedInstance} from './ReactDOMComponentTree';
 import {hasRole} from './DOMAccessibilityRoles';
 import {
   createElement,
@@ -1210,7 +1210,3 @@ export function setupIntersectionObserver(
     },
   };
 }
-
-export function unmountNode(node: any): void {
-  unmountNodeFromDOMTree(node);
-}

packages/react-native-renderer/src/ReactFabricHostConfig.js

@@ -482,6 +482,6 @@ export function preparePortalMount(portalInstance: Instance): void {
   // noop
 }
 
-export function unmountNode(node: any): void {
+export function detachDeletedInstance(node: Instance): void {
   // noop
 }

packages/react-native-renderer/src/ReactNativeHostConfig.js

@@ -539,6 +539,6 @@ export function preparePortalMount(portalInstance: Instance): void {
   // noop
 }
 
-export function unmountNode(node: any): void {
+export function detachDeletedInstance(node: Instance): void {
   // noop
 }

packages/react-noop-renderer/src/createReactNoop.js

@@ -421,7 +421,7 @@ function createReactNoop(reconciler: Function, useMutation: boolean) {
       throw new Error('Not yet implemented.');
     },
 
-    unmountNode() {},
+    detachDeletedInstance() {},
   };
 
   const hostConfig = useMutation

packages/react-reconciler/src/ReactFiberCommitWork.new.js

@@ -61,7 +61,7 @@ import {
   hasCaughtError,
   clearCaughtError,
 } from 'shared/ReactErrorUtils';
-import {unmountNode} from './ReactFiberHostConfig';
+import {detachDeletedInstance} from './ReactFiberHostConfig';
 import {
   NoFlags,
   ContentReset,
@@ -1285,7 +1285,7 @@ function detachFiberAfterEffects(fiber: Fiber) {
       if (fiber.tag === HostComponent) {
         const hostInstance: Instance = fiber.stateNode;
         if (hostInstance !== null) {
-          unmountNode(hostInstance);
+          detachDeletedInstance(hostInstance);
         }
       }
 

packages/react-reconciler/src/ReactFiberCommitWork.old.js

@@ -61,7 +61,7 @@ import {
   hasCaughtError,
   clearCaughtError,
 } from 'shared/ReactErrorUtils';
-import {unmountNode} from './ReactFiberHostConfig';
+import {detachDeletedInstance} from './ReactFiberHostConfig';
 import {
   NoFlags,
   ContentReset,
@@ -1285,7 +1285,7 @@ function detachFiberAfterEffects(fiber: Fiber) {
       if (fiber.tag === HostComponent) {
         const hostInstance: Instance = fiber.stateNode;
         if (hostInstance !== null) {
-          unmountNode(hostInstance);
+          detachDeletedInstance(hostInstance);
         }
       }
 

packages/react-test-renderer/src/ReactTestHostConfig.js

@@ -361,6 +361,6 @@ export function getInstanceFromScope(scopeInstance: Object): null | Object {
   return nodeToInstanceMap.get(scopeInstance) || null;
 }
 
-export function unmountNode(node: any): void {
+export function detachDeletedInstance(node: Instance): void {
   // noop
 }