Flip the order of component descriptor registration (#53953)

Summary:
Pull Request resolved: #53953

For TurboModule registration, we register custom / app-specific TurboModules before core TurboModules to ensure that app-specific module overrides are preferred over core / default behaviors.

This provides a similar behavior for Fabric component descriptor registration.

## Changelog

[Internal]

Reviewed By: alanleedev

Differential Revision: D83273376

fbshipit-source-id: fd0fb5f10d2bc29839ae554f0e3cfc18d7957848

Author: rozele

packages/react-native/ReactAndroid/src/main/jni/react/fabric/CoreComponentsRegistry.cpp

@@ -32,54 +32,52 @@ namespace facebook::react::CoreComponentsRegistry {
 std::shared_ptr<const ComponentDescriptorProviderRegistry>
 sharedProviderRegistry() {
   static auto providerRegistry =
-      []() -> std::shared_ptr<ComponentDescriptorProviderRegistry> {
-    auto providerRegistry =
-        std::make_shared<ComponentDescriptorProviderRegistry>();
-
-    providerRegistry->add(concreteComponentDescriptorProvider<
-                          AndroidProgressBarComponentDescriptor>());
-    providerRegistry->add(concreteComponentDescriptorProvider<
-                          AndroidSwipeRefreshLayoutComponentDescriptor>());
-    providerRegistry->add(concreteComponentDescriptorProvider<
-                          ActivityIndicatorViewComponentDescriptor>());
-    providerRegistry->add(concreteComponentDescriptorProvider<
-                          AndroidTextInputComponentDescriptor>());
-    providerRegistry->add(
-        concreteComponentDescriptorProvider<ViewComponentDescriptor>());
-    providerRegistry->add(
-        concreteComponentDescriptorProvider<ImageComponentDescriptor>());
-    providerRegistry->add(concreteComponentDescriptorProvider<
-                          ModalHostViewComponentDescriptor>());
-    providerRegistry->add(concreteComponentDescriptorProvider<
-                          AndroidSwitchComponentDescriptor>());
-    providerRegistry->add(
-        concreteComponentDescriptorProvider<SafeAreaViewComponentDescriptor>());
-    providerRegistry->add(
-        concreteComponentDescriptorProvider<TextComponentDescriptor>());
-    providerRegistry->add(
-        concreteComponentDescriptorProvider<RawTextComponentDescriptor>());
-    providerRegistry->add(
-        concreteComponentDescriptorProvider<ScrollViewComponentDescriptor>());
-    providerRegistry->add(
-        concreteComponentDescriptorProvider<
-            AndroidHorizontalScrollContentViewComponentDescriptor>());
-    providerRegistry->add(
-        concreteComponentDescriptorProvider<ParagraphComponentDescriptor>());
-    providerRegistry->add(concreteComponentDescriptorProvider<
-                          AndroidDrawerLayoutComponentDescriptor>());
-    providerRegistry->add(concreteComponentDescriptorProvider<
-                          DebuggingOverlayComponentDescriptor>());
-    providerRegistry->add(concreteComponentDescriptorProvider<
-                          LayoutConformanceComponentDescriptor>());
-    providerRegistry->add(
-        concreteComponentDescriptorProvider<VirtualViewComponentDescriptor>());
-    providerRegistry->add(concreteComponentDescriptorProvider<
-                          VirtualViewExperimentalComponentDescriptor>());
-
-    return providerRegistry;
-  }();
-
+      std::make_shared<ComponentDescriptorProviderRegistry>();
   return providerRegistry;
 }
 
+void addCoreComponents(
+    std::shared_ptr<const ComponentDescriptorProviderRegistry>
+        providerRegistry) {
+  providerRegistry->add(concreteComponentDescriptorProvider<
+                        AndroidProgressBarComponentDescriptor>());
+  providerRegistry->add(concreteComponentDescriptorProvider<
+                        AndroidSwipeRefreshLayoutComponentDescriptor>());
+  providerRegistry->add(concreteComponentDescriptorProvider<
+                        ActivityIndicatorViewComponentDescriptor>());
+  providerRegistry->add(concreteComponentDescriptorProvider<
+                        AndroidTextInputComponentDescriptor>());
+  providerRegistry->add(
+      concreteComponentDescriptorProvider<ViewComponentDescriptor>());
+  providerRegistry->add(
+      concreteComponentDescriptorProvider<ImageComponentDescriptor>());
+  providerRegistry->add(
+      concreteComponentDescriptorProvider<ModalHostViewComponentDescriptor>());
+  providerRegistry->add(
+      concreteComponentDescriptorProvider<AndroidSwitchComponentDescriptor>());
+  providerRegistry->add(
+      concreteComponentDescriptorProvider<SafeAreaViewComponentDescriptor>());
+  providerRegistry->add(
+      concreteComponentDescriptorProvider<TextComponentDescriptor>());
+  providerRegistry->add(
+      concreteComponentDescriptorProvider<RawTextComponentDescriptor>());
+  providerRegistry->add(
+      concreteComponentDescriptorProvider<ScrollViewComponentDescriptor>());
+  providerRegistry->add(
+      concreteComponentDescriptorProvider<
+          AndroidHorizontalScrollContentViewComponentDescriptor>());
+  providerRegistry->add(
+      concreteComponentDescriptorProvider<ParagraphComponentDescriptor>());
+  providerRegistry->add(concreteComponentDescriptorProvider<
+                        AndroidDrawerLayoutComponentDescriptor>());
+  providerRegistry->add(concreteComponentDescriptorProvider<
+                        DebuggingOverlayComponentDescriptor>());
+  providerRegistry->add(concreteComponentDescriptorProvider<
+                        LayoutConformanceComponentDescriptor>());
+  providerRegistry->add(
+      concreteComponentDescriptorProvider<VirtualViewComponentDescriptor>());
+  providerRegistry->add(concreteComponentDescriptorProvider<
+                        VirtualViewExperimentalComponentDescriptor>());
+}
+
 } // namespace facebook::react::CoreComponentsRegistry

packages/react-native/ReactAndroid/src/main/jni/react/fabric/CoreComponentsRegistry.h

@@ -14,4 +14,7 @@ namespace facebook::react::CoreComponentsRegistry {
 std::shared_ptr<const ComponentDescriptorProviderRegistry>
 sharedProviderRegistry();
 
+void addCoreComponents(
+    std::shared_ptr<const ComponentDescriptorProviderRegistry> registry);
+
 } // namespace facebook::react::CoreComponentsRegistry

packages/react-native/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);