Skip to content

Commit f6894dc

Browse files
authored
Set fragment export flags to true (#11672)
1 parent a65a8ab commit f6894dc

File tree

9 files changed

+2
-19
lines changed

9 files changed

+2
-19
lines changed

packages/react-cs-renderer/src/ReactNativeCSFeatureFlags.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@ import typeof * as CSFeatureFlagsType from './ReactNativeCSFeatureFlags';
1515
export const debugRenderPhaseSideEffects = false;
1616
export const enableAsyncSubtreeAPI = true;
1717
export const enableAsyncSchedulingByDefaultInReactDOM = false;
18-
export const enableReactFragment = false;
1918
export const enableCreateRoot = false;
2019
export const enableUserTimingAPI = __DEV__;
2120

packages/react-dom/src/__tests__/ReactDOMServerIntegrationFragment-test.internal.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,6 @@ let ReactDOMServer;
1818
function initModules() {
1919
// Reset warning cache.
2020
jest.resetModuleRegistry();
21-
require('shared/ReactFeatureFlags').enableReactFragment = true;
2221
React = require('react');
2322
ReactDOM = require('react-dom');
2423
ReactDOMServer = require('react-dom/server');

packages/react-native-renderer/src/ReactNativeFeatureFlags.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,6 @@ export const {debugRenderPhaseSideEffects} = require('ReactFeatureFlags');
1818
// The rest of the flags are static for better dead code elimination.
1919
export const enableAsyncSubtreeAPI = true;
2020
export const enableAsyncSchedulingByDefaultInReactDOM = false;
21-
export const enableReactFragment = false;
2221
export const enableCreateRoot = false;
2322
export const enableUserTimingAPI = __DEV__;
2423
export const enableMutatingReconciler = true;

packages/react-reconciler/src/ReactChildFiber.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ import type {ReactCall, ReactPortal, ReactReturn} from 'shared/ReactTypes';
1212
import type {Fiber} from 'react-reconciler/src/ReactFiber';
1313
import type {ExpirationTime} from 'react-reconciler/src/ReactFiberExpirationTime';
1414

15-
import {enableReactFragment} from 'shared/ReactFeatureFlags';
1615
import {Placement, Deletion} from 'shared/ReactTypeOfSideEffect';
1716
import {
1817
getIteratorFn,
@@ -1376,7 +1375,6 @@ function ChildReconciler(shouldTrackSideEffects) {
13761375
// This leads to an ambiguity between <>{[...]}</> and <>...</>.
13771376
// We treat the ambiguous cases above the same.
13781377
if (
1379-
enableReactFragment &&
13801378
typeof newChild === 'object' &&
13811379
newChild !== null &&
13821380
newChild.type === REACT_FRAGMENT_TYPE &&

packages/react-reconciler/src/__tests__/ReactFragment-test.internal.js

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,9 +15,6 @@ describe('ReactFragment', () => {
1515
beforeEach(function() {
1616
jest.resetModules();
1717

18-
const ReactFeatureFlags = require('shared/ReactFeatureFlags');
19-
ReactFeatureFlags.enableReactFragment = true;
20-
2118
React = require('react');
2219
ReactNoop = require('react-noop-renderer');
2320
});

packages/react/src/React.js

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@
77

88
import assign from 'object-assign';
99
import ReactVersion from 'shared/ReactVersion';
10-
import {enableReactFragment} from 'shared/ReactFeatureFlags';
1110
import {REACT_FRAGMENT_TYPE} from 'shared/ReactSymbols';
1211

1312
import {Component, PureComponent, AsyncComponent} from './ReactBaseClasses';
@@ -39,6 +38,8 @@ var React = {
3938
PureComponent,
4039
unstable_AsyncComponent: AsyncComponent,
4140

41+
Fragment: REACT_FRAGMENT_TYPE,
42+
4243
createElement: __DEV__ ? createElementWithValidation : createElement,
4344
cloneElement: __DEV__ ? cloneElementWithValidation : cloneElement,
4445
createFactory: __DEV__ ? createFactoryWithValidation : createFactory,
@@ -53,10 +54,6 @@ var React = {
5354
},
5455
};
5556

56-
if (enableReactFragment) {
57-
React.Fragment = REACT_FRAGMENT_TYPE;
58-
}
59-
6057
if (__DEV__) {
6158
Object.assign(React.__SECRET_INTERNALS_DO_NOT_USE_OR_YOU_WILL_BE_FIRED, {
6259
// These should not be included in production.

packages/react/src/__tests__/ReactJSXElementValidator-test.internal.js

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,9 +16,6 @@ describe('ReactJSXElementValidator', () => {
1616
beforeEach(() => {
1717
jest.resetModules();
1818

19-
const ReactFeatureFlags = require('shared/ReactFeatureFlags');
20-
ReactFeatureFlags.enableReactFragment = true;
21-
2219
React = require('react');
2320
ReactTestUtils = require('react-dom/test-utils');
2421
});

packages/shared/ReactFeatureFlags.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,6 @@ import invariant from 'fbjs/lib/invariant';
1111

1212
export const enableAsyncSubtreeAPI = true;
1313
export const enableAsyncSchedulingByDefaultInReactDOM = false;
14-
// Exports React.Fragment
15-
export const enableReactFragment = false;
1614
// Exports ReactDOM.createRoot
1715
export const enableCreateRoot = false;
1816
export const enableUserTimingAPI = __DEV__;

scripts/rollup/shims/rollup/ReactFeatureFlags-www.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,6 @@ export const {
1818

1919
// The rest of the flags are static for better dead code elimination.
2020
export const enableAsyncSubtreeAPI = true;
21-
export const enableReactFragment = false;
2221
export const enableCreateRoot = true;
2322

2423
// The www bundles only use the mutating reconciler.

0 commit comments

Comments
 (0)