@samitouri / QOS-React / commits / 66bae80b08

Move validation calls behind DEV blocks (#8999)

This was done for Fiber but not Stack, causing these to throw in prod.

Sebastian Markbåge committed Feb 14, 2017 at 11:10 UTC 66bae80b0896456e54c2358aacb22bb331eebad2
1 file changed +9 -3
src/renderers/shared/stack/reconciler/ReactUpdateQueue.js
+9 -3
@@ -145,7 +145,9 @@ var ReactUpdateQueue = {
145
146 if (callback) {
147 callback = callback === undefined ? null : callback;
148 - warnOnInvalidCallback(callback, callerName);
148 + if (__DEV__) {
149 + warnOnInvalidCallback(callback, callerName);
150 + }
151 if (internalInstance._pendingCallbacks) {
152 internalInstance._pendingCallbacks.push(callback);
153 } else {
@@ -186,7 +188,9 @@ var ReactUpdateQueue = {
188
189 if (callback) {
190 callback = callback === undefined ? null : callback;
189 - warnOnInvalidCallback(callback, callerName);
191 + if (__DEV__) {
192 + warnOnInvalidCallback(callback, callerName);
193 + }
194 if (internalInstance._pendingCallbacks) {
195 internalInstance._pendingCallbacks.push(callback);
196 } else {
@@ -235,7 +239,9 @@ var ReactUpdateQueue = {
239
240 if (callback) {
241 callback = callback === undefined ? null : callback;
238 - warnOnInvalidCallback(callback, callerName);
242 + if (__DEV__) {
243 + warnOnInvalidCallback(callback, callerName);
244 + }
245 if (internalInstance._pendingCallbacks) {
246 internalInstance._pendingCallbacks.push(callback);
247 } else {