From 0681ffedd37b1a77f44fe50f6abe93c1d2d4f38d Mon Sep 17 00:00:00 2001
From: Broda Noel <brodanoel@gmail.com>
Date: Sun, 21 May 2017 16:42:28 -0300
Subject: [PATCH] Wrap console calls into a check (#2301)

* Wrap console calls into a check

* Add another check
---
 .../react-dev-utils/webpackHotDevClient.js    | 10 +++----
 .../src/effects/proxyConsole.js               | 28 ++++++++++---------
 2 files changed, 20 insertions(+), 18 deletions(-)

diff --git a/packages/react-dev-utils/webpackHotDevClient.js b/packages/react-dev-utils/webpackHotDevClient.js
index 05c2bb2b7ab..cf4deae7c6d 100644
--- a/packages/react-dev-utils/webpackHotDevClient.js
+++ b/packages/react-dev-utils/webpackHotDevClient.js
@@ -172,7 +172,7 @@ var connection = new SockJS(
 // to avoid spamming the console. Disconnect usually happens
 // when developer stops the server.
 connection.onclose = function() {
-  if (typeof console !== 'undefined') {
+  if (typeof console !== 'undefined' && typeof console.info === 'function') {
     console.info(
       'The development server has disconnected.\nRefresh the page if necessary.'
     );
@@ -186,8 +186,8 @@ var hasCompileErrors = false;
 
 function clearOutdatedErrors() {
   // Clean up outdated compile errors, if any.
-  if (typeof console !== 'undefined') {
-    if (hasCompileErrors && typeof console.clear === 'function') {
+  if (typeof console !== 'undefined' && typeof console.clear === 'function') {
+    if (hasCompileErrors) {
       console.clear();
     }
   }
@@ -226,7 +226,7 @@ function handleWarnings(warnings) {
       errors: [],
     });
 
-    if (typeof console !== 'undefined') {
+    if (typeof console !== 'undefined' && typeof console.warn === 'function') {
       for (var i = 0; i < formatted.warnings.length; i++) {
         console.warn(stripAnsi(formatted.warnings[i]));
       }
@@ -266,7 +266,7 @@ function handleErrors(errors) {
   showErrorOverlay(formatted.errors[0]);
 
   // Also log them to the console.
-  if (typeof console !== 'undefined') {
+  if (typeof console !== 'undefined' && typeof console.error === 'function') {
     for (var i = 0; i < formatted.errors.length; i++) {
       console.error(stripAnsi(formatted.errors[i]));
     }
diff --git a/packages/react-error-overlay/src/effects/proxyConsole.js b/packages/react-error-overlay/src/effects/proxyConsole.js
index f17d01890c2..ec50ae2ca23 100644
--- a/packages/react-error-overlay/src/effects/proxyConsole.js
+++ b/packages/react-error-overlay/src/effects/proxyConsole.js
@@ -50,20 +50,22 @@ const permanentRegister = function proxyConsole(
 ) {
   if (typeof console !== 'undefined') {
     const orig = console[type];
-    console[type] = function __stack_frame_overlay_proxy_console__() {
-      try {
-        const message = arguments[0];
-        if (typeof message === 'string' && reactFrameStack.length > 0) {
-          callback(message, reactFrameStack[reactFrameStack.length - 1]);
+    if (typeof orig === 'function') {
+      console[type] = function __stack_frame_overlay_proxy_console__() {
+        try {
+          const message = arguments[0];
+          if (typeof message === 'string' && reactFrameStack.length > 0) {
+            callback(message, reactFrameStack[reactFrameStack.length - 1]);
+          }
+        } catch (err) {
+          // Warnings must never crash. Rethrow with a clean stack.
+          setTimeout(function() {
+            throw err;
+          });
         }
-      } catch (err) {
-        // Warnings must never crash. Rethrow with a clean stack.
-        setTimeout(function() {
-          throw err;
-        });
-      }
-      return orig.apply(this, arguments);
-    };
+        return orig.apply(this, arguments);
+      };
+    }
   }
 };