Correctly batch reportMount calls (#50090)

rubennorte · fabriziocucci · commit 4e4b9baf3c75 · 2025-03-26T13:47:29.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
@@ -1260,6 +1260,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
`@@ -1260,6 +1260,8 @@ public void didMountItems(@Nullable List<MountItem> mountItems) {`
`1260`	`1260`	`}`
`1261`	`1261`
`1262`	`1262`	`if (!mMountNotificationScheduled && !mMountedSurfaceIds.isEmpty()) {`
	`1263`	`+ mMountNotificationScheduled = true;`
	`1264`	`+`
`1263`	`1265`	`// Notify mount when the effects are visible and prevent mount hooks to`
`1264`	`1266`	`// delay paint.`
`1265`	`1267`	`UiThreadUtil.getUiThreadHandler()`