From 3b597c0576977773910c77e075cc6d6308decb04 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sebastian=20Markb=C3=A5ge?= Date: Tue, 10 Dec 2024 11:24:59 -0500 Subject: [PATCH] Clean up findFiberByHostInstance from DevTools Hook (#31711) The need for this was removed in https://github.com/facebook/react/pull/30831 Since the new DevTools version has been released for a while and we expect people to more or less auto-update. Future versions of React don't need this. Once we remove the remaining uses of `getInstanceFromNode` e.g. in the deprecated internal `findDOMNode`/`findNodeHandle` and the event system, we can completely remove the tagging of DOM nodes. --- packages/react-reconciler/src/ReactFiberReconciler.js | 2 -- 1 file changed, 2 deletions(-) diff --git a/packages/react-reconciler/src/ReactFiberReconciler.js b/packages/react-reconciler/src/ReactFiberReconciler.js index 94f1397eb129e..31a7fbeccf36b 100644 --- a/packages/react-reconciler/src/ReactFiberReconciler.js +++ b/packages/react-reconciler/src/ReactFiberReconciler.js @@ -42,7 +42,6 @@ import {enableSchedulingProfiler} from 'shared/ReactFeatureFlags'; import ReactSharedInternals from 'shared/ReactSharedInternals'; import { getPublicInstance, - getInstanceFromNode, rendererVersion, rendererPackageName, extraDevToolsConfig, @@ -847,7 +846,6 @@ export function injectIntoDevTools(): boolean { version: rendererVersion, rendererPackageName: rendererPackageName, currentDispatcherRef: ReactSharedInternals, - findFiberByHostInstance: getInstanceFromNode, // Enables DevTools to detect reconciler version rather than renderer version // which may not match for third party renderers. reconcilerVersion: ReactVersion,