Replace NativeRunnable with fbjni implementation (#33776)

Summary:
Pull Request resolved: #33776

The implementations of these modules is pretty much identical, and we're already shipping the fbjni version of this anyway.

Changelog: [Internal]

Reviewed By: mhorowitz

Differential Revision: D36200330

fbshipit-source-id: 135ee621e1e4c5eb9616ce7f442fc6d4b946f865

ReactAndroid/src/main/java/com/facebook/react/fabric/jni/JBackgroundExecutor.cpp

@@ -5,26 +5,22 @@
  * LICENSE file in the root directory of this source tree.
  */
 
-#include <fbjni/fbjni.h>
-#include <react/jni/JNativeRunnable.h>
-
 #include "JBackgroundExecutor.h"
 
+#include <fbjni/NativeRunnable.h>
+#include <fbjni/fbjni.h>
+
 namespace facebook {
 namespace react {
 
 using namespace facebook::jni;
 
-using facebook::react::JNativeRunnable;
-using facebook::react::Runnable;
-
 BackgroundExecutor JBackgroundExecutor::create(const std::string &name) {
   auto instance = make_global(newInstance(name));
   return [instance = std::move(instance)](std::function<void()> &&runnable) {
     static auto method =
-        javaClassStatic()->getMethod<void(Runnable::javaobject)>(
+        javaClassStatic()->getMethod<void(JRunnable::javaobject)>(
             "queueRunnable");
-
     auto jrunnable = JNativeRunnable::newObjectCxxArgs(std::move(runnable));
     method(instance, jrunnable.get());
   };

ReactAndroid/src/main/jni/react/jni/BUCK

@@ -8,7 +8,6 @@ EXPORTED_HEADERS = [
     "JavaScriptExecutorHolder.h",
     "JCallback.h",
     "JMessageQueueThread.h",
-    "JNativeRunnable.h",
     "JReactMarker.h",
     "JSLoader.h",
     "JSLogging.h",

ReactAndroid/src/main/jni/react/jni/CatalystInstanceImpl.cpp

@@ -35,7 +35,6 @@
 #include <logger/react_native_log.h>
 
 #include "CxxModuleWrapper.h"
-#include "JNativeRunnable.h"
 #include "JReactCxxErrorHandler.h"
 #include "JReactSoftExceptionLogger.h"
 #include "JavaScriptExecutorHolder.h"
@@ -155,8 +154,6 @@ void CatalystInstanceImpl::registerNatives() {
           "warnOnLegacyNativeModuleSystemUse",
           CatalystInstanceImpl::warnOnLegacyNativeModuleSystemUse),
   });
-
-  JNativeRunnable::registerNatives();
 }
 
 void log(ReactNativeLogLevel level, const char *message) {

ReactAndroid/src/main/jni/react/jni/JMessageQueueThread.cpp

@@ -11,11 +11,10 @@
 #include <mutex>
 
 #include <fb/log.h>
+#include <fbjni/NativeRunnable.h>
 #include <fbjni/fbjni.h>
 #include <jsi/jsi.h>
 
-#include "JNativeRunnable.h"
-
 namespace facebook {
 namespace react {
 
@@ -69,11 +68,10 @@ void JMessageQueueThread::runOnQueue(std::function<void()> &&runnable) {
   jni::ThreadScope guard;
   static auto method =
       JavaMessageQueueThread::javaClassStatic()
-          ->getMethod<jboolean(Runnable::javaobject)>("runOnQueue");
-  method(
-      m_jobj,
-      JNativeRunnable::newObjectCxxArgs(wrapRunnable(std::move(runnable)))
-          .get());
+          ->getMethod<jboolean(JRunnable::javaobject)>("runOnQueue");
+  auto jrunnable =
+      JNativeRunnable::newObjectCxxArgs(wrapRunnable(std::move(runnable)));
+  method(m_jobj, jrunnable.get());
 }
 
 void JMessageQueueThread::runOnQueueSync(std::function<void()> &&runnable) {