Skip to content

Commit

Permalink
Set enableFastJSX flag to true (#30343)
Browse files Browse the repository at this point in the history
When these to diffs are landed, we can merge this

- [x] D59772879
- [x] D59773043

DiffTrain build for [d025ddd](d025ddd)
  • Loading branch information
kassens committed Jul 22, 2024
1 parent 48b7a7e commit 32ff1a4
Show file tree
Hide file tree
Showing 36 changed files with 92 additions and 148 deletions.
11 changes: 2 additions & 9 deletions compiled/facebook-www/JSXDEVRuntime-dev.classic.js
Original file line number Diff line number Diff line change
Expand Up @@ -638,11 +638,7 @@ __DEV__ &&
hasValidKey(config) &&
(checkKeyStringCoercion(config.key), (children = "" + config.key));
hasValidRef(config) && warnIfStringRefCannotBeAutoConverted(config, self);
if (
(!enableFastJSXWithoutStringRefs &&
(!enableFastJSXWithStringRefs || "ref" in config)) ||
"key" in config
) {
if ("ref" in config || "key" in config) {
maybeKey = {};
for (var propName in config)
"key" !== propName &&
Expand Down Expand Up @@ -813,7 +809,6 @@ __DEV__ &&
disableDefaultPropsExceptForClasses =
dynamicFeatureFlags.disableDefaultPropsExceptForClasses,
enableDebugTracing = dynamicFeatureFlags.enableDebugTracing,
enableFastJSX = dynamicFeatureFlags.enableFastJSX,
enableRenderableContext = dynamicFeatureFlags.enableRenderableContext,
enableTransitionTracing = dynamicFeatureFlags.enableTransitionTracing,
renameElementSymbol = dynamicFeatureFlags.renameElementSymbol,
Expand Down Expand Up @@ -867,9 +862,7 @@ __DEV__ &&
specialPropKeyWarningShown;
var didWarnAboutStringRefs = {};
var didWarnAboutElementRef = {};
var enableFastJSXWithStringRefs = enableFastJSX && !0,
enableFastJSXWithoutStringRefs = enableFastJSXWithStringRefs && !1,
didWarnAboutKeySpread = {},
var didWarnAboutKeySpread = {},
ownerHasKeyUseWarning = {};
exports.Fragment = REACT_FRAGMENT_TYPE;
exports.jsxDEV = function (
Expand Down
11 changes: 2 additions & 9 deletions compiled/facebook-www/JSXDEVRuntime-dev.modern.js
Original file line number Diff line number Diff line change
Expand Up @@ -635,11 +635,7 @@ __DEV__ &&
hasValidKey(config) &&
(checkKeyStringCoercion(config.key), (children = "" + config.key));
hasValidRef(config) && warnIfStringRefCannotBeAutoConverted(config, self);
if (
(!enableFastJSXWithoutStringRefs &&
(!enableFastJSXWithStringRefs || "ref" in config)) ||
"key" in config
) {
if ("ref" in config || "key" in config) {
maybeKey = {};
for (var propName in config)
"key" !== propName &&
Expand Down Expand Up @@ -810,7 +806,6 @@ __DEV__ &&
disableDefaultPropsExceptForClasses =
dynamicFeatureFlags.disableDefaultPropsExceptForClasses,
enableDebugTracing = dynamicFeatureFlags.enableDebugTracing,
enableFastJSX = dynamicFeatureFlags.enableFastJSX,
enableRenderableContext = dynamicFeatureFlags.enableRenderableContext,
enableTransitionTracing = dynamicFeatureFlags.enableTransitionTracing;
dynamicFeatureFlags = dynamicFeatureFlags.renameElementSymbol;
Expand Down Expand Up @@ -863,9 +858,7 @@ __DEV__ &&
specialPropKeyWarningShown;
var didWarnAboutStringRefs = {};
var didWarnAboutElementRef = {};
var enableFastJSXWithStringRefs = enableFastJSX && !0,
enableFastJSXWithoutStringRefs = enableFastJSXWithStringRefs && !1,
didWarnAboutKeySpread = {},
var didWarnAboutKeySpread = {},
ownerHasKeyUseWarning = {};
exports.Fragment = REACT_FRAGMENT_TYPE;
exports.jsxDEV = function (
Expand Down
2 changes: 1 addition & 1 deletion compiled/facebook-www/REVISION
Original file line number Diff line number Diff line change
@@ -1 +1 @@
b15c1983dcf96f19400b0ca7337be1e1fb1a8717
d025ddd3b954dfc52ad7e6a036913946a8ca2644
2 changes: 1 addition & 1 deletion compiled/facebook-www/REVISION_TRANSFORMS
Original file line number Diff line number Diff line change
@@ -1 +1 @@
b15c1983dcf96f19400b0ca7337be1e1fb1a8717
d025ddd3b954dfc52ad7e6a036913946a8ca2644
13 changes: 3 additions & 10 deletions compiled/facebook-www/React-dev.classic.js
Original file line number Diff line number Diff line change
Expand Up @@ -700,11 +700,7 @@ __DEV__ &&
hasValidKey(config) &&
(checkKeyStringCoercion(config.key), (children = "" + config.key));
hasValidRef(config) && warnIfStringRefCannotBeAutoConverted(config, self);
if (
(!enableFastJSXWithoutStringRefs &&
(!enableFastJSXWithStringRefs || "ref" in config)) ||
"key" in config
) {
if ("ref" in config || "key" in config) {
maybeKey = {};
for (var propName in config)
"key" !== propName &&
Expand Down Expand Up @@ -1202,7 +1198,6 @@ __DEV__ &&
disableDefaultPropsExceptForClasses =
dynamicFeatureFlags.disableDefaultPropsExceptForClasses,
enableDebugTracing = dynamicFeatureFlags.enableDebugTracing,
enableFastJSX = dynamicFeatureFlags.enableFastJSX,
enableRenderableContext = dynamicFeatureFlags.enableRenderableContext,
enableTransitionTracing = dynamicFeatureFlags.enableTransitionTracing,
renameElementSymbol = dynamicFeatureFlags.renameElementSymbol,
Expand Down Expand Up @@ -1319,9 +1314,7 @@ __DEV__ &&
didWarnAboutOldJSXRuntime;
var didWarnAboutStringRefs = {};
var didWarnAboutElementRef = {};
var enableFastJSXWithStringRefs = enableFastJSX && !0,
enableFastJSXWithoutStringRefs = enableFastJSXWithStringRefs && !1,
didWarnAboutKeySpread = {},
var didWarnAboutKeySpread = {},
ownerHasKeyUseWarning = {},
didWarnAboutMaps = !1,
userProvidedKeyEscapeRegex = /\/+/g,
Expand Down Expand Up @@ -1998,7 +1991,7 @@ __DEV__ &&
exports.useTransition = function () {
return resolveDispatcher().useTransition();
};
exports.version = "19.0.0-www-classic-b15c1983-20240722";
exports.version = "19.0.0-www-classic-d025ddd3-20240722";
"undefined" !== typeof __REACT_DEVTOOLS_GLOBAL_HOOK__ &&
"function" ===
typeof __REACT_DEVTOOLS_GLOBAL_HOOK__.registerInternalModuleStop &&
Expand Down
13 changes: 3 additions & 10 deletions compiled/facebook-www/React-dev.modern.js
Original file line number Diff line number Diff line change
Expand Up @@ -697,11 +697,7 @@ __DEV__ &&
hasValidKey(config) &&
(checkKeyStringCoercion(config.key), (children = "" + config.key));
hasValidRef(config) && warnIfStringRefCannotBeAutoConverted(config, self);
if (
(!enableFastJSXWithoutStringRefs &&
(!enableFastJSXWithStringRefs || "ref" in config)) ||
"key" in config
) {
if ("ref" in config || "key" in config) {
maybeKey = {};
for (var propName in config)
"key" !== propName &&
Expand Down Expand Up @@ -1199,7 +1195,6 @@ __DEV__ &&
disableDefaultPropsExceptForClasses =
dynamicFeatureFlags.disableDefaultPropsExceptForClasses,
enableDebugTracing = dynamicFeatureFlags.enableDebugTracing,
enableFastJSX = dynamicFeatureFlags.enableFastJSX,
enableRenderableContext = dynamicFeatureFlags.enableRenderableContext,
enableTransitionTracing = dynamicFeatureFlags.enableTransitionTracing,
renameElementSymbol = dynamicFeatureFlags.renameElementSymbol,
Expand Down Expand Up @@ -1315,9 +1310,7 @@ __DEV__ &&
didWarnAboutOldJSXRuntime;
var didWarnAboutStringRefs = {};
var didWarnAboutElementRef = {};
var enableFastJSXWithStringRefs = enableFastJSX && !0,
enableFastJSXWithoutStringRefs = enableFastJSXWithStringRefs && !1,
didWarnAboutKeySpread = {},
var didWarnAboutKeySpread = {},
ownerHasKeyUseWarning = {},
didWarnAboutMaps = !1,
userProvidedKeyEscapeRegex = /\/+/g,
Expand Down Expand Up @@ -1978,7 +1971,7 @@ __DEV__ &&
exports.useTransition = function () {
return resolveDispatcher().useTransition();
};
exports.version = "19.0.0-www-modern-b15c1983-20240722";
exports.version = "19.0.0-www-modern-d025ddd3-20240722";
"undefined" !== typeof __REACT_DEVTOOLS_GLOBAL_HOOK__ &&
"function" ===
typeof __REACT_DEVTOOLS_GLOBAL_HOOK__.registerInternalModuleStop &&
Expand Down
11 changes: 2 additions & 9 deletions compiled/facebook-www/React-prod.classic.js
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@
var dynamicFeatureFlags = require("ReactFeatureFlags"),
disableDefaultPropsExceptForClasses =
dynamicFeatureFlags.disableDefaultPropsExceptForClasses,
enableFastJSX = dynamicFeatureFlags.enableFastJSX,
enableRenderableContext = dynamicFeatureFlags.enableRenderableContext,
enableTransitionTracing = dynamicFeatureFlags.enableTransitionTracing,
renameElementSymbol = dynamicFeatureFlags.renameElementSymbol,
Expand Down Expand Up @@ -97,8 +96,6 @@ function getOwner() {
var dispatcher = ReactSharedInternals.A;
return null === dispatcher ? null : dispatcher.getOwner();
}
var enableFastJSXWithStringRefs = enableFastJSX && !0,
enableFastJSXWithoutStringRefs = enableFastJSXWithStringRefs && !1;
function ReactElement(type, key, _ref, self, source, owner, props) {
_ref = props.ref;
return {
Expand All @@ -114,11 +111,7 @@ function jsxProd(type, config, maybeKey) {
var key = null;
void 0 !== maybeKey && (key = "" + maybeKey);
void 0 !== config.key && (key = "" + config.key);
if (
(!enableFastJSXWithoutStringRefs &&
(!enableFastJSXWithStringRefs || "ref" in config)) ||
"key" in config
) {
if ("ref" in config || "key" in config) {
maybeKey = {};
for (var propName in config)
"key" !== propName &&
Expand Down Expand Up @@ -669,4 +662,4 @@ exports.useSyncExternalStore = function (
exports.useTransition = function () {
return ReactSharedInternals.H.useTransition();
};
exports.version = "19.0.0-www-classic-b15c1983-20240722";
exports.version = "19.0.0-www-classic-d025ddd3-20240722";
11 changes: 2 additions & 9 deletions compiled/facebook-www/React-prod.modern.js
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@
var dynamicFeatureFlags = require("ReactFeatureFlags"),
disableDefaultPropsExceptForClasses =
dynamicFeatureFlags.disableDefaultPropsExceptForClasses,
enableFastJSX = dynamicFeatureFlags.enableFastJSX,
enableRenderableContext = dynamicFeatureFlags.enableRenderableContext,
enableTransitionTracing = dynamicFeatureFlags.enableTransitionTracing,
renameElementSymbol = dynamicFeatureFlags.renameElementSymbol,
Expand Down Expand Up @@ -97,8 +96,6 @@ function getOwner() {
var dispatcher = ReactSharedInternals.A;
return null === dispatcher ? null : dispatcher.getOwner();
}
var enableFastJSXWithStringRefs = enableFastJSX && !0,
enableFastJSXWithoutStringRefs = enableFastJSXWithStringRefs && !1;
function ReactElement(type, key, _ref, self, source, owner, props) {
_ref = props.ref;
return {
Expand All @@ -114,11 +111,7 @@ function jsxProd(type, config, maybeKey) {
var key = null;
void 0 !== maybeKey && (key = "" + maybeKey);
void 0 !== config.key && (key = "" + config.key);
if (
(!enableFastJSXWithoutStringRefs &&
(!enableFastJSXWithStringRefs || "ref" in config)) ||
"key" in config
) {
if ("ref" in config || "key" in config) {
maybeKey = {};
for (var propName in config)
"key" !== propName &&
Expand Down Expand Up @@ -669,4 +662,4 @@ exports.useSyncExternalStore = function (
exports.useTransition = function () {
return ReactSharedInternals.H.useTransition();
};
exports.version = "19.0.0-www-modern-b15c1983-20240722";
exports.version = "19.0.0-www-modern-d025ddd3-20240722";
11 changes: 2 additions & 9 deletions compiled/facebook-www/React-profiling.classic.js
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@
var dynamicFeatureFlags = require("ReactFeatureFlags"),
disableDefaultPropsExceptForClasses =
dynamicFeatureFlags.disableDefaultPropsExceptForClasses,
enableFastJSX = dynamicFeatureFlags.enableFastJSX,
enableRenderableContext = dynamicFeatureFlags.enableRenderableContext,
enableTransitionTracing = dynamicFeatureFlags.enableTransitionTracing,
renameElementSymbol = dynamicFeatureFlags.renameElementSymbol,
Expand Down Expand Up @@ -101,8 +100,6 @@ function getOwner() {
var dispatcher = ReactSharedInternals.A;
return null === dispatcher ? null : dispatcher.getOwner();
}
var enableFastJSXWithStringRefs = enableFastJSX && !0,
enableFastJSXWithoutStringRefs = enableFastJSXWithStringRefs && !1;
function ReactElement(type, key, _ref, self, source, owner, props) {
_ref = props.ref;
return {
Expand All @@ -118,11 +115,7 @@ function jsxProd(type, config, maybeKey) {
var key = null;
void 0 !== maybeKey && (key = "" + maybeKey);
void 0 !== config.key && (key = "" + config.key);
if (
(!enableFastJSXWithoutStringRefs &&
(!enableFastJSXWithStringRefs || "ref" in config)) ||
"key" in config
) {
if ("ref" in config || "key" in config) {
maybeKey = {};
for (var propName in config)
"key" !== propName &&
Expand Down Expand Up @@ -673,7 +666,7 @@ exports.useSyncExternalStore = function (
exports.useTransition = function () {
return ReactSharedInternals.H.useTransition();
};
exports.version = "19.0.0-www-classic-b15c1983-20240722";
exports.version = "19.0.0-www-classic-d025ddd3-20240722";
"undefined" !== typeof __REACT_DEVTOOLS_GLOBAL_HOOK__ &&
"function" ===
typeof __REACT_DEVTOOLS_GLOBAL_HOOK__.registerInternalModuleStop &&
Expand Down
11 changes: 2 additions & 9 deletions compiled/facebook-www/React-profiling.modern.js
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@
var dynamicFeatureFlags = require("ReactFeatureFlags"),
disableDefaultPropsExceptForClasses =
dynamicFeatureFlags.disableDefaultPropsExceptForClasses,
enableFastJSX = dynamicFeatureFlags.enableFastJSX,
enableRenderableContext = dynamicFeatureFlags.enableRenderableContext,
enableTransitionTracing = dynamicFeatureFlags.enableTransitionTracing,
renameElementSymbol = dynamicFeatureFlags.renameElementSymbol,
Expand Down Expand Up @@ -101,8 +100,6 @@ function getOwner() {
var dispatcher = ReactSharedInternals.A;
return null === dispatcher ? null : dispatcher.getOwner();
}
var enableFastJSXWithStringRefs = enableFastJSX && !0,
enableFastJSXWithoutStringRefs = enableFastJSXWithStringRefs && !1;
function ReactElement(type, key, _ref, self, source, owner, props) {
_ref = props.ref;
return {
Expand All @@ -118,11 +115,7 @@ function jsxProd(type, config, maybeKey) {
var key = null;
void 0 !== maybeKey && (key = "" + maybeKey);
void 0 !== config.key && (key = "" + config.key);
if (
(!enableFastJSXWithoutStringRefs &&
(!enableFastJSXWithStringRefs || "ref" in config)) ||
"key" in config
) {
if ("ref" in config || "key" in config) {
maybeKey = {};
for (var propName in config)
"key" !== propName &&
Expand Down Expand Up @@ -673,7 +666,7 @@ exports.useSyncExternalStore = function (
exports.useTransition = function () {
return ReactSharedInternals.H.useTransition();
};
exports.version = "19.0.0-www-modern-b15c1983-20240722";
exports.version = "19.0.0-www-modern-d025ddd3-20240722";
"undefined" !== typeof __REACT_DEVTOOLS_GLOBAL_HOOK__ &&
"function" ===
typeof __REACT_DEVTOOLS_GLOBAL_HOOK__.registerInternalModuleStop &&
Expand Down
4 changes: 2 additions & 2 deletions compiled/facebook-www/ReactART-dev.classic.js
Original file line number Diff line number Diff line change
Expand Up @@ -17032,14 +17032,14 @@ __DEV__ &&
scheduleRoot: scheduleRoot,
setRefreshHandler: setRefreshHandler,
getCurrentFiber: getCurrentFiberForDevTools,
reconcilerVersion: "19.0.0-www-classic-b15c1983-20240722"
reconcilerVersion: "19.0.0-www-classic-d025ddd3-20240722"
});
})({
findFiberByHostInstance: function () {
return null;
},
bundleType: 1,
version: "19.0.0-www-classic-b15c1983-20240722",
version: "19.0.0-www-classic-d025ddd3-20240722",
rendererPackageName: "react-art"
});
var ClippingRectangle = TYPES.CLIPPING_RECTANGLE,
Expand Down
4 changes: 2 additions & 2 deletions compiled/facebook-www/ReactART-dev.modern.js
Original file line number Diff line number Diff line change
Expand Up @@ -16443,14 +16443,14 @@ __DEV__ &&
scheduleRoot: scheduleRoot,
setRefreshHandler: setRefreshHandler,
getCurrentFiber: getCurrentFiberForDevTools,
reconcilerVersion: "19.0.0-www-modern-b15c1983-20240722"
reconcilerVersion: "19.0.0-www-modern-d025ddd3-20240722"
});
})({
findFiberByHostInstance: function () {
return null;
},
bundleType: 1,
version: "19.0.0-www-modern-b15c1983-20240722",
version: "19.0.0-www-modern-d025ddd3-20240722",
rendererPackageName: "react-art"
});
var ClippingRectangle = TYPES.CLIPPING_RECTANGLE,
Expand Down
4 changes: 2 additions & 2 deletions compiled/facebook-www/ReactART-prod.classic.js
Original file line number Diff line number Diff line change
Expand Up @@ -10753,7 +10753,7 @@ var slice = Array.prototype.slice,
return null;
},
bundleType: 0,
version: "19.0.0-www-classic-b15c1983-20240722",
version: "19.0.0-www-classic-d025ddd3-20240722",
rendererPackageName: "react-art"
};
var internals$jscomp$inline_1386 = {
Expand Down Expand Up @@ -10784,7 +10784,7 @@ var internals$jscomp$inline_1386 = {
scheduleRoot: null,
setRefreshHandler: null,
getCurrentFiber: null,
reconcilerVersion: "19.0.0-www-classic-b15c1983-20240722"
reconcilerVersion: "19.0.0-www-classic-d025ddd3-20240722"
};
if ("undefined" !== typeof __REACT_DEVTOOLS_GLOBAL_HOOK__) {
var hook$jscomp$inline_1387 = __REACT_DEVTOOLS_GLOBAL_HOOK__;
Expand Down
4 changes: 2 additions & 2 deletions compiled/facebook-www/ReactART-prod.modern.js
Original file line number Diff line number Diff line change
Expand Up @@ -10202,7 +10202,7 @@ var slice = Array.prototype.slice,
return null;
},
bundleType: 0,
version: "19.0.0-www-modern-b15c1983-20240722",
version: "19.0.0-www-modern-d025ddd3-20240722",
rendererPackageName: "react-art"
};
var internals$jscomp$inline_1372 = {
Expand Down Expand Up @@ -10233,7 +10233,7 @@ var internals$jscomp$inline_1372 = {
scheduleRoot: null,
setRefreshHandler: null,
getCurrentFiber: null,
reconcilerVersion: "19.0.0-www-modern-b15c1983-20240722"
reconcilerVersion: "19.0.0-www-modern-d025ddd3-20240722"
};
if ("undefined" !== typeof __REACT_DEVTOOLS_GLOBAL_HOOK__) {
var hook$jscomp$inline_1373 = __REACT_DEVTOOLS_GLOBAL_HOOK__;
Expand Down
Loading

0 comments on commit 32ff1a4

Please sign in to comment.