[RN] Use microtasks when using bridgeless mode in React Native Fabric renderer (OSS version)

rubennorte · rubennorte · commit 6e0301ba8a2e · 2024-03-08T20:35:36.000Z
diff --git a/.eslintrc.js b/.eslintrc.js
@@ -427,6 +427,7 @@ module.exports = {
       files: ['packages/react-native-renderer/**/*.js'],
       globals: {
         nativeFabricUIManager: 'readonly',
+        RN$enableMicrotasksInReact: 'readonly',
       },
     },
     {
diff --git a/packages/react-native-renderer/src/ReactFiberConfigFabric.js b/packages/react-native-renderer/src/ReactFiberConfigFabric.js
@@ -47,10 +47,7 @@ const {
   unstable_getCurrentEventPriority: fabricGetCurrentEventPriority,
 } = nativeFabricUIManager;
 
-import {
-  useMicrotasksForSchedulingInFabric,
-  passChildrenWhenCloningPersistedNodes,
-} from 'shared/ReactFeatureFlags';
+import {passChildrenWhenCloningPersistedNodes} from 'shared/ReactFeatureFlags';
 
 const {get: getViewConfigForType} = ReactNativeViewConfigRegistry;
 
@@ -507,6 +504,10 @@ export const NotPendingTransition: TransitionStatus = null;
 // -------------------
 //     Microtasks
 // -------------------
-export const supportsMicrotasks = useMicrotasksForSchedulingInFabric;
+
+export const supportsMicrotasks: boolean =
+  typeof RN$enableMicrotasksInReact !== 'undefined' &&
+  !!RN$enableMicrotasksInReact;
+
 export const scheduleMicrotask: any =
   typeof queueMicrotask === 'function' ? queueMicrotask : scheduleTimeout;
diff --git a/packages/shared/ReactFeatureFlags.js b/packages/shared/ReactFeatureFlags.js
@@ -115,8 +115,6 @@ export const enableFizzExternalRuntime = true;
 
 export const alwaysThrottleRetries = true;
 
-export const useMicrotasksForSchedulingInFabric = false;
-
 export const passChildrenWhenCloningPersistedNodes = false;
 
 export const enableUseDeferredValueInitialArg = __EXPERIMENTAL__;
diff --git a/packages/shared/forks/ReactFeatureFlags.native-fb-dynamic.js b/packages/shared/forks/ReactFeatureFlags.native-fb-dynamic.js
@@ -25,7 +25,6 @@ export const enableComponentStackLocations = __VARIANT__;
 export const enableDeferRootSchedulingToMicrotask = __VARIANT__;
 export const enableUseRefAccessWarning = __VARIANT__;
 export const passChildrenWhenCloningPersistedNodes = __VARIANT__;
-export const useMicrotasksForSchedulingInFabric = __VARIANT__;
 export const enableUnifiedSyncLane = __VARIANT__;
 export const enableInfiniteRenderLoopDetection = __VARIANT__;
 
diff --git a/packages/shared/forks/ReactFeatureFlags.native-fb.js b/packages/shared/forks/ReactFeatureFlags.native-fb.js
@@ -23,7 +23,6 @@ export const {
   enableDeferRootSchedulingToMicrotask,
   enableUseRefAccessWarning,
   passChildrenWhenCloningPersistedNodes,
-  useMicrotasksForSchedulingInFabric,
   enableUnifiedSyncLane,
   enableInfiniteRenderLoopDetection,
 } = dynamicFlags;
diff --git a/packages/shared/forks/ReactFeatureFlags.native-oss.js b/packages/shared/forks/ReactFeatureFlags.native-oss.js
@@ -82,7 +82,6 @@ export const enableAsyncActions = false;
 
 export const alwaysThrottleRetries = false;
 
-export const useMicrotasksForSchedulingInFabric = false;
 export const passChildrenWhenCloningPersistedNodes = false;
 export const enableUseDeferredValueInitialArg = __EXPERIMENTAL__;
 export const disableClientCache = true;
diff --git a/packages/shared/forks/ReactFeatureFlags.test-renderer.js b/packages/shared/forks/ReactFeatureFlags.test-renderer.js
@@ -81,7 +81,6 @@ export const enableAsyncActions = true;
 
 export const alwaysThrottleRetries = true;
 
-export const useMicrotasksForSchedulingInFabric = false;
 export const passChildrenWhenCloningPersistedNodes = false;
 export const enableUseDeferredValueInitialArg = __EXPERIMENTAL__;
 export const disableClientCache = true;
diff --git a/packages/shared/forks/ReactFeatureFlags.test-renderer.native.js b/packages/shared/forks/ReactFeatureFlags.test-renderer.native.js
@@ -79,7 +79,6 @@ export const enableAsyncActions = true;
 
 export const alwaysThrottleRetries = true;
 
-export const useMicrotasksForSchedulingInFabric = false;
 export const passChildrenWhenCloningPersistedNodes = false;
 export const enableUseDeferredValueInitialArg = __EXPERIMENTAL__;
 export const disableClientCache = true;
diff --git a/packages/shared/forks/ReactFeatureFlags.test-renderer.www.js b/packages/shared/forks/ReactFeatureFlags.test-renderer.www.js
@@ -81,7 +81,6 @@ export const enableAsyncActions = true;
 
 export const alwaysThrottleRetries = true;
 
-export const useMicrotasksForSchedulingInFabric = false;
 export const passChildrenWhenCloningPersistedNodes = false;
 export const enableUseDeferredValueInitialArg = true;
 export const disableClientCache = true;
diff --git a/packages/shared/forks/ReactFeatureFlags.www.js b/packages/shared/forks/ReactFeatureFlags.www.js
@@ -109,7 +109,6 @@ export const enableFizzExternalRuntime = true;
 
 export const forceConcurrentByDefaultForTesting = false;
 
-export const useMicrotasksForSchedulingInFabric = false;
 export const passChildrenWhenCloningPersistedNodes = false;
 
 export const enableAsyncDebugInfo = false;
diff --git a/scripts/flow/react-native-host-hooks.js b/scripts/flow/react-native-host-hooks.js
@@ -167,6 +167,8 @@ declare module 'react-native/Libraries/ReactPrivate/ReactNativePrivateInterface'
 declare module 'react-native/Libraries/ReactPrivate/ReactNativePrivateInitializeCore' {
 }
 
+declare const RN$enableMicrotasksInReact: boolean;
+
 // This is needed for a short term solution.
 // See https://github.com/facebook/react/pull/15490 for more info
 declare var nativeFabricUIManager: {
diff --git a/scripts/flow/xplat.js b/scripts/flow/xplat.js
@@ -13,7 +13,6 @@ declare module 'ReactNativeInternalFeatureFlags' {
   declare export var enableDeferRootSchedulingToMicrotask: boolean;
   declare export var enableUseRefAccessWarning: boolean;
   declare export var passChildrenWhenCloningPersistedNodes: boolean;
-  declare export var useMicrotasksForSchedulingInFabric: boolean;
   declare export var enableUnifiedSyncLane: boolean;
   declare export var enableInfiniteRenderLoopDetection: boolean;
 }
diff --git a/scripts/rollup/validate/eslintrc.rn.js b/scripts/rollup/validate/eslintrc.rn.js
@@ -42,6 +42,8 @@ module.exports = {
     // Fabric. See https://github.com/facebook/react/pull/15490
     // for more information
     nativeFabricUIManager: 'readonly',
+    // RN flag to enable microtasks
+    RN$enableMicrotasksInReact: 'readonly',
     // Trusted Types
     trustedTypes: 'readonly',
     // RN supports this

Original file line number	Diff line number	Diff line change
`@@ -427,6 +427,7 @@ module.exports = {`
`427`	`427`	`files: ['packages/react-native-renderer/*/.js'],`
`428`	`428`	`globals: {`
`429`	`429`	`nativeFabricUIManager: 'readonly',`
	`430`	`+ RN$enableMicrotasksInReact: 'readonly',`
`430`	`431`	`},`
`431`	`432`	`},`
`432`	`433`	`{`