From 4f7b4d8cbbb957e3cac606aa07e157719e247f32 Mon Sep 17 00:00:00 2001 From: David Vacca Date: Tue, 18 Oct 2022 13:32:03 -0700 Subject: [PATCH] Ship enableAggressiveEventEmitterCleanup Summary: This diff ships the experiment enableAggressiveEventEmitterCleanup changelog: [internal] internal Reviewed By: sammy-SC Differential Revision: D40356592 fbshipit-source-id: 27da82a0b958cedc1283a54de175a6eb6b14004b --- .../java/com/facebook/react/config/ReactFeatureFlags.java | 2 -- .../react/fabric/mounting/SurfaceMountingManager.java | 8 +++----- 2 files changed, 3 insertions(+), 7 deletions(-) diff --git a/ReactAndroid/src/main/java/com/facebook/react/config/ReactFeatureFlags.java b/ReactAndroid/src/main/java/com/facebook/react/config/ReactFeatureFlags.java index 133bba6a9f5..7a3e4959673 100644 --- a/ReactAndroid/src/main/java/com/facebook/react/config/ReactFeatureFlags.java +++ b/ReactAndroid/src/main/java/com/facebook/react/config/ReactFeatureFlags.java @@ -82,8 +82,6 @@ public class ReactFeatureFlags { /** Feature Flag to use overflowInset values provided by Yoga */ public static boolean useOverflowInset = false; - public static boolean enableAggressiveEventEmitterCleanup = false; - public static boolean insertZReorderBarriersOnViewGroupChildren = true; /** diff --git a/ReactAndroid/src/main/java/com/facebook/react/fabric/mounting/SurfaceMountingManager.java b/ReactAndroid/src/main/java/com/facebook/react/fabric/mounting/SurfaceMountingManager.java index a7abe4ec6b0..804a77ade78 100644 --- a/ReactAndroid/src/main/java/com/facebook/react/fabric/mounting/SurfaceMountingManager.java +++ b/ReactAndroid/src/main/java/com/facebook/react/fabric/mounting/SurfaceMountingManager.java @@ -283,11 +283,9 @@ public class SurfaceMountingManager { viewState.mStateWrapper.destroyState(); viewState.mStateWrapper = null; } - if (ReactFeatureFlags.enableAggressiveEventEmitterCleanup) { - if (viewState.mEventEmitter != null) { - viewState.mEventEmitter.destroy(); - viewState.mEventEmitter = null; - } + if (viewState.mEventEmitter != null) { + viewState.mEventEmitter.destroy(); + viewState.mEventEmitter = null; } }