Remove propTypes checks for legacy context (#28324)

Part of #28207, this is easy to land in isolation. The approach I'm taking is slightly different — instead of leaving validation on for legacy context, I disable the validation (it's DEV-only) and leave just the parts that drive the runtime logic. I.e. `contexTypes` and `childContextTypes` *values* are now ignored, the keys are used just like before. DiffTrain build for commit 64e755d.
facebook · Feb 14, 2024 · 3b09625 · 3b09625
1 parent c8055d7
commit 3b09625
Show file tree

Hide file tree

Showing 13 changed files with 724 additions and 769 deletions.
diff --git a/...ook-fbsource/xplat/js/RKJSModules/vendor/react-test-renderer/cjs/ReactTestRenderer-dev.js b/...ook-fbsource/xplat/js/RKJSModules/vendor/react-test-renderer/cjs/ReactTestRenderer-dev.js
diff --git a/...ok-fbsource/xplat/js/RKJSModules/vendor/react-test-renderer/cjs/ReactTestRenderer-prod.js b/...ok-fbsource/xplat/js/RKJSModules/vendor/react-test-renderer/cjs/ReactTestRenderer-prod.js
@@ -7,7 +7,7 @@
  * @noflow
  * @nolint
  * @preventMunge
- * @generated SignedSource<<adb01bab728a3c61bc023c0af7267d95>>
+ * @generated SignedSource<<0cb4809a577a97130ecfbee50ab3e6b1>>
  */
 
 "use strict";
@@ -555,18 +555,7 @@ function insertBefore(parentInstance, child, beforeChild) {
  parentInstance.children.splice(beforeChild, 0, child);
 }
 var scheduleTimeout = setTimeout,
- cancelTimeout = clearTimeout;
-function describeComponentFrame(name, ownerName) {
- var sourceInfo = "";
- ownerName && (sourceInfo = " (created by " + ownerName + ")");
- return "\n in " + (name || "Unknown") + sourceInfo;
-}
-function describeFunctionComponentFrame(fn) {
- return fn
- ? describeComponentFrame(fn.displayName || fn.name || null, null)
- : "";
-}
-var hasOwnProperty = Object.prototype.hasOwnProperty,
+ cancelTimeout = clearTimeout,
  valueStack = [],
  index = -1;
 function createCursor(defaultValue) {
@@ -1305,6 +1294,17 @@ function commitCallbacks(updateQueue, context) {
  )
  callCallback(callbacks[updateQueue], context);
 }
+function describeComponentFrame(name, ownerName) {
+ var sourceInfo = "";
+ ownerName && (sourceInfo = " (created by " + ownerName + ")");
+ return "\n in " + (name || "Unknown") + sourceInfo;
+}
+function describeFunctionComponentFrame(fn) {
+ return fn
+ ? describeComponentFrame(fn.displayName || fn.name || null, null)
+ : "";
+}
+var hasOwnProperty = Object.prototype.hasOwnProperty;
 function shallowEqual(objA, objB) {
  if (objectIs(objA, objB)) return !0;
  if (
@@ -9169,7 +9169,7 @@ var devToolsConfig$jscomp$inline_1024 = {
  throw Error("TestRenderer does not support findFiberByHostInstance()");
  },
  bundleType: 0,
- version: "18.3.0-canary-f0e808e5b-20240214",
+ version: "18.3.0-canary-64e755d73-20240214",
  rendererPackageName: "react-test-renderer"
 };
 var internals$jscomp$inline_1205 = {
@@ -9200,7 +9200,7 @@ var internals$jscomp$inline_1205 = {
  scheduleRoot: null,
  setRefreshHandler: null,
  getCurrentFiber: null,
- reconcilerVersion: "18.3.0-canary-f0e808e5b-20240214"
+ reconcilerVersion: "18.3.0-canary-64e755d73-20240214"
 };
 if ("undefined" !== typeof __REACT_DEVTOOLS_GLOBAL_HOOK__) {
  var hook$jscomp$inline_1206 = __REACT_DEVTOOLS_GLOBAL_HOOK__;

diff --git a/...source/xplat/js/RKJSModules/vendor/react-test-renderer/cjs/ReactTestRenderer-profiling.js b/...source/xplat/js/RKJSModules/vendor/react-test-renderer/cjs/ReactTestRenderer-profiling.js
@@ -7,7 +7,7 @@
  * @noflow
  * @nolint
  * @preventMunge
- * @generated SignedSource<<56211850a5bfe49e2a3557f65407e9d3>>
+ * @generated SignedSource<<e2fda40292ee00c06b02f55448c4e2b7>>
  */
 
 "use strict";
@@ -573,18 +573,7 @@ function insertBefore(parentInstance, child, beforeChild) {
  parentInstance.children.splice(beforeChild, 0, child);
 }
 var scheduleTimeout = setTimeout,
- cancelTimeout = clearTimeout;
-function describeComponentFrame(name, ownerName) {
- var sourceInfo = "";
- ownerName && (sourceInfo = " (created by " + ownerName + ")");
- return "\n in " + (name || "Unknown") + sourceInfo;
-}
-function describeFunctionComponentFrame(fn) {
- return fn
- ? describeComponentFrame(fn.displayName || fn.name || null, null)
- : "";
-}
-var hasOwnProperty = Object.prototype.hasOwnProperty,
+ cancelTimeout = clearTimeout,
  valueStack = [],
  index = -1;
 function createCursor(defaultValue) {
@@ -1325,6 +1314,17 @@ function commitCallbacks(updateQueue, context) {
  )
  callCallback(callbacks[updateQueue], context);
 }
+function describeComponentFrame(name, ownerName) {
+ var sourceInfo = "";
+ ownerName && (sourceInfo = " (created by " + ownerName + ")");
+ return "\n in " + (name || "Unknown") + sourceInfo;
+}
+function describeFunctionComponentFrame(fn) {
+ return fn
+ ? describeComponentFrame(fn.displayName || fn.name || null, null)
+ : "";
+}
+var hasOwnProperty = Object.prototype.hasOwnProperty;
 function shallowEqual(objA, objB) {
  if (objectIs(objA, objB)) return !0;
  if (
@@ -9597,7 +9597,7 @@ var devToolsConfig$jscomp$inline_1066 = {
  throw Error("TestRenderer does not support findFiberByHostInstance()");
  },
  bundleType: 0,
- version: "18.3.0-canary-f0e808e5b-20240214",
+ version: "18.3.0-canary-64e755d73-20240214",
  rendererPackageName: "react-test-renderer"
 };
 var internals$jscomp$inline_1246 = {
@@ -9628,7 +9628,7 @@ var internals$jscomp$inline_1246 = {
  scheduleRoot: null,
  setRefreshHandler: null,
  getCurrentFiber: null,
- reconcilerVersion: "18.3.0-canary-f0e808e5b-20240214"
+ reconcilerVersion: "18.3.0-canary-64e755d73-20240214"
 };
 if ("undefined" !== typeof __REACT_DEVTOOLS_GLOBAL_HOOK__) {
  var hook$jscomp$inline_1247 = __REACT_DEVTOOLS_GLOBAL_HOOK__;

diff --git a/compiled-rn/facebook-fbsource/xplat/js/RKJSModules/vendor/react/cjs/React-dev.js b/compiled-rn/facebook-fbsource/xplat/js/RKJSModules/vendor/react/cjs/React-dev.js
@@ -24,7 +24,7 @@ if (__DEV__) {
  ) {
  __REACT_DEVTOOLS_GLOBAL_HOOK__.registerInternalModuleStart(new Error());
  }
- var ReactVersion = "18.3.0-canary-f0e808e5b-20240214";
+ var ReactVersion = "18.3.0-canary-64e755d73-20240214";
 
  // ATTENTION
  // When adding new symbols to this file,

diff --git a/compiled-rn/facebook-fbsource/xplat/js/RKJSModules/vendor/react/cjs/React-prod.js b/compiled-rn/facebook-fbsource/xplat/js/RKJSModules/vendor/react/cjs/React-prod.js
@@ -590,4 +590,4 @@ exports.useSyncExternalStore = function (
 exports.useTransition = function () {
  return ReactCurrentDispatcher.current.useTransition();
 };
-exports.version = "18.3.0-canary-f0e808e5b-20240214";
+exports.version = "18.3.0-canary-64e755d73-20240214";
diff --git a/compiled-rn/facebook-fbsource/xplat/js/RKJSModules/vendor/react/cjs/React-profiling.js b/compiled-rn/facebook-fbsource/xplat/js/RKJSModules/vendor/react/cjs/React-profiling.js
@@ -586,7 +586,7 @@ exports.useSyncExternalStore = function (
 exports.useTransition = function () {
  return ReactCurrentDispatcher.current.useTransition();
 };
-exports.version = "18.3.0-canary-f0e808e5b-20240214";
+exports.version = "18.3.0-canary-64e755d73-20240214";
 "undefined" !== typeof __REACT_DEVTOOLS_GLOBAL_HOOK__ &&
  "function" ===
  typeof __REACT_DEVTOOLS_GLOBAL_HOOK__.registerInternalModuleStop &&

diff --git a/compiled-rn/facebook-fbsource/xplat/js/react-native-github/Libraries/Renderer/REVISION b/compiled-rn/facebook-fbsource/xplat/js/react-native-github/Libraries/Renderer/REVISION
@@ -1 +1 @@
-f0e808e5bcbfdaa24a8a0893cd718f4d189f2369
+64e755d7374d753c75dc16ba1a4a09ce7b8689dd