Merge pull request Expensify#68834 from software-mansion-labs/fix/animated-memory-leak

Fix memory leak with Animated API

Author: mountiny

patches/react-native-web/details.md

@@ -119,3 +119,14 @@
 - Upstream PR/issue: The patch isn't something we can apply to upstream because we are specifically allowing custom props used by Fullstory to be forwarded to their respective DOM elements.
 - E/App issue: As explained above the current solution can't be applied to upstream because it's tailored to Fullstory needs.
 - PR introducing patch: https://github.com/Expensify/App/pull/67552
+
+### [react-native-web+0.20.0+011+fix-memory-leak.patch](react-native-web+0.20.0+011+fix-memory-leak.patch)
+- Reason:
+  
+    ```
+    Fixes memory leak on web caused by Animated API used in @react-navigation package
+    ```
+  
+- Upstream PR/issue: https://github.com/necolas/react-native-web/pull/2800
+- E/App issue: https://github.com/Expensify/App/issues/65820
+- PR introducing patch: https://github.com/Expensify/App/pull/68834

patches/react-native-web/react-native-web+0.20.0+011+fix-memory-leak.patch

@@ -0,0 +1,12 @@
+diff --git a/node_modules/react-native-web/dist/vendor/react-native/Animated/nodes/AnimatedProps.js b/node_modules/react-native-web/dist/vendor/react-native/Animated/nodes/AnimatedProps.js
+index 9ba3336..2d67005 100644
+--- a/node_modules/react-native-web/dist/vendor/react-native/Animated/nodes/AnimatedProps.js
++++ b/node_modules/react-native-web/dist/vendor/react-native/Animated/nodes/AnimatedProps.js
+@@ -68,6 +68,7 @@ class AnimatedProps extends AnimatedNode {
+     if (this.__isNative && this._animatedView) {
+       this.__disconnectAnimatedView();
+     }
++    this._animatedView = null
+     for (var key in this._props) {
+       var value = this._props[key];
+       if (value instanceof AnimatedNode) {