Correctly batch reportMount calls (#50090)

rubennorte · react-native-bot · commit eb8c1c3e95c9 · 2025-04-01T09:22:57.000Z
Summary: Pull Request resolved: #50090 Changelog: [internal] I refactored `FabricUIManager` in D54547194 / #43337 and accidentally removed setting this flag to avoid scheduling redundant tasks in the UI thread to report mount. This fixes it. Reviewed By: javache Differential Revision: D71387374 fbshipit-source-id: cad8a3ead2434738325560902cbab817e5d5dde7
diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/fabric/FabricUIManager.java b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/fabric/FabricUIManager.java
@@ -1256,6 +1256,8 @@ public void didMountItems(@Nullable List<MountItem> mountItems) {
       }
 
       if (!mMountNotificationScheduled && !mMountedSurfaceIds.isEmpty()) {
+        mMountNotificationScheduled = true;
+
         // Notify mount when the effects are visible and prevent mount hooks to
         // delay paint.
         UiThreadUtil.getUiThreadHandler()

Original file line number	Diff line number	Diff line change
`@@ -1256,6 +1256,8 @@ public void didMountItems(@Nullable List<MountItem> mountItems) {`
`1256`	`1256`	`}`
`1257`	`1257`
`1258`	`1258`	`if (!mMountNotificationScheduled && !mMountedSurfaceIds.isEmpty()) {`
	`1259`	`+ mMountNotificationScheduled = true;`
	`1260`	`+`
`1259`	`1261`	`// Notify mount when the effects are visible and prevent mount hooks to`
`1260`	`1262`	`// delay paint.`
`1261`	`1263`	`UiThreadUtil.getUiThreadHandler()`