Check nativeCallSyncHook on both dev and ship (#92)

statm · web-flow · commit 4a211cc136c5 · 2019-06-18T13:57:43.000-07:00
diff --git a/Libraries/ReactNative/UIManager.js b/Libraries/ReactNative/UIManager.js
@@ -59,10 +59,8 @@ UIManager.getViewManagerConfig = function(viewManagerName: string) {
 
   // If we're in the Chrome Debugger, let's not even try calling the sync
   // method.
-  if (__DEV__) {
-    if (!global.nativeCallSyncHook) {
-      return config;
-    }
+  if (!global.nativeCallSyncHook) {
+    return config;
   }
 
   if (UIManager.lazilyLoadView && !triedLoadingConfig.has(viewManagerName)) {
@@ -155,7 +153,7 @@ if (
   }
 }
 
-if (__DEV__) {
+if (!global.nativeCallSyncHook) {
   Object.keys(UIManager).forEach(viewManagerName => {
     if (!UIManagerProperties.includes(viewManagerName)) {
       if (!viewManagerConfigs[viewManagerName]) {

Original file line number	Diff line number	Diff line change
`@@ -59,10 +59,8 @@ UIManager.getViewManagerConfig = function(viewManagerName: string) {`
`59`	`59`
`60`	`60`	`// If we're in the Chrome Debugger, let's not even try calling the sync`
`61`	`61`	`// method.`
`62`		`- if (__DEV__) {`
`63`		`- if (!global.nativeCallSyncHook) {`
`64`		`- return config;`
`65`		`- }`
	`62`	`+ if (!global.nativeCallSyncHook) {`
	`63`	`+ return config;`
`66`	`64`	`}`
`67`	`65`
`68`	`66`	`if (UIManager.lazilyLoadView && !triedLoadingConfig.has(viewManagerName)) {`
`@@ -155,7 +153,7 @@ if (`
`155`	`153`	`}`
`156`	`154`	`}`
`157`	`155`
`158`		`-if (__DEV__) {`
	`156`	`+if (!global.nativeCallSyncHook) {`
`159`	`157`	`Object.keys(UIManager).forEach(viewManagerName => {`
`160`	`158`	`if (!UIManagerProperties.includes(viewManagerName)) {`
`161`	`159`	`if (!viewManagerConfigs[viewManagerName]) {`