From 9cdd0db13879cd806a3b0dece8550835328f3d41 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Dawid=20Ma=C5=82ecki?= Date: Tue, 27 May 2025 03:10:14 -0700 Subject: [PATCH] Remove JSEventLoopWatchdog from react-native package (#51614) Summary: Pull Request resolved: https://github.com/facebook/react-native/pull/51614 `JSEventLoopWatchdog` is not used in react-native package. This diff moves it to rn-tester which previously deep imported it from react-native (which we want to avoid). Changelog: [Internal] Reviewed By: huntie Differential Revision: D75410548 fbshipit-source-id: d4996742578e3b068e7acad9479394388b1907ac --- .../__snapshots__/public-api-test.js.snap | 16 ---------------- .../js/utils}/JSEventLoopWatchdog.js | 0 packages/rn-tester/js/utils/useJsStalls.js | 3 +-- 3 files changed, 1 insertion(+), 18 deletions(-) rename packages/{react-native/Libraries/Interaction => rn-tester/js/utils}/JSEventLoopWatchdog.js (100%) diff --git a/packages/react-native/Libraries/__tests__/__snapshots__/public-api-test.js.snap b/packages/react-native/Libraries/__tests__/__snapshots__/public-api-test.js.snap index 8ace056326d..b8d5903830c 100644 --- a/packages/react-native/Libraries/__tests__/__snapshots__/public-api-test.js.snap +++ b/packages/react-native/Libraries/__tests__/__snapshots__/public-api-test.js.snap @@ -4722,22 +4722,6 @@ declare export default typeof InteractionManagerStub; " `; -exports[`public API should not change unintentionally Libraries/Interaction/JSEventLoopWatchdog.js 1`] = ` -"type Handler = { - onIterate?: () => void, - onStall: (params: { lastInterval: number, busyTime: number, ... }) => ?string, - ... -}; -declare const JSEventLoopWatchdog: { - getStats: () => Object, - reset: () => void, - addHandler: (handler: Handler) => void, - install: ($$PARAM_0$$: { thresholdMS: number, ... }) => void, -}; -declare export default typeof JSEventLoopWatchdog; -" -`; - exports[`public API should not change unintentionally Libraries/Interaction/NativeFrameRateLogger.js 1`] = ` "export * from \\"../../src/private/specs_DEPRECATED/modules/NativeFrameRateLogger\\"; declare export default typeof NativeFrameRateLogger; diff --git a/packages/react-native/Libraries/Interaction/JSEventLoopWatchdog.js b/packages/rn-tester/js/utils/JSEventLoopWatchdog.js similarity index 100% rename from packages/react-native/Libraries/Interaction/JSEventLoopWatchdog.js rename to packages/rn-tester/js/utils/JSEventLoopWatchdog.js diff --git a/packages/rn-tester/js/utils/useJsStalls.js b/packages/rn-tester/js/utils/useJsStalls.js index a59dcc8c6cb..e56a7e9f480 100644 --- a/packages/rn-tester/js/utils/useJsStalls.js +++ b/packages/rn-tester/js/utils/useJsStalls.js @@ -56,8 +56,7 @@ const useJsStalls = (): ({ }, []); const onEnableJsStallsTracking = useCallback(() => { - const JSEventLoopWatchdog = - require('react-native/Libraries/Interaction/JSEventLoopWatchdog').default; + const JSEventLoopWatchdog = require('./JSEventLoopWatchdog').default; JSEventLoopWatchdog.install({thresholdMS: 25});