Add a separate feature flag

facebook · Nov 29, 2022 · e2872ea · e2872ea
1 parent df90b2b
commit e2872ea
Show file tree

Hide file tree

Showing 24 changed files with 145 additions and 148 deletions.
diff --git a/packages/react-dom/src/__tests__/ReactDOMFiberAsync-test.js b/packages/react-dom/src/__tests__/ReactDOMFiberAsync-test.js
@@ -275,15 +275,15 @@ describe('ReactDOMFiberAsync', () => {
         expect(ops).toEqual([]);
       });
       // Only the active updates have flushed
-      if (gate(flags => flags.enableSyncDefaultUpdates)) {
+      if (gate(flags => flags.enableUnifiedSyncLane)) {
         expect(container.textContent).toEqual('ABC');
         expect(ops).toEqual(['ABC']);
       } else {
         expect(container.textContent).toEqual('BC');
         expect(ops).toEqual(['BC']);
       }
 
-      if (gate(flags => flags.enableSyncDefaultUpdates)) {
+      if (gate(flags => flags.enableUnifiedSyncLane)) {
         instance.push('D');
         expect(container.textContent).toEqual('ABC');
         expect(ops).toEqual(['ABC']);
@@ -296,7 +296,7 @@ describe('ReactDOMFiberAsync', () => {
       // Flush the async updates
       Scheduler.unstable_flushAll();
       expect(container.textContent).toEqual('ABCD');
-      if (gate(flags => flags.enableSyncDefaultUpdates)) {
+      if (gate(flags => flags.enableUnifiedSyncLane)) {
         expect(ops).toEqual(['ABC', 'ABCD']);
       } else {
         expect(ops).toEqual(['BC', 'ABCD']);

diff --git a/packages/react-reconciler/src/ReactFiberLane.new.js b/packages/react-reconciler/src/ReactFiberLane.new.js
@@ -23,7 +23,7 @@ import {
   enableUpdaterTracking,
   allowConcurrentByDefault,
   enableTransitionTracing,
-  enableSyncDefaultUpdates,
+  enableUnifiedSyncLane,
 } from 'shared/ReactFeatureFlags';
 import {isDevToolsPresent} from './ReactFiberDevToolsHook.new';
 import {ConcurrentUpdatesByDefaultMode, NoMode} from './ReactTypeOfMode';
@@ -136,7 +136,7 @@ let nextRetryLane: Lane = RetryLane1;
 function getHighestPriorityLanes(lanes: Lanes | Lane): Lanes {
   switch (getHighestPriorityLane(lanes)) {
     case SyncHydrationLane:
-      if (enableSyncDefaultUpdates) {
+      if (enableUnifiedSyncLane) {
         let ret = SyncHydrationLane;
         if (lanes & DefaultHydrationLane) {
           ret |= DefaultHydrationLane;
@@ -148,7 +148,7 @@ function getHighestPriorityLanes(lanes: Lanes | Lane): Lanes {
       }
       return SyncHydrationLane;
     case SyncLane:
-      if (enableSyncDefaultUpdates) {
+      if (enableUnifiedSyncLane) {
         let ret = SyncLane;
         if (lanes & DefaultLane) {
           ret |= DefaultLane;
@@ -273,7 +273,7 @@ export function getNextLanes(root: FiberRoot, wipLanes: Lanes): Lanes {
       // only difference between default updates and transition updates is that
       // default updates do not support refresh transitions.
       // Interrupt transtion if default is batched with sync.
-      (!enableSyncDefaultUpdates &&
+      (!enableUnifiedSyncLane &&
         nextLane === DefaultLane &&
         (wipLane & TransitionLanes) !== NoLanes)
     ) {

diff --git a/packages/react-reconciler/src/ReactFiberLane.old.js b/packages/react-reconciler/src/ReactFiberLane.old.js
@@ -23,7 +23,7 @@ import {
   enableUpdaterTracking,
   allowConcurrentByDefault,
   enableTransitionTracing,
-  enableSyncDefaultUpdates,
+  enableUnifiedSyncLane,
 } from 'shared/ReactFeatureFlags';
 import {isDevToolsPresent} from './ReactFiberDevToolsHook.old';
 import {ConcurrentUpdatesByDefaultMode, NoMode} from './ReactTypeOfMode';
@@ -136,7 +136,7 @@ let nextRetryLane: Lane = RetryLane1;
 function getHighestPriorityLanes(lanes: Lanes | Lane): Lanes {
   switch (getHighestPriorityLane(lanes)) {
     case SyncHydrationLane:
-      if (enableSyncDefaultUpdates) {
+      if (enableUnifiedSyncLane) {
         let ret = SyncHydrationLane;
         if (lanes & DefaultHydrationLane) {
           ret |= DefaultHydrationLane;
@@ -148,7 +148,7 @@ function getHighestPriorityLanes(lanes: Lanes | Lane): Lanes {
       }
       return SyncHydrationLane;
     case SyncLane:
-      if (enableSyncDefaultUpdates) {
+      if (enableUnifiedSyncLane) {
         let ret = SyncLane;
         if (lanes & DefaultLane) {
           ret |= DefaultLane;
@@ -273,7 +273,7 @@ export function getNextLanes(root: FiberRoot, wipLanes: Lanes): Lanes {
       // only difference between default updates and transition updates is that
       // default updates do not support refresh transitions.
       // Interrupt transtion if default is batched with sync.
-      (!enableSyncDefaultUpdates &&
+      (!enableUnifiedSyncLane &&
         nextLane === DefaultLane &&
         (wipLane & TransitionLanes) !== NoLanes)
     ) {

diff --git a/packages/react-reconciler/src/__tests__/ReactBatching-test.internal.js b/packages/react-reconciler/src/__tests__/ReactBatching-test.internal.js
@@ -158,7 +158,7 @@ describe('ReactBlockingMode', () => {
     );
 
     // Now flush the first update
-    if (gate(flags => flags.enableSyncDefaultUpdates)) {
+    if (gate(flags => flags.enableUnifiedSyncLane)) {
       expect(Scheduler).toHaveYielded(['A1', 'B1']);
       expect(root).toMatchRenderedOutput('A1B1');
     } else {

diff --git a/packages/react-reconciler/src/__tests__/ReactClassSetStateCallback-test.js b/packages/react-reconciler/src/__tests__/ReactClassSetStateCallback-test.js
@@ -35,7 +35,7 @@ describe('ReactClassSetStateCallback', () => {
     expect(Scheduler).toHaveYielded([0]);
 
     await act(async () => {
-      if (gate(flags => flags.enableSyncDefaultUpdates)) {
+      if (gate(flags => flags.enableUnifiedSyncLane)) {
         React.startTransition(() => {
           app.setState({step: 1}, () =>
             Scheduler.unstable_yieldValue('Callback 1'),

diff --git a/packages/react-reconciler/src/__tests__/ReactFlushSync-test.js b/packages/react-reconciler/src/__tests__/ReactFlushSync-test.js
@@ -55,15 +55,15 @@ describe('ReactFlushSync', () => {
       // They should commit in two separate batches. First the sync one.
       expect(() => {
         expect(Scheduler).toFlushUntilNextPaint(
-          gate(flags => flags.enableSyncDefaultUpdates) ? ['1, 1'] : ['1, 0'],
+          gate(flags => flags.enableUnifiedSyncLane) ? ['1, 1'] : ['1, 0'],
         );
       }).toErrorDev('flushSync was called from inside a lifecycle method');
 
       // The remaining update is not sync
       ReactNoop.flushSync();
       expect(Scheduler).toHaveYielded([]);
 
-      if (gate(flags => flags.enableSyncDefaultUpdates)) {
+      if (gate(flags => flags.enableUnifiedSyncLane)) {
         expect(Scheduler).toFlushUntilNextPaint([]);
       } else {
         // Now flush it.

diff --git a/packages/react-reconciler/src/__tests__/ReactHooks-test.internal.js b/packages/react-reconciler/src/__tests__/ReactHooks-test.internal.js
@@ -568,7 +568,7 @@ describe('ReactHooks', () => {
       });
     };
 
-    if (gate(flags => flags.enableSyncDefaultUpdates)) {
+    if (gate(flags => flags.enableUnifiedSyncLane)) {
       // Update at transition priority
       React.startTransition(() => update(n => n * 100));
     } else {

diff --git a/packages/react-reconciler/src/__tests__/ReactHooksWithNoopRenderer-test.js b/packages/react-reconciler/src/__tests__/ReactHooksWithNoopRenderer-test.js
@@ -961,7 +961,7 @@ describe('ReactHooksWithNoopRenderer', () => {
       ReactNoop.flushSync(() => {
         counter.current.dispatch(INCREMENT);
       });
-      if (gate(flags => flags.enableSyncDefaultUpdates)) {
+      if (gate(flags => flags.enableUnifiedSyncLane)) {
         expect(Scheduler).toHaveYielded(['Count: 4']);
         expect(ReactNoop.getChildren()).toEqual([span('Count: 4')]);
       } else {
@@ -1727,7 +1727,7 @@ describe('ReactHooksWithNoopRenderer', () => {
       // As a result we, somewhat surprisingly, commit them in the opposite order.
       // This should be fine because any non-discrete set of work doesn't guarantee order
       // and easily could've happened slightly later too.
-      if (gate(flags => flags.enableSyncDefaultUpdates)) {
+      if (gate(flags => flags.enableUnifiedSyncLane)) {
         expect(Scheduler).toHaveYielded(['Will set count to 1', 'Count: 1']);
       } else {
         expect(Scheduler).toHaveYielded([

diff --git a/packages/react-reconciler/src/__tests__/ReactIncremental-test.js b/packages/react-reconciler/src/__tests__/ReactIncremental-test.js
@@ -1910,7 +1910,7 @@ describe('ReactIncremental', () => {
         <ShowBoth />
       </Intl>,
     );
-    if (gate(flags => flags.enableSyncDefaultUpdates)) {
+    if (gate(flags => flags.enableUnifiedSyncLane)) {
       expect(Scheduler).toFlushAndYield([
         'Intl {}',
         'ShowLocale {"locale":"en"}',
@@ -2774,7 +2774,7 @@ describe('ReactIncremental', () => {
     // Interrupt at same priority
     ReactNoop.render(<Parent step={2} />);
 
-    if (gate(flags => flags.enableSyncDefaultUpdates)) {
+    if (gate(flags => flags.enableUnifiedSyncLane)) {
       expect(Scheduler).toFlushAndYield(['Parent: 2', 'Child: 2']);
     } else {
       expect(Scheduler).toFlushAndYield(['Child: 1', 'Parent: 2', 'Child: 2']);
@@ -2805,7 +2805,7 @@ describe('ReactIncremental', () => {
     ReactNoop.expire(2000);
     ReactNoop.render(<Parent step={2} />);
 
-    if (gate(flags => flags.enableSyncDefaultUpdates)) {
+    if (gate(flags => flags.enableUnifiedSyncLane)) {
       expect(Scheduler).toFlushAndYield(['Parent: 2', 'Child: 2']);
     } else {
       expect(Scheduler).toFlushAndYield(['Child: 1', 'Parent: 2', 'Child: 2']);