From 801dd2d133898749b662a537ac3c0183a384a353 Mon Sep 17 00:00:00 2001 From: Ben Alpert Date: Mon, 17 Aug 2015 14:56:10 -0700 Subject: [PATCH] [ReactNative] Don't redbox for React warnings when not using Chrome executor --- react-packager/src/DependencyResolver/polyfills/console.js | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/react-packager/src/DependencyResolver/polyfills/console.js b/react-packager/src/DependencyResolver/polyfills/console.js index ff2ff39f..e0459740 100644 --- a/react-packager/src/DependencyResolver/polyfills/console.js +++ b/react-packager/src/DependencyResolver/polyfills/console.js @@ -376,6 +376,12 @@ var str = Array.prototype.map.call(arguments, function(arg) { return inspect(arg, {depth: 10}); }).join(', '); + if (str.slice(0, 10) === "'Warning: " && level >= LOG_LEVELS.error) { + // React warnings use console.error so that a stack trace is shown, + // but we don't (currently) want these to show a redbox + // (Note: Logic duplicated in ExceptionsManager.js.) + level = LOG_LEVELS.warn; + } global.nativeLoggingHook(str, level); }; }