fix: disable shared global context and fix reanimated hanging issue (#210)

Kudo · web-flow · commit 8733225d76aa · 2024-08-21T02:05:56.000+08:00
# Why fixes #208 # How - the reanimated problem was originally mentioned at software-mansion/react-native-reanimated#4953. since reanimated doesn't use the shared global context. this pr just comment out the code and unblock the reanimated integration. - also did some refactoring to remove outdated code
diff --git a/android/build.gradle b/android/build.gradle
@@ -23,7 +23,7 @@ File findNodePackageDir(String packageName, boolean absolute = true) {
   return absolute ? dir.getAbsoluteFile() : dir
 }
 
-String toPlatformFileString(File path) {
+static String toPlatformFileString(File path) {
   def result = path.toString()
   if (Os.isFamily(Os.FAMILY_WINDOWS)) {
     result = result.replace(File.separatorChar, '/' as char)
@@ -89,6 +89,7 @@ if (v8AndroidVersionMajor < 11) {
 ext.CODECACHE_MODE_NONE = 0
 ext.CODECACHE_MODE_NORMAL = 1
 ext.CODECACHE_MODE_STUB_BUNDLE = 2
+
 def parseCacheMode(cacheMode) {
   switch (cacheMode) {
     case null:
@@ -118,6 +119,7 @@ def reactNativeArchitectures() {
 apply plugin: "com.android.library"
 
 android {
+  namespace "io.csie.kudo.reactnative.v8"
   compileSdkVersion safeExtGet("compileSdkVersion", 33)
 
   if (rootProject.hasProperty("ndkPath")) {
@@ -206,20 +208,28 @@ android {
   }
 }
 
-task cleanCmakeCache() {
-  tasks.getByName("clean").dependsOn(cleanCmakeCache)
+def cleanCmakeCache = tasks.register('cleanCmakeCache') {
   doFirst {
     delete "${projectDir}/.cxx"
   }
 }
+tasks.named("clean").configure {
+  dependsOn(cleanCmakeCache)
+}
 
-task prepareHeadersForPrefab(type: Copy) {
+tasks.register('prepareHeadersForPrefabV8Runtime', Copy) {
   from("${projectDir}/../src/v8runtime")
   include("**/*.h")
   into(file("${prefabHeadersDir}/v8runtime"))
 }
 
-task extractSOFiles {
+tasks.register('prepareHeadersForPrefabV8', Copy) {
+  from("${v8AndroidDir}/dist/include")
+  include("**/*.h")
+  into(file("${prefabHeadersDir}"))
+}
+
+tasks.register('extractSOFiles') {
   doLast {
     configurations.extractSO.files.each {
       def file = it.absoluteFile
@@ -259,10 +269,15 @@ def nativeBuildDependsOn(dependsOnTask, buildTypesIncludes) {
 }
 
 afterEvaluate {
-  preBuild.dependsOn(prepareHeadersForPrefab)
+  preBuild.dependsOn(prepareHeadersForPrefabV8Runtime)
+  preBuild.dependsOn(prepareHeadersForPrefabV8)
 
   nativeBuildDependsOn(extractSOFiles, null)
 
-  tasks.findByName("mergeReleaseJniLibFolders")?.dependsOn(extractSOFiles)
-  tasks.findByName("mergeDebugJniLibFolders")?.dependsOn(extractSOFiles)
+  tasks.named("mergeReleaseJniLibFolders")?.configure {
+    dependsOn(extractSOFiles)
+  }
+  tasks.named("mergeDebugJniLibFolders")?.configure {
+    dependsOn(extractSOFiles)
+  }
 }
diff --git a/android/src/main/AndroidManifest.xml b/android/src/main/AndroidManifest.xml
@@ -1,5 +1,2 @@
-
-<manifest xmlns:android="http://schemas.android.com/apk/res/android"
-          package="io.csie.kudo.reactnative.v8">
-
+<manifest xmlns:android="http://schemas.android.com/apk/res/android">
 </manifest>
diff --git a/android/src/main/java/io/csie/kudo/reactnative/v8/executor/V8Executor.java b/android/src/main/java/io/csie/kudo/reactnative/v8/executor/V8Executor.java
@@ -15,7 +15,6 @@
 import com.facebook.react.bridge.RuntimeExecutor;
 import com.facebook.soloader.SoLoader;
 import io.csie.kudo.reactnative.v8.BuildConfig;
-import java.io.File;
 
 public class V8Executor extends JavaScriptExecutor {
   static {
diff --git a/android/src/main/java/io/csie/kudo/reactnative/v8/executor/V8Module.java b/android/src/main/java/io/csie/kudo/reactnative/v8/executor/V8Module.java
@@ -20,9 +20,9 @@ public V8Module(ReactApplicationContext reactContext) {
   }
 
   @Override
-  public void onCatalystInstanceDestroy() {
+  public void invalidate() {
     unregisterMainIdleHandler();
-    super.onCatalystInstanceDestroy();
+    super.invalidate();
   }
 
   @Override
diff --git a/expo/build.gradle b/expo/build.gradle
@@ -15,6 +15,7 @@ buildscript {
 }
 
 android {
+  namespace "io.csie.kudo.reactnative.v8.expo"
   compileSdkVersion safeExtGet("compileSdkVersion", 31)
 
   compileOptions {
diff --git a/expo/src/main/AndroidManifest.xml b/expo/src/main/AndroidManifest.xml
@@ -1,3 +1,2 @@
-<manifest package="io.csie.kudo.reactnative.v8.expo" xmlns:android="http://schemas.android.com/apk/res/android">
-
+<manifest xmlns:android="http://schemas.android.com/apk/res/android">
 </manifest>
diff --git a/src/v8runtime/V8Runtime.cpp b/src/v8runtime/V8Runtime.cpp
@@ -88,6 +88,44 @@ V8Runtime::V8Runtime(
     const V8Runtime *v8Runtime,
     std::unique_ptr<V8RuntimeConfig> config)
     : config_(std::move(config)) {
+  arrayBufferAllocator_.reset(
+      v8::ArrayBuffer::Allocator::NewDefaultAllocator());
+  v8::Isolate::CreateParams createParams;
+  createParams.array_buffer_allocator = arrayBufferAllocator_.get();
+  if (v8Runtime->config_->snapshotBlob) {
+    snapshotBlob_ = std::make_unique<v8::StartupData>();
+    snapshotBlob_->data = v8Runtime->config_->snapshotBlob->c_str();
+    snapshotBlob_->raw_size =
+        static_cast<int>(v8Runtime->config_->snapshotBlob->size());
+    createParams.snapshot_blob = snapshotBlob_.get();
+  }
+  config_->codecacheMode = V8RuntimeConfig::CodecacheMode::kNone;
+
+  isolate_ = v8::Isolate::New(createParams);
+#if defined(__ANDROID__)
+  if (!v8Runtime->config_->timezoneId.empty()) {
+    isolate_->DateTimeConfigurationChangeNotification(
+        v8::Isolate::TimeZoneDetection::kCustom,
+        v8Runtime->config_->timezoneId.c_str());
+  }
+#endif
+  v8::Locker locker(isolate_);
+  v8::Isolate::Scope scopedIsolate(isolate_);
+  v8::HandleScope scopedHandle(isolate_);
+  context_.Reset(isolate_, CreateGlobalContext(isolate_));
+  v8::Context::Scope scopedContext(context_.Get(isolate_));
+  jsQueue_ = v8Runtime->jsQueue_;
+
+  if (config_->enableInspector) {
+    inspectorClient_ = std::make_shared<InspectorClient>(
+        jsQueue_,
+        context_.Get(isolate_),
+        config_->appName,
+        config_->deviceName);
+    inspectorClient_->ConnectToReactFrontend();
+  }
+
+#if 0 // Experimental shared global context
   isSharedRuntime_ = true;
   isolate_ = v8Runtime->isolate_;
   jsQueue_ = v8Runtime->jsQueue_;
@@ -121,6 +159,7 @@ V8Runtime::V8Runtime(
         config_->deviceName);
     inspectorClient_->ConnectToReactFrontend();
   }
+#endif
 }
 
 V8Runtime::~V8Runtime() {
@@ -414,11 +453,13 @@ jsi::Value V8Runtime::evaluatePreparedJavaScript(
   return evaluateJavaScript(sourceJs, sourceJs->sourceURL());
 }
 
-#if REACT_NATIVE_MINOR_VERSION >= 75 || (REACT_NATIVE_MINOR_VERSION >= 74 && REACT_NATIVE_PATCH_VERSION >= 3)
+#if REACT_NATIVE_MINOR_VERSION >= 75 || \
+    (REACT_NATIVE_MINOR_VERSION >= 74 && REACT_NATIVE_PATCH_VERSION >= 3)
 void V8Runtime::queueMicrotask(const jsi::Function &callback) {
   // TODO: add this when we revisit new architecture support
 }
-#endif // REACT_NATIVE_MINOR_VERSION >= 75 || (REACT_NATIVE_MINOR_VERSION >= 74 && REACT_NATIVE_PATCH_VERSION >= 3
+#endif // REACT_NATIVE_MINOR_VERSION >= 75 || (REACT_NATIVE_MINOR_VERSION >= 74
+       // && REACT_NATIVE_PATCH_VERSION >= 3
 
 bool V8Runtime::drainMicrotasks(int maxMicrotasksHint) {
   v8::Locker locker(isolate_);
@@ -1559,8 +1600,9 @@ bool V8Runtime::instanceOf(const jsi::Object &o, const jsi::Function &f) {
 }
 
 #if REACT_NATIVE_MINOR_VERSION >= 74
-void V8Runtime::setExternalMemoryPressure(const jsi::Object &obj, size_t amount) {
-}
+void V8Runtime::setExternalMemoryPressure(
+    const jsi::Object &obj,
+    size_t amount) {}
 #endif
 
 //
diff --git a/src/v8runtime/V8Runtime.h b/src/v8runtime/V8Runtime.h
@@ -75,9 +75,11 @@ class V8Runtime : public facebook::jsi::Runtime {
       const std::shared_ptr<const facebook::jsi::PreparedJavaScript> &js)
       override;
 
-#if REACT_NATIVE_MINOR_VERSION >= 75 || (REACT_NATIVE_MINOR_VERSION >= 74 && REACT_NATIVE_PATCH_VERSION >= 3)
+#if REACT_NATIVE_MINOR_VERSION >= 75 || \
+    (REACT_NATIVE_MINOR_VERSION >= 74 && REACT_NATIVE_PATCH_VERSION >= 3)
   void queueMicrotask(const facebook::jsi::Function &callback) override;
-#endif // REACT_NATIVE_MINOR_VERSION >= 75 || (REACT_NATIVE_MINOR_VERSION >= 74 && REACT_NATIVE_PATCH_VERSION >= 3
+#endif // REACT_NATIVE_MINOR_VERSION >= 75 || (REACT_NATIVE_MINOR_VERSION >= 74
+       // && REACT_NATIVE_PATCH_VERSION >= 3
   bool drainMicrotasks(int maxMicrotasksHint = -1) override;
 
   facebook::jsi::Object global() override;

Original file line number	Diff line number	Diff line change
`@@ -20,9 +20,9 @@ public V8Module(ReactApplicationContext reactContext) {`
`20`	`20`	`}`
`21`	`21`
`22`	`22`	`@Override`
`23`		`- public void onCatalystInstanceDestroy() {`
	`23`	`+ public void invalidate() {`
`24`	`24`	`unregisterMainIdleHandler();`
`25`		`- super.onCatalystInstanceDestroy();`
	`25`	`+ super.invalidate();`
`26`	`26`	`}`
`27`	`27`
`28`	`28`	`@Override`
Original file line number	Diff line number	Diff line change
`@@ -15,6 +15,7 @@ buildscript {`
`15`	`15`	`}`
`16`	`16`
`17`	`17`	`android {`
	`18`	`+ namespace "io.csie.kudo.reactnative.v8.expo"`
`18`	`19`	`compileSdkVersion safeExtGet("compileSdkVersion", 31)`
`19`	`20`
`20`	`21`	`compileOptions {`