Always generate an EmptyNativeState (#34750)

Summary: Pull Request resolved: #34750 This Diff is the first step of enabling the CodeGen to parse and generate a NativeState for the components. The feature has been largely requested by the OSS community but it could be also helpful for people in Meta. To make this work, we also have to always generate a ViewEventEmitter: that's because the ConcreteShadowNode template lists the Generics with this order: Name, Props, EventEmitter, Others... If we skip the EventEmitters and we put the State, React Native would think that the State is actually an EventEmitter and the build step will fail. ## Changelog [General][Added] - Always generate an empty NativeState for Fabric Components Differential Revision: D39509869 fbshipit-source-id: 0c83443b185c49339a456caea1feba6023f8e1df
facebook · Sep 21, 2022 · b9057c1 · b9057c1
1 parent eddff32
commit b9057c1
Show file tree

Hide file tree

Showing 17 changed files with 732 additions and 72 deletions.
diff --git a/ReactAndroid/cmake-utils/ReactNative-application.cmake b/ReactAndroid/cmake-utils/ReactNative-application.cmake
@@ -18,14 +18,14 @@ set(CMAKE_VERBOSE_MAKEFILE on)
 
 include(${REACT_ANDROID_DIR}/cmake-utils/Android-prebuilt.cmake)
 
-file(GLOB input_SRC CONFIGURE_DEPENDS 
+file(GLOB input_SRC CONFIGURE_DEPENDS
         *.cpp
         ${PROJECT_BUILD_DIR}/generated/rncli/src/main/jni/*.cpp)
 
 add_library(${CMAKE_PROJECT_NAME} SHARED ${input_SRC})
 
-target_include_directories(${CMAKE_PROJECT_NAME} 
-        PUBLIC 
+target_include_directories(${CMAKE_PROJECT_NAME}
+        PUBLIC
                 ${CMAKE_CURRENT_SOURCE_DIR}
                 ${PROJECT_BUILD_DIR}/generated/rncli/src/main/jni)
 
@@ -45,6 +45,7 @@ target_link_libraries(${CMAKE_PROJECT_NAME}
         react_render_core
         react_render_debug
         react_render_graphics
+        react_render_mapbuffer
         rrc_view
         runtimeexecutor
         turbomodulejsijni

diff --git a/packages/react-native-codegen/DEFS.bzl b/packages/react-native-codegen/DEFS.bzl
@@ -263,6 +263,8 @@ def rn_codegen_components(
     generate_event_emitter_h_name = "generate_event_emitter_h-{}".format(name)
     generate_props_cpp_name = "generate_props_cpp-{}".format(name)
     generate_props_h_name = "generated_props_h-{}".format(name)
+    generate_state_cpp_name = "generate_state_cpp-{}".format(name)
+    generate_state_h_name = "generated_state_h-{}".format(name)
     generate_tests_cpp_name = "generate_tests_cpp-{}".format(name)
     generate_shadow_node_cpp_name = "generated_shadow_node_cpp-{}".format(name)
     generate_shadow_node_h_name = "generated_shadow_node_h-{}".format(name)
@@ -314,6 +316,13 @@ def rn_codegen_components(
         labels = ["codegen_rule"],
     )
 
+    fb_native.genrule(
+        name = generate_state_cpp_name,
+        cmd = "cp $(location :{})/States.cpp $OUT".format(generate_fixtures_rule_name),
+        out = "States.cpp",
+        labels = ["codegen_rule"],
+    )
+
     fb_native.genrule(
         name = generate_tests_cpp_name,
         cmd = "cp $(location :{})/Tests.cpp $OUT".format(generate_fixtures_rule_name),
@@ -328,6 +337,13 @@ def rn_codegen_components(
         labels = ["codegen_rule"],
     )
 
+    fb_native.genrule(
+        name = generate_state_h_name,
+        cmd = "cp $(location :{})/States.h $OUT".format(generate_fixtures_rule_name),
+        out = "States.h",
+        labels = ["codegen_rule"],
+    )
+
     fb_native.genrule(
         name = copy_generated_java_files,
         # TODO: support different package name internally.
@@ -388,12 +404,14 @@ def rn_codegen_components(
                 srcs = [
                     ":{}".format(generate_event_emitter_cpp_name),
                     ":{}".format(generate_props_cpp_name),
+                    ":{}".format(generate_state_cpp_name),
                     ":{}".format(generate_shadow_node_cpp_name),
                 ],
                 headers = [
                     ":{}".format(generate_component_descriptor_h_name),
                     ":{}".format(generate_event_emitter_h_name),
                     ":{}".format(generate_props_h_name),
+                    ":{}".format(generate_state_h_name),
                     ":{}".format(generate_shadow_node_h_name),
                 ],
                 header_namespace = "react/renderer/components/{}".format(name),
@@ -403,6 +421,7 @@ def rn_codegen_components(
                     "Props.h": ":{}".format(generate_props_h_name),
                     "RCTComponentViewHelpers.h": ":{}".format(generate_component_hobjcpp_name),
                     "ShadowNodes.h": ":{}".format(generate_shadow_node_h_name),
+                    "States.h": ":{}".format(generate_state_h_name),
                 },
                 fbobjc_compiler_flags = get_apple_compiler_flags(),
                 fbobjc_preprocessor_flags = get_preprocessor_flags_for_build_mode() + get_apple_inspector_flags(),

diff --git a/...-native-codegen/e2e/__tests__/components/__snapshots__/GenerateEventEmitterH-test.js.snap b/...-native-codegen/e2e/__tests__/components/__snapshots__/GenerateEventEmitterH-test.js.snap
@@ -19,7 +19,14 @@ Object {
 namespace facebook {
 namespace react {
 
+class JSI_EXPORT ArrayPropsNativeComponentViewEventEmitter : public ViewEventEmitter {
+ public:
+  using ViewEventEmitter::ViewEventEmitter;
 
+  
+
+  
+};
 
 } // namespace react
 } // namespace facebook
@@ -46,7 +53,14 @@ Object {
 namespace facebook {
 namespace react {
 
+class JSI_EXPORT BooleanPropNativeComponentViewEventEmitter : public ViewEventEmitter {
+ public:
+  using ViewEventEmitter::ViewEventEmitter;
 
+  
+
+  
+};
 
 } // namespace react
 } // namespace facebook
@@ -73,7 +87,14 @@ Object {
 namespace facebook {
 namespace react {
 
+class JSI_EXPORT ColorPropNativeComponentViewEventEmitter : public ViewEventEmitter {
+ public:
+  using ViewEventEmitter::ViewEventEmitter;
+
+  
 
+  
+};
 
 } // namespace react
 } // namespace facebook
@@ -100,7 +121,14 @@ Object {
 namespace facebook {
 namespace react {
 
+class JSI_EXPORT EdgeInsetsPropNativeComponentViewEventEmitter : public ViewEventEmitter {
+ public:
+  using ViewEventEmitter::ViewEventEmitter;
+
+  
 
+  
+};
 
 } // namespace react
 } // namespace facebook
@@ -127,7 +155,14 @@ Object {
 namespace facebook {
 namespace react {
 
+class JSI_EXPORT EnumPropNativeComponentViewEventEmitter : public ViewEventEmitter {
+ public:
+  using ViewEventEmitter::ViewEventEmitter;
+
+  
 
+  
+};
 
 } // namespace react
 } // namespace facebook
@@ -281,7 +316,14 @@ Object {
 namespace facebook {
 namespace react {
 
+class JSI_EXPORT FloatPropsNativeComponentViewEventEmitter : public ViewEventEmitter {
+ public:
+  using ViewEventEmitter::ViewEventEmitter;
+
+  
 
+  
+};
 
 } // namespace react
 } // namespace facebook
@@ -308,7 +350,14 @@ Object {
 namespace facebook {
 namespace react {
 
+class JSI_EXPORT ImagePropNativeComponentViewEventEmitter : public ViewEventEmitter {
+ public:
+  using ViewEventEmitter::ViewEventEmitter;
+
+  
 
+  
+};
 
 } // namespace react
 } // namespace facebook
@@ -335,7 +384,14 @@ Object {
 namespace facebook {
 namespace react {
 
+class JSI_EXPORT IntegerPropNativeComponentViewEventEmitter : public ViewEventEmitter {
+ public:
+  using ViewEventEmitter::ViewEventEmitter;
 
+  
+
+  
+};
 
 } // namespace react
 } // namespace facebook
@@ -398,7 +454,14 @@ Object {
 namespace facebook {
 namespace react {
 
+class JSI_EXPORT MultiNativePropNativeComponentViewEventEmitter : public ViewEventEmitter {
+ public:
+  using ViewEventEmitter::ViewEventEmitter;
+
+  
 
+  
+};
 
 } // namespace react
 } // namespace facebook
@@ -425,7 +488,14 @@ Object {
 namespace facebook {
 namespace react {
 
+class JSI_EXPORT NoPropsNoEventsNativeComponentViewEventEmitter : public ViewEventEmitter {
+ public:
+  using ViewEventEmitter::ViewEventEmitter;
+
+  
 
+  
+};
 
 } // namespace react
 } // namespace facebook
@@ -452,7 +522,14 @@ Object {
 namespace facebook {
 namespace react {
 
+class JSI_EXPORT ObjectPropsNativeComponentEventEmitter : public ViewEventEmitter {
+ public:
+  using ViewEventEmitter::ViewEventEmitter;
 
+  
+
+  
+};
 
 } // namespace react
 } // namespace facebook
@@ -479,7 +556,14 @@ Object {
 namespace facebook {
 namespace react {
 
+class JSI_EXPORT PointPropNativeComponentViewEventEmitter : public ViewEventEmitter {
+ public:
+  using ViewEventEmitter::ViewEventEmitter;
 
+  
+
+  
+};
 
 } // namespace react
 } // namespace facebook
@@ -506,7 +590,14 @@ Object {
 namespace facebook {
 namespace react {
 
+class JSI_EXPORT StringPropNativeComponentViewEventEmitter : public ViewEventEmitter {
+ public:
+  using ViewEventEmitter::ViewEventEmitter;
+
+  
 
+  
+};
 
 } // namespace react
 } // namespace facebook