react-native-tvos 0.82.1-0 → 0.83.0-0

package/ReactAndroid/src/main/jni/react/jni/SafeReleaseJniRef.h

@@ -37,43 +37,51 @@ class SafeReleaseJniRef {
  public:
   SafeReleaseJniRef() = default;
   /* explicit */ SafeReleaseJniRef(RefT ref) : ref_(std::move(ref)) {}
-  SafeReleaseJniRef(const SafeReleaseJniRef& other) = default;
-  SafeReleaseJniRef(SafeReleaseJniRef&& other) = default;
-  SafeReleaseJniRef& operator=(const SafeReleaseJniRef& other) = default;
-  SafeReleaseJniRef& operator=(SafeReleaseJniRef&& other) = default;
+  SafeReleaseJniRef(const SafeReleaseJniRef &other) = default;
+  SafeReleaseJniRef(SafeReleaseJniRef &&other) = default;
+  SafeReleaseJniRef &operator=(const SafeReleaseJniRef &other) = default;
+  SafeReleaseJniRef &operator=(SafeReleaseJniRef &&other) = default;
 
-  ~SafeReleaseJniRef() {
+  ~SafeReleaseJniRef()
+  {
     if (ref_) {
       ensureThreadDurationJNIEnvAttached();
       ref_.reset();
     }
   }
 
-  operator bool() const noexcept {
+  operator bool() const noexcept
+  {
     return (bool)ref_;
   }
 
-  T& operator*() noexcept {
+  T &operator*() noexcept
+  {
     return *ref_;
   }
 
-  T* operator->() noexcept {
+  T *operator->() noexcept
+  {
     return &*ref_;
   }
 
-  const T& operator*() const noexcept {
+  const T &operator*() const noexcept
+  {
     return *ref_;
   }
 
-  const T* operator->() const noexcept {
+  const T *operator->() const noexcept
+  {
     return &*ref_;
   }
 
-  operator RefT() const {
+  operator RefT() const
+  {
     return ref_;
   }
 
-  RefT::javaobject get() const {
+  RefT::javaobject get() const
+  {
     return ref_.get();
   }
 

package/ReactAndroid/src/main/jni/react/jni/TransformHelper.h

@@ -13,8 +13,7 @@ namespace facebook::react {
 
 class TransformHelper : public jni::JavaClass<TransformHelper> {
  public:
-  static auto constexpr* kJavaDescriptor =
-      "Lcom/facebook/react/uimanager/TransformHelper;";
+  static auto constexpr *kJavaDescriptor = "Lcom/facebook/react/uimanager/TransformHelper;";
 
   static void registerNatives();
 };

package/ReactAndroid/src/main/jni/react/jni/WritableNativeArray.h

@@ -18,17 +18,14 @@ namespace facebook::react {
 struct ReadableNativeMap;
 
 struct WritableArray : jni::JavaClass<WritableArray> {
-  static auto constexpr kJavaDescriptor =
-      "Lcom/facebook/react/bridge/WritableArray;";
+  static auto constexpr kJavaDescriptor = "Lcom/facebook/react/bridge/WritableArray;";
 };
 
-struct WritableNativeArray
-    : public jni::HybridClass<WritableNativeArray, ReadableNativeArray> {
-  static constexpr const char* kJavaDescriptor =
-      "Lcom/facebook/react/bridge/WritableNativeArray;";
+struct WritableNativeArray : public jni::HybridClass<WritableNativeArray, ReadableNativeArray> {
+  static constexpr const char *kJavaDescriptor = "Lcom/facebook/react/bridge/WritableNativeArray;";
 
   WritableNativeArray();
-  WritableNativeArray(folly::dynamic&& val);
+  WritableNativeArray(folly::dynamic &&val);
 
   static void initHybrid(jni::alias_ref<jhybridobject> jobj);
 
@@ -38,8 +35,8 @@ struct WritableNativeArray
   void pushInt(jint value);
   void pushLong(jlong value);
   void pushString(jstring value);
-  void pushNativeArray(ReadableNativeArray* otherArray);
-  void pushNativeMap(ReadableNativeMap* map);
+  void pushNativeArray(ReadableNativeArray *otherArray);
+  void pushNativeMap(ReadableNativeMap *map);
 
   static void registerNatives();
 };

package/ReactAndroid/src/main/jni/react/jni/WritableNativeMap.h

@@ -17,17 +17,14 @@
 namespace facebook::react {
 
 struct WritableMap : jni::JavaClass<WritableMap> {
-  static auto constexpr kJavaDescriptor =
-      "Lcom/facebook/react/bridge/WritableMap;";
+  static auto constexpr kJavaDescriptor = "Lcom/facebook/react/bridge/WritableMap;";
 };
 
-struct WritableNativeMap
-    : jni::HybridClass<WritableNativeMap, ReadableNativeMap> {
-  static auto constexpr kJavaDescriptor =
-      "Lcom/facebook/react/bridge/WritableNativeMap;";
+struct WritableNativeMap : jni::HybridClass<WritableNativeMap, ReadableNativeMap> {
+  static auto constexpr kJavaDescriptor = "Lcom/facebook/react/bridge/WritableNativeMap;";
 
   WritableNativeMap();
-  WritableNativeMap(folly::dynamic&& val);
+  WritableNativeMap(folly::dynamic &&val);
 
   static void initHybrid(jni::alias_ref<jhybridobject> jobj);
 
@@ -37,9 +34,9 @@ struct WritableNativeMap
   void putInt(std::string key, int val);
   void putLong(std::string key, jlong val);
   void putString(std::string key, jni::alias_ref<jstring> val);
-  void putNativeArray(std::string key, ReadableNativeArray* otherArray);
-  void putNativeMap(std::string key, ReadableNativeMap* otherMap);
-  void mergeNativeMap(ReadableNativeMap* other);
+  void putNativeArray(std::string key, ReadableNativeArray *otherArray);
+  void putNativeMap(std::string key, ReadableNativeMap *otherMap);
+  void mergeNativeMap(ReadableNativeMap *other);
 
   static void registerNatives();
 

package/ReactAndroid/src/main/jni/react/mapbuffer/react/common/mapbuffer/JReadableMapBuffer.h

@@ -17,13 +17,11 @@ namespace facebook::react {
 
 class JReadableMapBuffer : public jni::HybridClass<JReadableMapBuffer> {
  public:
-  static auto constexpr kJavaDescriptor =
-      "Lcom/facebook/react/common/mapbuffer/ReadableMapBuffer;";
+  static auto constexpr kJavaDescriptor = "Lcom/facebook/react/common/mapbuffer/ReadableMapBuffer;";
 
-  static jni::local_ref<JReadableMapBuffer::jhybridobject> createWithContents(
-      MapBuffer&& map);
+  static jni::local_ref<JReadableMapBuffer::jhybridobject> createWithContents(MapBuffer &&map);
 
-  explicit JReadableMapBuffer(MapBuffer&& map);
+  explicit JReadableMapBuffer(MapBuffer &&map);
 
   std::vector<uint8_t> data() const;
 

package/ReactAndroid/src/main/jni/react/mapbuffer/react/common/mapbuffer/JWritableMapBuffer.h

@@ -14,8 +14,7 @@ namespace facebook::react {
 
 class JWritableMapBuffer : public jni::JavaClass<JWritableMapBuffer> {
  public:
-  static auto constexpr kJavaDescriptor =
-      "Lcom/facebook/react/common/mapbuffer/WritableMapBuffer;";
+  static auto constexpr kJavaDescriptor = "Lcom/facebook/react/common/mapbuffer/WritableMapBuffer;";
 
   MapBuffer getMapBuffer();
 };

package/ReactAndroid/src/main/jni/react/newarchdefaults/DefaultComponentsRegistry.cpp

@@ -36,18 +36,19 @@ void DefaultComponentsRegistry::setRegistryRunction(
         .flavor = nullptr};
 
     auto providerRegistry = CoreComponentsRegistry::sharedProviderRegistry();
-    if (registerCodegenComponentDescriptorsFromEntryPoint) {
-      registerCodegenComponentDescriptorsFromEntryPoint(providerRegistry);
-    } else {
-      LOG(WARNING)
-          << "Codegen component descriptors were not configured from JNI_OnLoad";
-    }
     if (registerComponentDescriptorsFromEntryPoint) {
       registerComponentDescriptorsFromEntryPoint(providerRegistry);
     } else {
       LOG(WARNING)
           << "Custom component descriptors were not configured from JNI_OnLoad";
     }
+    if (registerCodegenComponentDescriptorsFromEntryPoint) {
+      registerCodegenComponentDescriptorsFromEntryPoint(providerRegistry);
+    } else {
+      LOG(WARNING)
+          << "Codegen component descriptors were not configured from JNI_OnLoad";
+    }
+    CoreComponentsRegistry::addCoreComponents(providerRegistry);
 
     auto registry = providerRegistry->createComponentDescriptorRegistry(params);
     auto& mutableRegistry = const_cast<ComponentDescriptorRegistry&>(*registry);

package/ReactAndroid/src/main/jni/react/newarchdefaults/DefaultComponentsRegistry.h

@@ -15,26 +15,20 @@
 
 namespace facebook::react {
 
-class DefaultComponentsRegistry
-    : public facebook::jni::JavaClass<DefaultComponentsRegistry> {
+class DefaultComponentsRegistry : public facebook::jni::JavaClass<DefaultComponentsRegistry> {
  public:
-  constexpr static auto kJavaDescriptor =
-      "Lcom/facebook/react/defaults/DefaultComponentsRegistry;";
+  constexpr static auto kJavaDescriptor = "Lcom/facebook/react/defaults/DefaultComponentsRegistry;";
 
   static void registerNatives();
 
-  static std::function<void(
-      std::shared_ptr<const ComponentDescriptorProviderRegistry>)>
+  static std::function<void(std::shared_ptr<const ComponentDescriptorProviderRegistry>)>
       registerComponentDescriptorsFromEntryPoint;
 
-  static std::function<void(
-      std::shared_ptr<const ComponentDescriptorProviderRegistry>)>
+  static std::function<void(std::shared_ptr<const ComponentDescriptorProviderRegistry>)>
       registerCodegenComponentDescriptorsFromEntryPoint;
 
  private:
-  static void setRegistryRunction(
-      jni::alias_ref<jclass>,
-      ComponentFactory* delegate);
+  static void setRegistryRunction(jni::alias_ref<jclass> /*unused*/, ComponentFactory *delegate);
 };
 
 } // namespace facebook::react

package/ReactAndroid/src/main/jni/react/newarchdefaults/DefaultTurboModuleManagerDelegate.h

@@ -17,35 +17,27 @@
 
 namespace facebook::react {
 
-class DefaultTurboModuleManagerDelegate : public jni::HybridClass<
-                                              DefaultTurboModuleManagerDelegate,
-                                              TurboModuleManagerDelegate> {
+class DefaultTurboModuleManagerDelegate
+    : public jni::HybridClass<DefaultTurboModuleManagerDelegate, TurboModuleManagerDelegate> {
  public:
-  static constexpr auto kJavaDescriptor =
-      "Lcom/facebook/react/defaults/DefaultTurboModuleManagerDelegate;";
+  static constexpr auto kJavaDescriptor = "Lcom/facebook/react/defaults/DefaultTurboModuleManagerDelegate;";
 
   static jni::local_ref<jhybriddata> initHybrid(
-      jni::alias_ref<jclass>,
-      jni::alias_ref<jni::JList<CxxReactPackage::javaobject>::javaobject>);
+      jni::alias_ref<jclass> /*unused*/,
+      jni::alias_ref<jni::JList<CxxReactPackage::javaobject>::javaobject> /*cxxReactPackages*/);
 
   static void registerNatives();
 
-  static std::function<std::shared_ptr<TurboModule>(
-      const std::string&,
-      const std::shared_ptr<CallInvoker>&)>
+  static std::function<std::shared_ptr<TurboModule>(const std::string &, const std::shared_ptr<CallInvoker> &)>
       cxxModuleProvider;
 
-  static std::function<std::shared_ptr<TurboModule>(
-      const std::string&,
-      const JavaTurboModule::InitParams&)>
+  static std::function<std::shared_ptr<TurboModule>(const std::string &, const JavaTurboModule::InitParams &)>
       javaModuleProvider;
 
-  std::shared_ptr<TurboModule> getTurboModule(
-      const std::string& name,
-      const std::shared_ptr<CallInvoker>& jsInvoker) override;
-  std::shared_ptr<TurboModule> getTurboModule(
-      const std::string& name,
-      const JavaTurboModule::InitParams& params) override;
+  std::shared_ptr<TurboModule> getTurboModule(const std::string &name, const std::shared_ptr<CallInvoker> &jsInvoker)
+      override;
+  std::shared_ptr<TurboModule> getTurboModule(const std::string &name, const JavaTurboModule::InitParams &params)
+      override;
 
  private:
   friend HybridBase;
@@ -54,8 +46,7 @@ class DefaultTurboModuleManagerDelegate : public jni::HybridClass<
   std::vector<jni::global_ref<CxxReactPackage::javaobject>> cxxReactPackages_;
 
   DefaultTurboModuleManagerDelegate(
-      jni::alias_ref<jni::JList<CxxReactPackage::javaobject>::javaobject>
-          cxxReactPackage);
+      jni::alias_ref<jni::JList<CxxReactPackage::javaobject>::javaobject> cxxReactPackage);
 };
 
 } // namespace facebook::react

package/ReactAndroid/src/main/jni/react/reactnativeblob/BlobCollector.h

@@ -12,21 +12,17 @@
 
 namespace facebook::react {
 
-class BlobCollector : public jni::HybridClass<BlobCollector>,
-                      public jsi::HostObject {
+class BlobCollector : public jni::HybridClass<BlobCollector>, public jsi::HostObject {
  public:
   BlobCollector(jni::global_ref<jobject> blobModule, std::string blobId);
   ~BlobCollector();
 
   size_t getBlobLength();
 
-  static constexpr auto kJavaDescriptor =
-      "Lcom/facebook/react/modules/blob/BlobCollector;";
+  static constexpr auto kJavaDescriptor = "Lcom/facebook/react/modules/blob/BlobCollector;";
 
-  static void nativeInstall(
-      jni::alias_ref<jclass>,
-      jni::alias_ref<jobject> blobModule,
-      jlong jsContextNativePointer);
+  static void
+  nativeInstall(jni::alias_ref<jclass> /*unused*/, jni::alias_ref<jobject> blobModule, jlong jsContextNativePointer);
 
   static void registerNatives();
 

package/ReactAndroid/src/main/jni/react/reactperflogger/reactperflogger/JNativeModulePerfLogger.h

@@ -13,11 +13,9 @@
 
 namespace facebook::react {
 
-class JNativeModulePerfLogger
-    : public jni::HybridClass<JNativeModulePerfLogger> {
+class JNativeModulePerfLogger : public jni::HybridClass<JNativeModulePerfLogger> {
  public:
-  static auto constexpr kJavaDescriptor =
-      "Lcom/facebook/react/reactperflogger/NativeModulePerfLogger;";
+  static auto constexpr kJavaDescriptor = "Lcom/facebook/react/reactperflogger/NativeModulePerfLogger;";
 
   virtual std::unique_ptr<facebook::react::NativeModulePerfLogger> get() = 0;
 

package/ReactAndroid/src/main/jni/react/runtime/cxxreactpackage/ReactCommon/CxxReactPackage.h

@@ -17,12 +17,11 @@ class CallInvoker;
 
 class CxxReactPackage : public jni::HybridClass<CxxReactPackage> {
  public:
-  static auto constexpr kJavaDescriptor =
-      "Lcom/facebook/react/runtime/cxxreactpackage/CxxReactPackage;";
+  static auto constexpr kJavaDescriptor = "Lcom/facebook/react/runtime/cxxreactpackage/CxxReactPackage;";
 
   virtual std::shared_ptr<TurboModule> getModule(
-      const std::string& name,
-      const std::shared_ptr<CallInvoker>& jsInvoker) = 0;
+      const std::string &name,
+      const std::shared_ptr<CallInvoker> &jsInvoker) = 0;
 
  private:
   friend HybridBase;

package/ReactAndroid/src/main/jni/react/runtime/hermes/jni/JHermesInstance.h

@@ -19,23 +19,17 @@
 
 namespace facebook::react {
 
-class JHermesInstance
-    : public jni::HybridClass<JHermesInstance, JJSRuntimeFactory> {
+class JHermesInstance : public jni::HybridClass<JHermesInstance, JJSRuntimeFactory> {
  public:
-  static constexpr auto kJavaDescriptor =
-      "Lcom/facebook/react/runtime/hermes/HermesInstance;";
+  static constexpr auto kJavaDescriptor = "Lcom/facebook/react/runtime/hermes/HermesInstance;";
 
-  static jni::local_ref<jhybriddata> initHybrid(
-      jni::alias_ref<jclass> /* unused */,
-      bool allocInOldGenBeforeTTI);
+  static jni::local_ref<jhybriddata> initHybrid(jni::alias_ref<jclass> /* unused */, bool allocInOldGenBeforeTTI);
 
   static void registerNatives();
 
-  JHermesInstance(bool allocInOldGenBeforeTTI)
-      : allocInOldGenBeforeTTI_(allocInOldGenBeforeTTI){};
+  JHermesInstance(bool allocInOldGenBeforeTTI) : allocInOldGenBeforeTTI_(allocInOldGenBeforeTTI) {};
 
-  std::unique_ptr<JSRuntime> createJSRuntime(
-      std::shared_ptr<MessageQueueThread> msgQueueThread) noexcept;
+  std::unique_ptr<JSRuntime> createJSRuntime(std::shared_ptr<MessageQueueThread> msgQueueThread) noexcept;
 
   ~JHermesInstance() {}
 

package/ReactAndroid/src/main/jni/react/runtime/jni/JBindingsInstaller.h

@@ -15,11 +15,9 @@
 
 namespace facebook::react {
 
-class JBindingsInstaller : public jni::HybridClass<JBindingsInstaller>,
-                           public BindingsInstaller {
+class JBindingsInstaller : public jni::HybridClass<JBindingsInstaller>, public BindingsInstaller {
  public:
-  static constexpr auto kJavaDescriptor =
-      "Lcom/facebook/react/runtime/BindingsInstaller;";
+  static constexpr auto kJavaDescriptor = "Lcom/facebook/react/runtime/BindingsInstaller;";
 
   ~JBindingsInstaller() {}
 

package/ReactAndroid/src/main/jni/react/runtime/jni/JJSRuntimeFactory.h

@@ -13,11 +13,9 @@
 
 namespace facebook::react {
 
-class JJSRuntimeFactory : public jni::HybridClass<JJSRuntimeFactory>,
-                          public JSRuntimeFactory {
+class JJSRuntimeFactory : public jni::HybridClass<JJSRuntimeFactory>, public JSRuntimeFactory {
  public:
-  static auto constexpr kJavaDescriptor =
-      "Lcom/facebook/react/runtime/JSRuntimeFactory;";
+  static auto constexpr kJavaDescriptor = "Lcom/facebook/react/runtime/JSRuntimeFactory;";
 
  private:
   friend HybridBase;

package/ReactAndroid/src/main/jni/react/runtime/jni/JJSTimerExecutor.cpp

@@ -12,6 +12,11 @@
 
 namespace facebook::react {
 
+void JJSTimerExecutor::initHybrid(
+    jni::alias_ref<JJSTimerExecutor::jhybridobject> jobj) {
+  setCxxInstance(jobj);
+}
+
 void JJSTimerExecutor::setTimerManager(
     std::weak_ptr<TimerManager> timerManager) {
   timerManager_ = timerManager;
@@ -28,6 +33,7 @@ void JJSTimerExecutor::callTimers(WritableNativeArray* timerIDs) {
 void JJSTimerExecutor::registerNatives() {
   registerHybrid({
       makeNativeMethod("callTimers", JJSTimerExecutor::callTimers),
+      makeNativeMethod("initHybrid", JJSTimerExecutor::initHybrid),
   });
 }
 

package/ReactAndroid/src/main/jni/react/runtime/jni/JJSTimerExecutor.h

@@ -17,18 +17,19 @@ namespace facebook::react {
 
 class JJSTimerExecutor : public jni::HybridClass<JJSTimerExecutor> {
  public:
-  JJSTimerExecutor() = default;
-
-  constexpr static auto kJavaDescriptor =
-      "Lcom/facebook/react/runtime/JSTimerExecutor;";
+  constexpr static auto kJavaDescriptor = "Lcom/facebook/react/runtime/JSTimerExecutor;";
 
   static void registerNatives();
 
+  static void initHybrid(jni::alias_ref<jhybridobject> jobj);
+
   void setTimerManager(std::weak_ptr<TimerManager> timerManager);
 
-  void callTimers(WritableNativeArray* timerIDs);
+  void callTimers(WritableNativeArray *timerIDs);
 
  private:
+  JJSTimerExecutor() = default;
+
   friend HybridBase;
 
   std::weak_ptr<TimerManager> timerManager_;

package/ReactAndroid/src/main/jni/react/runtime/jni/JJavaTimerManager.h

@@ -15,8 +15,7 @@
 namespace facebook::react {
 
 struct JJavaTimerManager : jni::JavaClass<JJavaTimerManager> {
-  static auto constexpr kJavaDescriptor =
-      "Lcom/facebook/react/modules/core/JavaTimerManager;";
+  static auto constexpr kJavaDescriptor = "Lcom/facebook/react/modules/core/JavaTimerManager;";
 
   void createTimer(uint32_t timerID, double duration, bool repeat);
 

package/ReactAndroid/src/main/jni/react/runtime/jni/JReactExceptionManager.h

@@ -13,15 +13,11 @@
 
 namespace facebook::react {
 
-class JReactExceptionManager
-    : public facebook::jni::JavaClass<JReactExceptionManager> {
+class JReactExceptionManager : public facebook::jni::JavaClass<JReactExceptionManager> {
  public:
-  static auto constexpr kJavaDescriptor =
-      "Lcom/facebook/react/interfaces/exceptionmanager/ReactJsExceptionHandler;";
+  static auto constexpr kJavaDescriptor = "Lcom/facebook/react/interfaces/exceptionmanager/ReactJsExceptionHandler;";
 
-  void reportJsException(
-      jsi::Runtime& runtime,
-      const JsErrorHandler::ProcessedError& error);
+  void reportJsException(jsi::Runtime &runtime, const JsErrorHandler::ProcessedError &error);
 };
 
 } // namespace facebook::react

package/ReactAndroid/src/main/jni/react/runtime/jni/JReactHostInspectorTarget.cpp

@@ -120,15 +120,11 @@ void JReactHostInspectorTarget::onSetPausedInDebuggerMessage(
   }
 }
 
-void JReactHostInspectorTarget::unstable_onPerfMonitorUpdate(
-    const PerfMonitorUpdateRequest& request) {
-  static auto method = javaClassStatic()->getMethod<void(jint, jint, jint)>(
-      "handleNativePerfMonitorMetricUpdate");
-  method(
-      jobj_,
-      request.activeInteraction.duration,
-      static_cast<jint>(request.activeInteraction.responsivenessScore),
-      request.activeInteraction.ttl);
+void JReactHostInspectorTarget::unstable_onPerfIssueAdded(
+    const PerfIssuePayload& issue) {
+  static auto method = javaClassStatic()->getMethod<void(local_ref<jstring>)>(
+      "handleNativePerfIssueAdded");
+  method(jobj_, make_jstring(issue.name));
 }
 
 void JReactHostInspectorTarget::loadNetworkResource(
@@ -168,9 +164,16 @@ tracing::TraceRecordingState JReactHostInspectorTarget::stopTracing() {
   }
 }
 
-void JReactHostInspectorTarget::stopAndStashBackgroundTrace() {
+jboolean JReactHostInspectorTarget::stopAndMaybeEmitBackgroundTrace() {
   auto capturedTrace = inspectorTarget_->stopTracing();
+  if (inspectorTarget_->hasActiveSessionWithFuseboxClient()) {
+    inspectorTarget_->emitTraceRecordingForFirstFuseboxClient(
+        std::move(capturedTrace));
+    return jboolean(true);
+  }
+
   stashTraceRecordingState(std::move(capturedTrace));
+  return jboolean(false);
 }
 
 void JReactHostInspectorTarget::stopAndDiscardBackgroundTrace() {
@@ -199,8 +202,8 @@ void JReactHostInspectorTarget::registerNatives() {
           "startBackgroundTrace",
           JReactHostInspectorTarget::startBackgroundTrace),
       makeNativeMethod(
-          "stopAndStashBackgroundTrace",
-          JReactHostInspectorTarget::stopAndStashBackgroundTrace),
+          "stopAndMaybeEmitBackgroundTrace",
+          JReactHostInspectorTarget::stopAndMaybeEmitBackgroundTrace),
       makeNativeMethod(
           "stopAndDiscardBackgroundTrace",
           JReactHostInspectorTarget::stopAndDiscardBackgroundTrace),