Rename JSEngineInstance -> JSRuntimeFactory (facebook#41548)

Summary:
Pull Request resolved: facebook#41548

`JSEngineInstance` is a misnomer - this interface actually creates `jsi::Runtime`s and doesn't represent an "instance of a JS engine". This diff renames it to `JSRuntimeFactory`.

Changelog: [Internal]

Reviewed By: huntie, arushikesarwani94

Differential Revision: D51447882

fbshipit-source-id: e118fe5c202607500a62d8e15afec088c4946969

Author: motiz88

packages/react-native/Libraries/AppDelegate/RCTAppDelegate.mm

@@ -38,7 +38,7 @@
 #import <react/config/ReactNativeConfig.h>
 #import <react/renderer/runtimescheduler/RuntimeScheduler.h>
 #import <react/renderer/runtimescheduler/RuntimeSchedulerCallInvoker.h>
-#import <react/runtime/JSEngineInstance.h>
+#import <react/runtime/JSRuntimeFactory.h>
 
 static NSString *const kRNConcurrentRoot = @"concurrentRoot";
 
@@ -293,8 +293,8 @@ - (void)createReactHost
   _reactHost = [[RCTHost alloc] initWithBundleURL:[self bundleURL]
                                      hostDelegate:nil
                        turboModuleManagerDelegate:self
-                                 jsEngineProvider:^std::shared_ptr<facebook::react::JSEngineInstance>() {
-                                   return [weakSelf createJSEngineInstance];
+                                 jsEngineProvider:^std::shared_ptr<facebook::react::JSRuntimeFactory>() {
+                                   return [weakSelf createJSRuntimeFactory];
                                  }];
   [_reactHost setBundleURLProvider:^NSURL *() {
     return [weakSelf bundleURL];
@@ -303,7 +303,7 @@ - (void)createReactHost
   [_reactHost start];
 }
 
-- (std::shared_ptr<facebook::react::JSEngineInstance>)createJSEngineInstance
+- (std::shared_ptr<facebook::react::JSRuntimeFactory>)createJSRuntimeFactory
 {
 #if RCT_USE_HERMES
   return std::make_shared<facebook::react::RCTHermesInstance>(_reactNativeConfig, nullptr);

packages/react-native/ReactAndroid/src/main/java/com/facebook/react/defaults/DefaultReactHost.kt

@@ -54,13 +54,13 @@ object DefaultReactHost {
     if (reactHost == null) {
       val jsBundleLoader =
           JSBundleLoader.createAssetLoader(context, "assets://$jsBundleAssetPath", true)
-      val jsEngineInstance = if (isHermesEnabled) HermesInstance() else JSCInstance()
+      val jsRuntimeFactory = if (isHermesEnabled) HermesInstance() else JSCInstance()
       val defaultReactHostDelegate =
           DefaultReactHostDelegate(
               jsMainModulePath = jsMainModulePath,
               jsBundleLoader = jsBundleLoader,
               reactPackages = packageList,
-              jsEngineInstance = jsEngineInstance,
+              jsRuntimeFactory = jsRuntimeFactory,
               turboModuleManagerDelegateBuilder = DefaultTurboModuleManagerDelegate.Builder())
       val reactJsExceptionHandler = ReactJsExceptionHandler { _ -> }
       val componentFactory = ComponentFactory()

packages/react-native/ReactAndroid/src/main/java/com/facebook/react/defaults/DefaultReactHostDelegate.kt

@@ -14,7 +14,7 @@ import com.facebook.react.bridge.JSBundleLoader
 import com.facebook.react.common.annotations.UnstableReactNativeAPI
 import com.facebook.react.fabric.ReactNativeConfig
 import com.facebook.react.runtime.BindingsInstaller
-import com.facebook.react.runtime.JSEngineInstance
+import com.facebook.react.runtime.JSRuntimeFactory
 import com.facebook.react.runtime.ReactHostDelegate
 import com.facebook.react.runtime.hermes.HermesInstance
 
@@ -28,7 +28,7 @@ import com.facebook.react.runtime.hermes.HermesInstance
  * @param jsBundleLoader Bundle loader to use when setting up JS environment. <p>Example:
  *   [JSBundleLoader.createFileLoader(application, bundleFile)]
  * @param reactPackages list of reactPackages to expose Native Modules and View Components to JS
- * @param jsEngineInstance Object that holds a native reference to the javascript engine
+ * @param jsRuntimeFactory Object that holds a native reference to the JS Runtime factory
  * @param bindingsInstaller Object that holds a native C++ references that allow host applications
  *   to install C++ objects into jsi::Runtime during the initialization of React Native
  * @param reactNativeConfig ReactNative Configuration that allows to customize the behavior of
@@ -42,7 +42,7 @@ class DefaultReactHostDelegate(
     override val jsMainModulePath: String,
     override val jsBundleLoader: JSBundleLoader,
     override val reactPackages: List<ReactPackage> = emptyList(),
-    override val jsEngineInstance: JSEngineInstance = HermesInstance(),
+    override val jsRuntimeFactory: JSRuntimeFactory = HermesInstance(),
     override val bindingsInstaller: BindingsInstaller = DefaultBindingsInstaller(),
     private val reactNativeConfig: ReactNativeConfig = ReactNativeConfig.DEFAULT_CONFIG,
     private val exceptionHandler: (Exception) -> Unit = {},

packages/react-native/ReactAndroid/src/main/java/com/facebook/react/runtime/JSCInstance.java

@@ -13,7 +13,7 @@
 import com.facebook.soloader.SoLoader;
 
 @Nullsafe(Nullsafe.Mode.LOCAL)
-public class JSCInstance extends JSEngineInstance {
+public class JSCInstance extends JSRuntimeFactory {
   static {
     SoLoader.loadLibrary("jscinstance");
   }

packages/react-native/ReactAndroid/src/main/java/com/facebook/react/runtime/JSEngineInstance.java renamed to packages/react-native/ReactAndroid/src/main/java/com/facebook/react/runtime/JSRuntimeFactory.java

@@ -13,14 +13,14 @@
 import com.facebook.soloader.SoLoader;
 
 @Nullsafe(Nullsafe.Mode.LOCAL)
-public abstract class JSEngineInstance {
+public abstract class JSRuntimeFactory {
   static {
     SoLoader.loadLibrary("rninstance");
   }
 
   @DoNotStrip private final HybridData mHybridData;
 
-  protected JSEngineInstance(HybridData hybridData) {
+  protected JSRuntimeFactory(HybridData hybridData) {
     mHybridData = hybridData;
   }
 }

packages/react-native/ReactAndroid/src/main/java/com/facebook/react/runtime/ReactHostDelegate.kt

@@ -38,7 +38,7 @@ interface ReactHostDelegate {
   val reactPackages: List<ReactPackage>
 
   /** Object that holds a native reference to the javascript engine */
-  val jsEngineInstance: JSEngineInstance
+  val jsRuntimeFactory: JSRuntimeFactory
 
   /**
    * Bundle loader to use when setting up JS environment. <p>Example:
@@ -68,7 +68,7 @@ interface ReactHostDelegate {
   class ReactHostDelegateBase(
       override val jsMainModulePath: String,
       override val jsBundleLoader: JSBundleLoader,
-      override val jsEngineInstance: JSEngineInstance,
+      override val jsRuntimeFactory: JSRuntimeFactory,
       override val turboModuleManagerDelegateBuilder:
           ReactPackageTurboModuleManagerDelegate.Builder,
       override val reactPackages: List<ReactPackage> = emptyList(),

packages/react-native/ReactAndroid/src/main/java/com/facebook/react/runtime/ReactInstance.java

@@ -167,7 +167,7 @@ public void onHostDestroy() {
           }
         });
 
-    JSEngineInstance jsEngineInstance = mDelegate.getJsEngineInstance();
+    JSRuntimeFactory jsRuntimeFactory = mDelegate.getJsRuntimeFactory();
     BindingsInstaller bindingsInstaller = mDelegate.getBindingsInstaller();
     // Notify JS if profiling is enabled
     boolean isProfiling =
@@ -177,7 +177,7 @@ public void onHostDestroy() {
     boolean useModernRuntimeScheduler = ReactFeatureFlags.useModernRuntimeScheduler;
     mHybridData =
         initHybrid(
-            jsEngineInstance,
+            jsRuntimeFactory,
             jsMessageQueueThread,
             nativeModulesMessageQueueThread,
             mJavaTimerManager,
@@ -432,7 +432,7 @@ public Collection<NativeModule> getNativeModules() {
 
   @DoNotStrip
   private native HybridData initHybrid(
-      JSEngineInstance jsEngineInstance,
+      JSRuntimeFactory jsRuntimeFactory,
       MessageQueueThread jsMessageQueueThread,
       MessageQueueThread nativeModulesMessageQueueThread,
       JavaTimerManager timerManager,

packages/react-native/ReactAndroid/src/main/java/com/facebook/react/runtime/hermes/HermesInstance.kt

@@ -10,11 +10,11 @@ package com.facebook.react.runtime.hermes
 import com.facebook.jni.HybridData
 import com.facebook.jni.annotations.DoNotStrip
 import com.facebook.react.fabric.ReactNativeConfig
-import com.facebook.react.runtime.JSEngineInstance
+import com.facebook.react.runtime.JSRuntimeFactory
 import com.facebook.soloader.SoLoader
 
 class HermesInstance constructor(reactNativeConfig: ReactNativeConfig?) :
-    JSEngineInstance(initHybrid(reactNativeConfig as Any?)) {
+    JSRuntimeFactory(initHybrid(reactNativeConfig as Any?)) {
 
   constructor() : this(null)
 

packages/react-native/ReactAndroid/src/main/jni/react/runtime/hermes/jni/JHermesInstance.h

@@ -14,14 +14,14 @@
 #include <jni.h>
 #include <jsi/jsi.h>
 #include <react/config/ReactNativeConfig.h>
-#include <react/runtime/JSEngineInstance.h>
+#include <react/runtime/JSRuntimeFactory.h>
 #include <react/runtime/hermes/HermesInstance.h>
-#include "../../jni/JJSEngineInstance.h"
+#include "../../jni/JJSRuntimeFactory.h"
 
 namespace facebook::react {
 
 class JHermesInstance
-    : public jni::HybridClass<JHermesInstance, JJSEngineInstance> {
+    : public jni::HybridClass<JHermesInstance, JJSRuntimeFactory> {
  public:
   static constexpr auto kJavaDescriptor =
       "Lcom/facebook/react/runtime/hermes/HermesInstance;";

packages/react-native/ReactAndroid/src/main/jni/react/runtime/jni/JJSEngineInstance.h renamed to packages/react-native/ReactAndroid/src/main/jni/react/runtime/jni/JJSRuntimeFactory.h

@@ -9,15 +9,15 @@
 
 #include <fbjni/fbjni.h>
 #include <jni.h>
-#include <react/runtime/JSEngineInstance.h>
+#include <react/runtime/JSRuntimeFactory.h>
 
 namespace facebook::react {
 
-class JJSEngineInstance : public jni::HybridClass<JJSEngineInstance>,
-                          public JSEngineInstance {
+class JJSRuntimeFactory : public jni::HybridClass<JJSRuntimeFactory>,
+                          public JSRuntimeFactory {
  public:
   static auto constexpr kJavaDescriptor =
-      "Lcom/facebook/react/runtime/JSEngineInstance;";
+      "Lcom/facebook/react/runtime/JSRuntimeFactory;";
 
  private:
   friend HybridBase;