Correctly batch reportMount calls (facebook#50090)

rubennorte · facebook-github-bot · commit e4d51cd8660c · 2025-03-18T05:59:50.000-07:00
Summary: Pull Request resolved: facebook#50090 Changelog: [internal] I refactored `FabricUIManager` in D54547194 / facebook#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()`