Fabric: Unconditional state management in Binding::schedulerDidFinishTransaction

Summary: Originally, moving the mount instruction generation under the `if` was a perf optimization but now, since we converge `LocalData` and `State`, this is no longer possible (because we need to treat State as LocalData in some cases).

Reviewed By: mdvacca

Differential Revision: D15962378

fbshipit-source-id: 37f9fadb72ac53450c2d499452610d9835f2964d

ReactAndroid/src/main/java/com/facebook/react/fabric/jni/Binding.cpp

@@ -559,12 +559,12 @@ void Binding::schedulerDidFinishTransaction(
                   deletedViewTags.end()) {
             mountItems[position++] =
                 createUpdatePropsMountItem(localJavaUIManager, mutation);
+          }
 
-            // State
-            if (mutation.newChildShadowView.state) {
-              mountItems[position++] =
-                  createUpdateStateMountItem(localJavaUIManager, mutation);
-            }
+          // State
+          if (mutation.newChildShadowView.state) {
+            mountItems[position++] =
+                createUpdateStateMountItem(localJavaUIManager, mutation);
           }
 
           // LocalData