Delete ReactFeatureFlags.enableReactContextCleanupFix

Summary:
This diff deletes the flag ReactFeatureFlags.enableReactContextCleanupFix, the flag was disabled for many months, I just disable it.

changelog: [internal] internal

Reviewed By: genkikondo, makovkastar

Differential Revision: D33781628

fbshipit-source-id: 4b5e22adf9d30da5b85bbbde8bdc98d98f5e8891

ReactAndroid/src/main/java/com/facebook/react/bridge/JSIModuleRegistry.java

@@ -8,7 +8,6 @@
 package com.facebook.react.bridge;
 
 import com.facebook.infer.annotation.Assertions;
-import com.facebook.react.config.ReactFeatureFlags;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
@@ -45,8 +44,5 @@ public class JSIModuleRegistry {
       JSIModuleHolder moduleHolder = entry.getValue();
       moduleHolder.notifyJSInstanceDestroy();
     }
-    if (ReactFeatureFlags.enableReactContextCleanupFix) {
-      mModules.clear();
-    }
   }
 }

ReactAndroid/src/main/java/com/facebook/react/bridge/NativeModuleRegistry.java

@@ -107,9 +107,6 @@ public class NativeModuleRegistry {
       for (ModuleHolder module : mModules.values()) {
         module.destroy();
       }
-      if (ReactFeatureFlags.enableReactContextCleanupFix) {
-        mModules.clear();
-      }
     } finally {
       Systrace.endSection(Systrace.TRACE_TAG_REACT_JAVA_BRIDGE);
     }

ReactAndroid/src/main/java/com/facebook/react/bridge/ReactContext.java

@@ -23,7 +23,6 @@ import com.facebook.react.bridge.queue.MessageQueueThread;
 import com.facebook.react.bridge.queue.ReactQueueConfiguration;
 import com.facebook.react.common.LifecycleState;
 import com.facebook.react.common.ReactConstants;
-import com.facebook.react.config.ReactFeatureFlags;
 import java.lang.ref.WeakReference;
 import java.util.concurrent.CopyOnWriteArraySet;
 
@@ -320,11 +319,6 @@ public class ReactContext extends ContextWrapper {
     if (mCatalystInstance != null) {
       mCatalystInstance.destroy();
     }
-    if (ReactFeatureFlags.enableReactContextCleanupFix) {
-      mLifecycleEventListeners.clear();
-      mActivityEventListeners.clear();
-      mWindowFocusEventListeners.clear();
-    }
   }
 
   /** Should be called by the hosting Fragment in {@link Fragment#onActivityResult} */

ReactAndroid/src/main/java/com/facebook/react/config/ReactFeatureFlags.java

@@ -71,9 +71,6 @@ public class ReactFeatureFlags {
 
   public static boolean enableRuntimeSchedulerInTurboModule = false;
 
-  /** Enables a more aggressive cleanup during destruction of ReactContext */
-  public static boolean enableReactContextCleanupFix = false;
-
   /** Feature flag to configure eager attachment of the root view/initialisation of the JS code */
   public static boolean enableEagerRootViewAttachment = false;
 

ReactAndroid/src/main/java/com/facebook/react/uimanager/UIManagerModule.java

@@ -234,9 +234,6 @@ public class UIManagerModule extends ReactContextBaseJavaModule
     mUIImplementation.onCatalystInstanceDestroyed();
 
     ReactApplicationContext reactApplicationContext = getReactApplicationContext();
-    if (ReactFeatureFlags.enableReactContextCleanupFix) {
-      reactApplicationContext.removeLifecycleEventListener(this);
-    }
     reactApplicationContext.unregisterComponentCallbacks(mMemoryTrimCallback);
     YogaNodePool.get().clear();
     ViewManagerPropertyUpdater.clear();