Remove disableHiddenPropDeprioritization flag (#18964)

This is rolled out to 100% public, so we can remove it.

Author: acdlite

packages/react-dom/src/client/ReactDOMHostConfig.js

@@ -78,7 +78,6 @@ import {
   enableModernEventSystem,
   enableCreateEventHandleAPI,
   enableScopeAPI,
-  disableHiddenPropDeprioritization,
 } from 'shared/ReactFeatureFlags';
 import {HostComponent, HostText} from 'react-reconciler/src/ReactWorkTags';
 import {TOP_BEFORE_BLUR, TOP_AFTER_BLUR} from '../events/DOMTopLevelEventTypes';
@@ -373,14 +372,9 @@ export function shouldSetTextContent(type: string, props: Props): boolean {
 }
 
 export function shouldDeprioritizeSubtree(type: string, props: Props): boolean {
-  if (disableHiddenPropDeprioritization) {
-    // This is obnoxiously specific so that nobody uses it, but we can still opt
-    // in via an infra-level userspace abstraction.
-    return props.hidden === 'unstable-do-not-use-legacy-hidden';
-  } else {
-    // Legacy behavior. Any truthy value works.
-    return !!props.hidden;
-  }
+  // This is obnoxiously specific so that nobody uses it, but we can still opt
+  // in via an infra-level userspace abstraction.
+  return props.hidden === 'unstable-do-not-use-legacy-hidden';
 }
 
 export function createTextInstance(

packages/shared/ReactFeatureFlags.js

@@ -138,6 +138,3 @@ export const enableLegacyFBSupport = false;
 // expiration time is currently rendering. Remove this flag once we have
 // migrated to the new behavior.
 export const deferRenderPhaseUpdateToNextBatch = true;
-
-// Flag used by www build so we can log occurrences of legacy hidden API
-export const disableHiddenPropDeprioritization = true;

packages/shared/forks/ReactFeatureFlags.native-fb.js

@@ -48,7 +48,6 @@ export const enableFilterEmptyStringAttributesDOM = false;
 
 export const enableNewReconciler = false;
 export const deferRenderPhaseUpdateToNextBatch = true;
-export const disableHiddenPropDeprioritization = true;
 
 // Flow magic to verify the exports of this file match the original version.
 // eslint-disable-next-line no-unused-vars

packages/shared/forks/ReactFeatureFlags.native-oss.js

@@ -47,7 +47,6 @@ export const enableFilterEmptyStringAttributesDOM = false;
 
 export const enableNewReconciler = false;
 export const deferRenderPhaseUpdateToNextBatch = true;
-export const disableHiddenPropDeprioritization = true;
 
 // Flow magic to verify the exports of this file match the original version.
 // eslint-disable-next-line no-unused-vars

packages/shared/forks/ReactFeatureFlags.test-renderer.js

@@ -47,7 +47,6 @@ export const enableFilterEmptyStringAttributesDOM = false;
 
 export const enableNewReconciler = false;
 export const deferRenderPhaseUpdateToNextBatch = true;
-export const disableHiddenPropDeprioritization = true;
 
 // Flow magic to verify the exports of this file match the original version.
 // eslint-disable-next-line no-unused-vars

packages/shared/forks/ReactFeatureFlags.test-renderer.www.js

@@ -47,7 +47,6 @@ export const enableFilterEmptyStringAttributesDOM = false;
 
 export const enableNewReconciler = false;
 export const deferRenderPhaseUpdateToNextBatch = true;
-export const disableHiddenPropDeprioritization = true;
 
 // Flow magic to verify the exports of this file match the original version.
 // eslint-disable-next-line no-unused-vars

packages/shared/forks/ReactFeatureFlags.testing.js

@@ -47,7 +47,6 @@ export const enableFilterEmptyStringAttributesDOM = false;
 
 export const enableNewReconciler = false;
 export const deferRenderPhaseUpdateToNextBatch = true;
-export const disableHiddenPropDeprioritization = true;
 
 // Flow magic to verify the exports of this file match the original version.
 // eslint-disable-next-line no-unused-vars

packages/shared/forks/ReactFeatureFlags.testing.www.js

@@ -47,7 +47,6 @@ export const enableFilterEmptyStringAttributesDOM = false;
 
 export const enableNewReconciler = false;
 export const deferRenderPhaseUpdateToNextBatch = true;
-export const disableHiddenPropDeprioritization = true;
 
 // Flow magic to verify the exports of this file match the original version.
 // eslint-disable-next-line no-unused-vars

packages/shared/forks/ReactFeatureFlags.www-dynamic.js

@@ -21,9 +21,6 @@ export const enableModernEventSystem = __VARIANT__;
 export const enableLegacyFBSupport = __VARIANT__;
 export const enableDebugTracing = !__VARIANT__;
 
-// Temporary flag, in case we need to re-enable this feature.
-export const disableHiddenPropDeprioritization = __VARIANT__;
-
 // This only has an effect in the new reconciler. But also, the new reconciler
 // is only enabled when __VARIANT__ is true. So this is set to the opposite of
 // __VARIANT__ so that it's `false` when running against the new reconciler.

packages/shared/forks/ReactFeatureFlags.www.js

@@ -27,7 +27,6 @@ export const {
   enableLegacyFBSupport,
   enableDebugTracing,
   deferRenderPhaseUpdateToNextBatch,
-  disableHiddenPropDeprioritization,
 } = dynamicFeatureFlags;
 
 // On WWW, __EXPERIMENTAL__ is used for a new modern build.