From 2abe623d9ff25530bf23421113f0daaf50a73fef Mon Sep 17 00:00:00 2001 From: Tim Yung Date: Wed, 26 Mar 2025 08:07:53 -0700 Subject: [PATCH] Animated: Omit `allowlist` in `avoidStateUpdateInAnimatedPropsMemo` Experiment (#50232) Summary: Pull Request resolved: https://github.com/facebook/react-native/pull/50232 While reviewing the `avoidStateUpdateInAnimatedPropsMemo` experiment, I noticed that the control and test groups were invoking `areCompositeKeysEqual` differently: - Test group was passing in `allowlist`. - Control group was not passing in `allowlist`. Passing it in is technically more correct, but let's restore the control group behavior for now to isolate the tested changes. Changelog: [Internal] Reviewed By: javache Differential Revision: D71746745 fbshipit-source-id: aa34db0532ef53ead08bbd632aab0994b31b9340 --- .../src/private/animated/createAnimatedPropsMemoHook.js | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/packages/react-native/src/private/animated/createAnimatedPropsMemoHook.js b/packages/react-native/src/private/animated/createAnimatedPropsMemoHook.js index 9d97a395e4f..4887e712d3f 100644 --- a/packages/react-native/src/private/animated/createAnimatedPropsMemoHook.js +++ b/packages/react-native/src/private/animated/createAnimatedPropsMemoHook.js @@ -91,8 +91,7 @@ export function createAnimatedPropsMemoHook( const prev = prevRef.current; const next = - prev != null && - areCompositeKeysEqual(prev.compositeKey, compositeKey, allowlist) + prev != null && areCompositeKeysEqual(prev.compositeKey, compositeKey) ? prev : { compositeKey,