[flags] cleanup renameElementSymbol (#35600)

Removed the feature flag completely, enabled by default. Will land once
I have everything ready on xplat side.

Author: hoxyq

packages/react-client/src/__tests__/ReactFlight-test.js

@@ -1085,7 +1085,6 @@ describe('ReactFlight', () => {
 });
 });
 
-// @gate renameElementSymbol
 it('should emit descriptions of errors in dev',async()=>{
 constClientErrorBoundary=clientReference(ErrorBoundary);
 

packages/react-dom/src/__tests__/ReactComponent-test.js

@@ -498,7 +498,6 @@ describe('ReactComponent', () => {
 );
 });
 
-// @gate renameElementSymbol
 it('throws if a legacy element is used as a child',async()=>{
 constinlinedElement={
 $$typeof: Symbol.for('react.element'),

packages/react-dom/src/__tests__/ReactDOMOption-test.js

@@ -138,12 +138,11 @@ describe('ReactDOMOption', () => {
 }).rejects.toThrow('Objects are not valid as a React child');
 });
 
-// @gate www && !renameElementSymbol
 it('should support element-ish child',async()=>{
 // This is similar to <fbt>.
 // We don't toString it because you must instead provide a value prop.
 constobj={
-$$typeof: Symbol.for('react.element'),
+$$typeof: Symbol.for('react.transitional.element'),
 type: props=>props.content,
 ref: null,
 key: null,

packages/shared/ReactFeatureFlags.js

@@ -159,9 +159,6 @@ export const enableInternalInstanceMap: boolean = false;
 
 // const __NEXT_MAJOR__ = __EXPERIMENTAL__;
 
-// Renames the internal symbol for elements since they have changed signature/constructor
-exportconstrenameElementSymbol: boolean=true;
-
 /**
  * Enables a fix to run insertion effect cleanup on hidden subtrees.
  */

packages/shared/ReactSymbols.js

@@ -7,17 +7,15 @@
  * @flow
  */
 
-import{renameElementSymbol}from'shared/ReactFeatureFlags';
-
 // ATTENTION
 // When adding new symbols to this file,
 // Please consider also adding to 'react-devtools-shared/src/backend/ReactSymbols'
 
 // The Symbol used to tag the ReactElement-like types.
 exportconstREACT_LEGACY_ELEMENT_TYPE: symbol=Symbol.for('react.element');
-exportconstREACT_ELEMENT_TYPE: symbol=renameElementSymbol
-? Symbol.for('react.transitional.element')
- : REACT_LEGACY_ELEMENT_TYPE;
+exportconstREACT_ELEMENT_TYPE: symbol=Symbol.for(
+'react.transitional.element',
+);
 exportconstREACT_PORTAL_TYPE: symbol=Symbol.for('react.portal');
 exportconstREACT_FRAGMENT_TYPE: symbol=Symbol.for('react.fragment');
 exportconstREACT_STRICT_MODE_TYPE: symbol=Symbol.for('react.strict_mode');

packages/shared/__tests__/ReactSymbols-test.internal.js

@@ -23,7 +23,6 @@ describe('ReactSymbols', () => {
 });
 };
 
-// @gate renameElementSymbol
 it('Symbol values should be unique',()=>{
 expectToBeUnique(Object.entries(require('shared/ReactSymbols')));
 });

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

@@ -22,7 +22,6 @@ export const enableObjectFiber = __VARIANT__;
 exportconstenableHiddenSubtreeInsertionEffectCleanup=__VARIANT__;
 exportconstenableEagerAlternateStateNodeCleanup=__VARIANT__;
 exportconstpassChildrenWhenCloningPersistedNodes=__VARIANT__;
-exportconstrenameElementSymbol=__VARIANT__;
 exportconstenableFragmentRefs=__VARIANT__;
 exportconstenableFragmentRefsScrollIntoView=__VARIANT__;
 exportconstenableFragmentRefsInstanceHandles=__VARIANT__;

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

@@ -24,7 +24,6 @@ export const {
  enableObjectFiber,
  enableEagerAlternateStateNodeCleanup,
  passChildrenWhenCloningPersistedNodes,
- renameElementSymbol,
  enableFragmentRefs,
  enableFragmentRefsScrollIntoView,
  enableFragmentRefsInstanceHandles,

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

@@ -52,7 +52,6 @@ export const enableTaint: boolean = true;
 exportconstenableTransitionTracing: boolean=false;
 exportconstenableTrustedTypesIntegration: boolean=false;
 exportconstpassChildrenWhenCloningPersistedNodes: boolean=false;
-exportconstrenameElementSymbol: boolean=true;
 exportconstretryLaneExpirationMs=5000;
 exportconstsyncLaneExpirationMs=250;
 exportconsttransitionLaneExpirationMs=5000;

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

@@ -55,7 +55,6 @@ export const disableClientCache: boolean = true;
 
 exportconstenableInfiniteRenderLoopDetection: boolean=false;
 
-exportconstrenameElementSymbol: boolean=true;
 exportconstenableEagerAlternateStateNodeCleanup: boolean=true;
 
 exportconstenableYieldingBeforePassive: boolean=true;

packages/react-client/src/__tests__/ReactFlight-test.js

@@ -1085,7 +1085,6 @@ describe('ReactFlight', () => {
 });
 });
 
-// @gate renameElementSymbol
 it('should emit descriptions of errors in dev',async()=>{
 constClientErrorBoundary=clientReference(ErrorBoundary);
 

packages/react-dom/src/__tests__/ReactComponent-test.js

@@ -498,7 +498,6 @@ describe('ReactComponent', () => {
 );
 });
 
-// @gate renameElementSymbol
 it('throws if a legacy element is used as a child',async()=>{
 constinlinedElement={
 $$typeof: Symbol.for('react.element'),

packages/react-dom/src/__tests__/ReactDOMOption-test.js

@@ -138,12 +138,11 @@ describe('ReactDOMOption', () => {
 }).rejects.toThrow('Objects are not valid as a React child');
 });
 
-// @gate www && !renameElementSymbol
 it('should support element-ish child',async()=>{
 // This is similar to <fbt>.
 // We don't toString it because you must instead provide a value prop.
 constobj={
-$$typeof: Symbol.for('react.element'),
+$$typeof: Symbol.for('react.transitional.element'),
 type: props=>props.content,
 ref: null,
 key: null,

packages/shared/ReactFeatureFlags.js

@@ -159,9 +159,6 @@ export const enableInternalInstanceMap: boolean = false;
 
 // const __NEXT_MAJOR__ = __EXPERIMENTAL__;
 
-// Renames the internal symbol for elements since they have changed signature/constructor
-exportconstrenameElementSymbol: boolean=true;
-
 /**
  * Enables a fix to run insertion effect cleanup on hidden subtrees.
  */

packages/shared/ReactSymbols.js

@@ -7,17 +7,15 @@
  * @flow
  */
 
-import{renameElementSymbol}from'shared/ReactFeatureFlags';
-
 // ATTENTION
 // When adding new symbols to this file,
 // Please consider also adding to 'react-devtools-shared/src/backend/ReactSymbols'
 
 // The Symbol used to tag the ReactElement-like types.
 exportconstREACT_LEGACY_ELEMENT_TYPE: symbol=Symbol.for('react.element');
-exportconstREACT_ELEMENT_TYPE: symbol=renameElementSymbol
-? Symbol.for('react.transitional.element')
- : REACT_LEGACY_ELEMENT_TYPE;
+exportconstREACT_ELEMENT_TYPE: symbol=Symbol.for(
+'react.transitional.element',
+);
 exportconstREACT_PORTAL_TYPE: symbol=Symbol.for('react.portal');
 exportconstREACT_FRAGMENT_TYPE: symbol=Symbol.for('react.fragment');
 exportconstREACT_STRICT_MODE_TYPE: symbol=Symbol.for('react.strict_mode');

packages/shared/__tests__/ReactSymbols-test.internal.js

@@ -23,7 +23,6 @@ describe('ReactSymbols', () => {
 });
 };
 
-// @gate renameElementSymbol
 it('Symbol values should be unique',()=>{
 expectToBeUnique(Object.entries(require('shared/ReactSymbols')));
 });

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

@@ -22,7 +22,6 @@ export const enableObjectFiber = __VARIANT__;
 exportconstenableHiddenSubtreeInsertionEffectCleanup=__VARIANT__;
 exportconstenableEagerAlternateStateNodeCleanup=__VARIANT__;
 exportconstpassChildrenWhenCloningPersistedNodes=__VARIANT__;
-exportconstrenameElementSymbol=__VARIANT__;
 exportconstenableFragmentRefs=__VARIANT__;
 exportconstenableFragmentRefsScrollIntoView=__VARIANT__;
 exportconstenableFragmentRefsInstanceHandles=__VARIANT__;

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

@@ -24,7 +24,6 @@ export const {
  enableObjectFiber,
  enableEagerAlternateStateNodeCleanup,
  passChildrenWhenCloningPersistedNodes,
- renameElementSymbol,
  enableFragmentRefs,
  enableFragmentRefsScrollIntoView,
  enableFragmentRefsInstanceHandles,

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

@@ -52,7 +52,6 @@ export const enableTaint: boolean = true;
 exportconstenableTransitionTracing: boolean=false;
 exportconstenableTrustedTypesIntegration: boolean=false;
 exportconstpassChildrenWhenCloningPersistedNodes: boolean=false;
-exportconstrenameElementSymbol: boolean=true;
 exportconstretryLaneExpirationMs=5000;
 exportconstsyncLaneExpirationMs=250;
 exportconsttransitionLaneExpirationMs=5000;

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

@@ -55,7 +55,6 @@ export const disableClientCache: boolean = true;
 
 exportconstenableInfiniteRenderLoopDetection: boolean=false;
 
-exportconstrenameElementSymbol: boolean=true;
 exportconstenableEagerAlternateStateNodeCleanup: boolean=true;
 
 exportconstenableYieldingBeforePassive: boolean=true;

packages/react-client/src/__tests__/ReactFlight-test.js

@@ -1085,7 +1085,6 @@ describe('ReactFlight', () => {
 });
 });
 
-// @gate renameElementSymbol
 it('should emit descriptions of errors in dev',async()=>{
 constClientErrorBoundary=clientReference(ErrorBoundary);
 

packages/react-dom/src/__tests__/ReactComponent-test.js

@@ -498,7 +498,6 @@ describe('ReactComponent', () => {
 );
 });
 
-// @gate renameElementSymbol
 it('throws if a legacy element is used as a child',async()=>{
 constinlinedElement={
 $$typeof: Symbol.for('react.element'),

packages/react-dom/src/__tests__/ReactDOMOption-test.js

@@ -138,12 +138,11 @@ describe('ReactDOMOption', () => {
 }).rejects.toThrow('Objects are not valid as a React child');
 });
 
-// @gate www && !renameElementSymbol
 it('should support element-ish child',async()=>{
 // This is similar to <fbt>.
 // We don't toString it because you must instead provide a value prop.
 constobj={
-$$typeof: Symbol.for('react.element'),
+$$typeof: Symbol.for('react.transitional.element'),
 type: props=>props.content,
 ref: null,
 key: null,

packages/shared/ReactFeatureFlags.js

@@ -159,9 +159,6 @@ export const enableInternalInstanceMap: boolean = false;
 
 // const __NEXT_MAJOR__ = __EXPERIMENTAL__;
 
-// Renames the internal symbol for elements since they have changed signature/constructor
-exportconstrenameElementSymbol: boolean=true;
-
 /**
  * Enables a fix to run insertion effect cleanup on hidden subtrees.
  */

packages/shared/ReactSymbols.js

@@ -7,17 +7,15 @@
  * @flow
  */
 
-import{renameElementSymbol}from'shared/ReactFeatureFlags';
-
 // ATTENTION
 // When adding new symbols to this file,
 // Please consider also adding to 'react-devtools-shared/src/backend/ReactSymbols'
 
 // The Symbol used to tag the ReactElement-like types.
 exportconstREACT_LEGACY_ELEMENT_TYPE: symbol=Symbol.for('react.element');
-exportconstREACT_ELEMENT_TYPE: symbol=renameElementSymbol
-? Symbol.for('react.transitional.element')
- : REACT_LEGACY_ELEMENT_TYPE;
+exportconstREACT_ELEMENT_TYPE: symbol=Symbol.for(
+'react.transitional.element',
+);
 exportconstREACT_PORTAL_TYPE: symbol=Symbol.for('react.portal');
 exportconstREACT_FRAGMENT_TYPE: symbol=Symbol.for('react.fragment');
 exportconstREACT_STRICT_MODE_TYPE: symbol=Symbol.for('react.strict_mode');

packages/shared/__tests__/ReactSymbols-test.internal.js

@@ -23,7 +23,6 @@ describe('ReactSymbols', () => {
 });
 };
 
-// @gate renameElementSymbol
 it('Symbol values should be unique',()=>{
 expectToBeUnique(Object.entries(require('shared/ReactSymbols')));
 });

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

@@ -22,7 +22,6 @@ export const enableObjectFiber = __VARIANT__;
 exportconstenableHiddenSubtreeInsertionEffectCleanup=__VARIANT__;
 exportconstenableEagerAlternateStateNodeCleanup=__VARIANT__;
 exportconstpassChildrenWhenCloningPersistedNodes=__VARIANT__;
-exportconstrenameElementSymbol=__VARIANT__;
 exportconstenableFragmentRefs=__VARIANT__;
 exportconstenableFragmentRefsScrollIntoView=__VARIANT__;
 exportconstenableFragmentRefsInstanceHandles=__VARIANT__;

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

@@ -24,7 +24,6 @@ export const {
  enableObjectFiber,
  enableEagerAlternateStateNodeCleanup,
  passChildrenWhenCloningPersistedNodes,
- renameElementSymbol,
  enableFragmentRefs,
  enableFragmentRefsScrollIntoView,
  enableFragmentRefsInstanceHandles,

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

@@ -52,7 +52,6 @@ export const enableTaint: boolean = true;
 exportconstenableTransitionTracing: boolean=false;
 exportconstenableTrustedTypesIntegration: boolean=false;
 exportconstpassChildrenWhenCloningPersistedNodes: boolean=false;
-exportconstrenameElementSymbol: boolean=true;
 exportconstretryLaneExpirationMs=5000;
 exportconstsyncLaneExpirationMs=250;
 exportconsttransitionLaneExpirationMs=5000;

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

@@ -55,7 +55,6 @@ export const disableClientCache: boolean = true;
 
 exportconstenableInfiniteRenderLoopDetection: boolean=false;
 
-exportconstrenameElementSymbol: boolean=true;
 exportconstenableEagerAlternateStateNodeCleanup: boolean=true;
 
 exportconstenableYieldingBeforePassive: boolean=true;

packages/react-client/src/__tests__/ReactFlight-test.js

@@ -1085,7 +1085,6 @@ describe('ReactFlight', () => {
 });
 });
 
-// @gate renameElementSymbol
 it('should emit descriptions of errors in dev',async()=>{
 constClientErrorBoundary=clientReference(ErrorBoundary);
 

packages/react-dom/src/__tests__/ReactComponent-test.js

@@ -498,7 +498,6 @@ describe('ReactComponent', () => {
 );
 });
 
-// @gate renameElementSymbol
 it('throws if a legacy element is used as a child',async()=>{
 constinlinedElement={
 $$typeof: Symbol.for('react.element'),

packages/react-dom/src/__tests__/ReactDOMOption-test.js

@@ -138,12 +138,11 @@ describe('ReactDOMOption', () => {
 }).rejects.toThrow('Objects are not valid as a React child');
 });
 
-// @gate www && !renameElementSymbol
 it('should support element-ish child',async()=>{
 // This is similar to <fbt>.
 // We don't toString it because you must instead provide a value prop.
 constobj={
-$$typeof: Symbol.for('react.element'),
+$$typeof: Symbol.for('react.transitional.element'),
 type: props=>props.content,
 ref: null,
 key: null,

packages/shared/ReactFeatureFlags.js

@@ -159,9 +159,6 @@ export const enableInternalInstanceMap: boolean = false;
 
 // const __NEXT_MAJOR__ = __EXPERIMENTAL__;
 
-// Renames the internal symbol for elements since they have changed signature/constructor
-exportconstrenameElementSymbol: boolean=true;
-
 /**
  * Enables a fix to run insertion effect cleanup on hidden subtrees.
  */

packages/shared/ReactSymbols.js

@@ -7,17 +7,15 @@
  * @flow
  */
 
-import{renameElementSymbol}from'shared/ReactFeatureFlags';
-
 // ATTENTION
 // When adding new symbols to this file,
 // Please consider also adding to 'react-devtools-shared/src/backend/ReactSymbols'
 
 // The Symbol used to tag the ReactElement-like types.
 exportconstREACT_LEGACY_ELEMENT_TYPE: symbol=Symbol.for('react.element');
-exportconstREACT_ELEMENT_TYPE: symbol=renameElementSymbol
-? Symbol.for('react.transitional.element')
- : REACT_LEGACY_ELEMENT_TYPE;
+exportconstREACT_ELEMENT_TYPE: symbol=Symbol.for(
+'react.transitional.element',
+);
 exportconstREACT_PORTAL_TYPE: symbol=Symbol.for('react.portal');
 exportconstREACT_FRAGMENT_TYPE: symbol=Symbol.for('react.fragment');
 exportconstREACT_STRICT_MODE_TYPE: symbol=Symbol.for('react.strict_mode');

packages/shared/__tests__/ReactSymbols-test.internal.js

@@ -23,7 +23,6 @@ describe('ReactSymbols', () => {
 });
 };
 
-// @gate renameElementSymbol
 it('Symbol values should be unique',()=>{
 expectToBeUnique(Object.entries(require('shared/ReactSymbols')));
 });

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

@@ -22,7 +22,6 @@ export const enableObjectFiber = __VARIANT__;
 exportconstenableHiddenSubtreeInsertionEffectCleanup=__VARIANT__;
 exportconstenableEagerAlternateStateNodeCleanup=__VARIANT__;
 exportconstpassChildrenWhenCloningPersistedNodes=__VARIANT__;
-exportconstrenameElementSymbol=__VARIANT__;
 exportconstenableFragmentRefs=__VARIANT__;
 exportconstenableFragmentRefsScrollIntoView=__VARIANT__;
 exportconstenableFragmentRefsInstanceHandles=__VARIANT__;

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

@@ -24,7 +24,6 @@ export const {
  enableObjectFiber,
  enableEagerAlternateStateNodeCleanup,
  passChildrenWhenCloningPersistedNodes,
- renameElementSymbol,
  enableFragmentRefs,
  enableFragmentRefsScrollIntoView,
  enableFragmentRefsInstanceHandles,

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

@@ -52,7 +52,6 @@ export const enableTaint: boolean = true;
 exportconstenableTransitionTracing: boolean=false;
 exportconstenableTrustedTypesIntegration: boolean=false;
 exportconstpassChildrenWhenCloningPersistedNodes: boolean=false;
-exportconstrenameElementSymbol: boolean=true;
 exportconstretryLaneExpirationMs=5000;
 exportconstsyncLaneExpirationMs=250;
 exportconsttransitionLaneExpirationMs=5000;

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

@@ -55,7 +55,6 @@ export const disableClientCache: boolean = true;
 
 exportconstenableInfiniteRenderLoopDetection: boolean=false;
 
-exportconstrenameElementSymbol: boolean=true;
 exportconstenableEagerAlternateStateNodeCleanup: boolean=true;
 
 exportconstenableYieldingBeforePassive: boolean=true;

packages/react-client/src/__tests__/ReactFlight-test.js

@@ -1085,7 +1085,6 @@ describe('ReactFlight', () => {
 });
 });
 
-// @gate renameElementSymbol
 it('should emit descriptions of errors in dev',async()=>{
 constClientErrorBoundary=clientReference(ErrorBoundary);
 

packages/react-dom/src/__tests__/ReactComponent-test.js

@@ -498,7 +498,6 @@ describe('ReactComponent', () => {
 );
 });
 
-// @gate renameElementSymbol
 it('throws if a legacy element is used as a child',async()=>{
 constinlinedElement={
 $$typeof: Symbol.for('react.element'),

packages/react-dom/src/__tests__/ReactDOMOption-test.js

@@ -138,12 +138,11 @@ describe('ReactDOMOption', () => {
 }).rejects.toThrow('Objects are not valid as a React child');
 });
 
-// @gate www && !renameElementSymbol
 it('should support element-ish child',async()=>{
 // This is similar to <fbt>.
 // We don't toString it because you must instead provide a value prop.
 constobj={
-$$typeof: Symbol.for('react.element'),
+$$typeof: Symbol.for('react.transitional.element'),
 type: props=>props.content,
 ref: null,
 key: null,

packages/shared/ReactFeatureFlags.js

@@ -159,9 +159,6 @@ export const enableInternalInstanceMap: boolean = false;
 
 // const __NEXT_MAJOR__ = __EXPERIMENTAL__;
 
-// Renames the internal symbol for elements since they have changed signature/constructor
-exportconstrenameElementSymbol: boolean=true;
-
 /**
  * Enables a fix to run insertion effect cleanup on hidden subtrees.
  */

packages/shared/ReactSymbols.js

@@ -7,17 +7,15 @@
  * @flow
  */
 
-import{renameElementSymbol}from'shared/ReactFeatureFlags';
-
 // ATTENTION
 // When adding new symbols to this file,
 // Please consider also adding to 'react-devtools-shared/src/backend/ReactSymbols'
 
 // The Symbol used to tag the ReactElement-like types.
 exportconstREACT_LEGACY_ELEMENT_TYPE: symbol=Symbol.for('react.element');
-exportconstREACT_ELEMENT_TYPE: symbol=renameElementSymbol
-? Symbol.for('react.transitional.element')
- : REACT_LEGACY_ELEMENT_TYPE;
+exportconstREACT_ELEMENT_TYPE: symbol=Symbol.for(
+'react.transitional.element',
+);
 exportconstREACT_PORTAL_TYPE: symbol=Symbol.for('react.portal');
 exportconstREACT_FRAGMENT_TYPE: symbol=Symbol.for('react.fragment');
 exportconstREACT_STRICT_MODE_TYPE: symbol=Symbol.for('react.strict_mode');

packages/shared/__tests__/ReactSymbols-test.internal.js

@@ -23,7 +23,6 @@ describe('ReactSymbols', () => {
 });
 };
 
-// @gate renameElementSymbol
 it('Symbol values should be unique',()=>{
 expectToBeUnique(Object.entries(require('shared/ReactSymbols')));
 });

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

@@ -22,7 +22,6 @@ export const enableObjectFiber = __VARIANT__;
 exportconstenableHiddenSubtreeInsertionEffectCleanup=__VARIANT__;
 exportconstenableEagerAlternateStateNodeCleanup=__VARIANT__;
 exportconstpassChildrenWhenCloningPersistedNodes=__VARIANT__;
-exportconstrenameElementSymbol=__VARIANT__;
 exportconstenableFragmentRefs=__VARIANT__;
 exportconstenableFragmentRefsScrollIntoView=__VARIANT__;
 exportconstenableFragmentRefsInstanceHandles=__VARIANT__;

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

@@ -24,7 +24,6 @@ export const {
  enableObjectFiber,
  enableEagerAlternateStateNodeCleanup,
  passChildrenWhenCloningPersistedNodes,
- renameElementSymbol,
  enableFragmentRefs,
  enableFragmentRefsScrollIntoView,
  enableFragmentRefsInstanceHandles,

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

@@ -52,7 +52,6 @@ export const enableTaint: boolean = true;
 exportconstenableTransitionTracing: boolean=false;
 exportconstenableTrustedTypesIntegration: boolean=false;
 exportconstpassChildrenWhenCloningPersistedNodes: boolean=false;
-exportconstrenameElementSymbol: boolean=true;
 exportconstretryLaneExpirationMs=5000;
 exportconstsyncLaneExpirationMs=250;
 exportconsttransitionLaneExpirationMs=5000;

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

@@ -55,7 +55,6 @@ export const disableClientCache: boolean = true;
 
 exportconstenableInfiniteRenderLoopDetection: boolean=false;
 
-exportconstrenameElementSymbol: boolean=true;
 exportconstenableEagerAlternateStateNodeCleanup: boolean=true;
 
 exportconstenableYieldingBeforePassive: boolean=true;

packages/react-client/src/__tests__/ReactFlight-test.js

@@ -1085,7 +1085,6 @@ describe('ReactFlight', () => {
 });
 });
 
-// @gate renameElementSymbol
 it('should emit descriptions of errors in dev',async()=>{
 constClientErrorBoundary=clientReference(ErrorBoundary);
 

packages/react-dom/src/__tests__/ReactComponent-test.js

@@ -498,7 +498,6 @@ describe('ReactComponent', () => {
 );
 });
 
-// @gate renameElementSymbol
 it('throws if a legacy element is used as a child',async()=>{
 constinlinedElement={
 $$typeof: Symbol.for('react.element'),

packages/react-dom/src/__tests__/ReactDOMOption-test.js

@@ -138,12 +138,11 @@ describe('ReactDOMOption', () => {
 }).rejects.toThrow('Objects are not valid as a React child');
 });
 
-// @gate www && !renameElementSymbol
 it('should support element-ish child',async()=>{
 // This is similar to <fbt>.
 // We don't toString it because you must instead provide a value prop.
 constobj={
-$$typeof: Symbol.for('react.element'),
+$$typeof: Symbol.for('react.transitional.element'),
 type: props=>props.content,
 ref: null,
 key: null,

packages/shared/ReactFeatureFlags.js

@@ -159,9 +159,6 @@ export const enableInternalInstanceMap: boolean = false;
 
 // const __NEXT_MAJOR__ = __EXPERIMENTAL__;
 
-// Renames the internal symbol for elements since they have changed signature/constructor
-exportconstrenameElementSymbol: boolean=true;
-
 /**
  * Enables a fix to run insertion effect cleanup on hidden subtrees.
  */

packages/shared/ReactSymbols.js

@@ -7,17 +7,15 @@
  * @flow
  */
 
-import{renameElementSymbol}from'shared/ReactFeatureFlags';
-
 // ATTENTION
 // When adding new symbols to this file,
 // Please consider also adding to 'react-devtools-shared/src/backend/ReactSymbols'
 
 // The Symbol used to tag the ReactElement-like types.
 exportconstREACT_LEGACY_ELEMENT_TYPE: symbol=Symbol.for('react.element');
-exportconstREACT_ELEMENT_TYPE: symbol=renameElementSymbol
-? Symbol.for('react.transitional.element')
- : REACT_LEGACY_ELEMENT_TYPE;
+exportconstREACT_ELEMENT_TYPE: symbol=Symbol.for(
+'react.transitional.element',
+);
 exportconstREACT_PORTAL_TYPE: symbol=Symbol.for('react.portal');
 exportconstREACT_FRAGMENT_TYPE: symbol=Symbol.for('react.fragment');
 exportconstREACT_STRICT_MODE_TYPE: symbol=Symbol.for('react.strict_mode');

packages/shared/__tests__/ReactSymbols-test.internal.js

@@ -23,7 +23,6 @@ describe('ReactSymbols', () => {
 });
 };
 
-// @gate renameElementSymbol
 it('Symbol values should be unique',()=>{
 expectToBeUnique(Object.entries(require('shared/ReactSymbols')));
 });

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

@@ -22,7 +22,6 @@ export const enableObjectFiber = __VARIANT__;
 exportconstenableHiddenSubtreeInsertionEffectCleanup=__VARIANT__;
 exportconstenableEagerAlternateStateNodeCleanup=__VARIANT__;
 exportconstpassChildrenWhenCloningPersistedNodes=__VARIANT__;
-exportconstrenameElementSymbol=__VARIANT__;
 exportconstenableFragmentRefs=__VARIANT__;
 exportconstenableFragmentRefsScrollIntoView=__VARIANT__;
 exportconstenableFragmentRefsInstanceHandles=__VARIANT__;

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

@@ -24,7 +24,6 @@ export const {
  enableObjectFiber,
  enableEagerAlternateStateNodeCleanup,
  passChildrenWhenCloningPersistedNodes,
- renameElementSymbol,
  enableFragmentRefs,
  enableFragmentRefsScrollIntoView,
  enableFragmentRefsInstanceHandles,

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

@@ -52,7 +52,6 @@ export const enableTaint: boolean = true;
 exportconstenableTransitionTracing: boolean=false;
 exportconstenableTrustedTypesIntegration: boolean=false;
 exportconstpassChildrenWhenCloningPersistedNodes: boolean=false;
-exportconstrenameElementSymbol: boolean=true;
 exportconstretryLaneExpirationMs=5000;
 exportconstsyncLaneExpirationMs=250;
 exportconsttransitionLaneExpirationMs=5000;

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

@@ -55,7 +55,6 @@ export const disableClientCache: boolean = true;
 
 exportconstenableInfiniteRenderLoopDetection: boolean=false;
 
-exportconstrenameElementSymbol: boolean=true;
 exportconstenableEagerAlternateStateNodeCleanup: boolean=true;
 
 exportconstenableYieldingBeforePassive: boolean=true;

packages/react-client/src/__tests__/ReactFlight-test.js

@@ -1085,7 +1085,6 @@ describe('ReactFlight', () => {
 });
 });
 
-// @gate renameElementSymbol
 it('should emit descriptions of errors in dev',async()=>{
 constClientErrorBoundary=clientReference(ErrorBoundary);
 

packages/react-dom/src/__tests__/ReactComponent-test.js

@@ -498,7 +498,6 @@ describe('ReactComponent', () => {
 );
 });
 
-// @gate renameElementSymbol
 it('throws if a legacy element is used as a child',async()=>{
 constinlinedElement={
 $$typeof: Symbol.for('react.element'),

packages/react-dom/src/__tests__/ReactDOMOption-test.js

@@ -138,12 +138,11 @@ describe('ReactDOMOption', () => {
 }).rejects.toThrow('Objects are not valid as a React child');
 });
 
-// @gate www && !renameElementSymbol
 it('should support element-ish child',async()=>{
 // This is similar to <fbt>.
 // We don't toString it because you must instead provide a value prop.
 constobj={
-$$typeof: Symbol.for('react.element'),
+$$typeof: Symbol.for('react.transitional.element'),
 type: props=>props.content,
 ref: null,
 key: null,

packages/shared/ReactFeatureFlags.js

@@ -159,9 +159,6 @@ export const enableInternalInstanceMap: boolean = false;
 
 // const __NEXT_MAJOR__ = __EXPERIMENTAL__;
 
-// Renames the internal symbol for elements since they have changed signature/constructor
-exportconstrenameElementSymbol: boolean=true;
-
 /**
  * Enables a fix to run insertion effect cleanup on hidden subtrees.
  */

packages/shared/ReactSymbols.js

@@ -7,17 +7,15 @@
  * @flow
  */
 
-import{renameElementSymbol}from'shared/ReactFeatureFlags';
-
 // ATTENTION
 // When adding new symbols to this file,
 // Please consider also adding to 'react-devtools-shared/src/backend/ReactSymbols'
 
 // The Symbol used to tag the ReactElement-like types.
 exportconstREACT_LEGACY_ELEMENT_TYPE: symbol=Symbol.for('react.element');
-exportconstREACT_ELEMENT_TYPE: symbol=renameElementSymbol
-? Symbol.for('react.transitional.element')
- : REACT_LEGACY_ELEMENT_TYPE;
+exportconstREACT_ELEMENT_TYPE: symbol=Symbol.for(
+'react.transitional.element',
+);
 exportconstREACT_PORTAL_TYPE: symbol=Symbol.for('react.portal');
 exportconstREACT_FRAGMENT_TYPE: symbol=Symbol.for('react.fragment');
 exportconstREACT_STRICT_MODE_TYPE: symbol=Symbol.for('react.strict_mode');

packages/shared/__tests__/ReactSymbols-test.internal.js

@@ -23,7 +23,6 @@ describe('ReactSymbols', () => {
 });
 };
 
-// @gate renameElementSymbol
 it('Symbol values should be unique',()=>{
 expectToBeUnique(Object.entries(require('shared/ReactSymbols')));
 });

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

@@ -22,7 +22,6 @@ export const enableObjectFiber = __VARIANT__;
 exportconstenableHiddenSubtreeInsertionEffectCleanup=__VARIANT__;
 exportconstenableEagerAlternateStateNodeCleanup=__VARIANT__;
 exportconstpassChildrenWhenCloningPersistedNodes=__VARIANT__;
-exportconstrenameElementSymbol=__VARIANT__;
 exportconstenableFragmentRefs=__VARIANT__;
 exportconstenableFragmentRefsScrollIntoView=__VARIANT__;
 exportconstenableFragmentRefsInstanceHandles=__VARIANT__;

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

@@ -24,7 +24,6 @@ export const {
  enableObjectFiber,
  enableEagerAlternateStateNodeCleanup,
  passChildrenWhenCloningPersistedNodes,
- renameElementSymbol,
  enableFragmentRefs,
  enableFragmentRefsScrollIntoView,
  enableFragmentRefsInstanceHandles,

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

@@ -52,7 +52,6 @@ export const enableTaint: boolean = true;
 exportconstenableTransitionTracing: boolean=false;
 exportconstenableTrustedTypesIntegration: boolean=false;
 exportconstpassChildrenWhenCloningPersistedNodes: boolean=false;
-exportconstrenameElementSymbol: boolean=true;
 exportconstretryLaneExpirationMs=5000;
 exportconstsyncLaneExpirationMs=250;
 exportconsttransitionLaneExpirationMs=5000;

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

@@ -55,7 +55,6 @@ export const disableClientCache: boolean = true;
 
 exportconstenableInfiniteRenderLoopDetection: boolean=false;
 
-exportconstrenameElementSymbol: boolean=true;
 exportconstenableEagerAlternateStateNodeCleanup: boolean=true;
 
 exportconstenableYieldingBeforePassive: boolean=true;

packages/react-client/src/__tests__/ReactFlight-test.js

@@ -1085,7 +1085,6 @@ describe('ReactFlight', () => {
 });
 });
 
-// @gate renameElementSymbol
 it('should emit descriptions of errors in dev',async()=>{
 constClientErrorBoundary=clientReference(ErrorBoundary);
 

packages/react-dom/src/__tests__/ReactComponent-test.js

@@ -498,7 +498,6 @@ describe('ReactComponent', () => {
 );
 });
 
-// @gate renameElementSymbol
 it('throws if a legacy element is used as a child',async()=>{
 constinlinedElement={
 $$typeof: Symbol.for('react.element'),

packages/react-dom/src/__tests__/ReactDOMOption-test.js

@@ -138,12 +138,11 @@ describe('ReactDOMOption', () => {
 }).rejects.toThrow('Objects are not valid as a React child');
 });
 
-// @gate www && !renameElementSymbol
 it('should support element-ish child',async()=>{
 // This is similar to <fbt>.
 // We don't toString it because you must instead provide a value prop.
 constobj={
-$$typeof: Symbol.for('react.element'),
+$$typeof: Symbol.for('react.transitional.element'),
 type: props=>props.content,
 ref: null,
 key: null,

packages/shared/ReactFeatureFlags.js

@@ -159,9 +159,6 @@ export const enableInternalInstanceMap: boolean = false;
 
 // const __NEXT_MAJOR__ = __EXPERIMENTAL__;
 
-// Renames the internal symbol for elements since they have changed signature/constructor
-exportconstrenameElementSymbol: boolean=true;
-
 /**
  * Enables a fix to run insertion effect cleanup on hidden subtrees.
  */

packages/shared/ReactSymbols.js

@@ -7,17 +7,15 @@
  * @flow
  */
 
-import{renameElementSymbol}from'shared/ReactFeatureFlags';
-
 // ATTENTION
 // When adding new symbols to this file,
 // Please consider also adding to 'react-devtools-shared/src/backend/ReactSymbols'
 
 // The Symbol used to tag the ReactElement-like types.
 exportconstREACT_LEGACY_ELEMENT_TYPE: symbol=Symbol.for('react.element');
-exportconstREACT_ELEMENT_TYPE: symbol=renameElementSymbol
-? Symbol.for('react.transitional.element')
- : REACT_LEGACY_ELEMENT_TYPE;
+exportconstREACT_ELEMENT_TYPE: symbol=Symbol.for(
+'react.transitional.element',
+);
 exportconstREACT_PORTAL_TYPE: symbol=Symbol.for('react.portal');
 exportconstREACT_FRAGMENT_TYPE: symbol=Symbol.for('react.fragment');
 exportconstREACT_STRICT_MODE_TYPE: symbol=Symbol.for('react.strict_mode');

packages/shared/__tests__/ReactSymbols-test.internal.js

@@ -23,7 +23,6 @@ describe('ReactSymbols', () => {
 });
 };
 
-// @gate renameElementSymbol
 it('Symbol values should be unique',()=>{
 expectToBeUnique(Object.entries(require('shared/ReactSymbols')));
 });

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

@@ -22,7 +22,6 @@ export const enableObjectFiber = __VARIANT__;
 exportconstenableHiddenSubtreeInsertionEffectCleanup=__VARIANT__;
 exportconstenableEagerAlternateStateNodeCleanup=__VARIANT__;
 exportconstpassChildrenWhenCloningPersistedNodes=__VARIANT__;
-exportconstrenameElementSymbol=__VARIANT__;
 exportconstenableFragmentRefs=__VARIANT__;
 exportconstenableFragmentRefsScrollIntoView=__VARIANT__;
 exportconstenableFragmentRefsInstanceHandles=__VARIANT__;

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

@@ -24,7 +24,6 @@ export const {
  enableObjectFiber,
  enableEagerAlternateStateNodeCleanup,
  passChildrenWhenCloningPersistedNodes,
- renameElementSymbol,
  enableFragmentRefs,
  enableFragmentRefsScrollIntoView,
  enableFragmentRefsInstanceHandles,

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

@@ -52,7 +52,6 @@ export const enableTaint: boolean = true;
 exportconstenableTransitionTracing: boolean=false;
 exportconstenableTrustedTypesIntegration: boolean=false;
 exportconstpassChildrenWhenCloningPersistedNodes: boolean=false;
-exportconstrenameElementSymbol: boolean=true;
 exportconstretryLaneExpirationMs=5000;
 exportconstsyncLaneExpirationMs=250;
 exportconsttransitionLaneExpirationMs=5000;

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

@@ -55,7 +55,6 @@ export const disableClientCache: boolean = true;
 
 exportconstenableInfiniteRenderLoopDetection: boolean=false;
 
-exportconstrenameElementSymbol: boolean=true;
 exportconstenableEagerAlternateStateNodeCleanup: boolean=true;
 
 exportconstenableYieldingBeforePassive: boolean=true;