All we ever needed was @synchronized

Author: jkoutavas

packages/react-native/ReactAndroid/src/main/java/com/facebook/react/fabric/internal/interop/InteropUiBlockListener.kt

@@ -41,31 +41,27 @@ internal class InteropUIBlockListener : UIManagerListener {
   @Synchronized
   override fun willMountItems(uiManager: UIManager) {
     if (beforeUIBlocks.isEmpty()) {
-       return
-     }
-    // avoid ConcurrentModificationException by iterating over a copy
-    val snapshot = ArrayList(beforeUIBlocks)
-    beforeUIBlocks.clear()
-    if (uiManager is UIBlockViewResolver) {
-      snapshot.forEach { block ->
-        block.execute(uiManager)
+      return
+    }
+    beforeUIBlocks.forEach {
+      if (uiManager is UIBlockViewResolver) {
+        it.execute(uiManager)
       }
     }
+    beforeUIBlocks.clear()
   }
 
   @Synchronized
   override fun didMountItems(uiManager: UIManager) {
     if (afterUIBlocks.isEmpty()) {
-       return
-     }
-    // avoid ConcurrentModificationException by iterating over a copy
-    val snapshot = ArrayList(afterUIBlocks)
-    afterUIBlocks.clear()
-    if (uiManager is UIBlockViewResolver) {
-      snapshot.forEach { block ->
-        block.execute(uiManager)
+      return
+    }
+    afterUIBlocks.forEach {
+      if (uiManager is UIBlockViewResolver) {
+        it.execute(uiManager)
       }
     }
+    afterUIBlocks.clear()
   }
 
   override fun didDispatchMountItems(uiManager: UIManager) = didMountItems(uiManager)