Add toggle to unpatch global console

Author: Jchinonso

packages/react-devtools-shared/src/devtools/views/Settings/DebuggingSettings.js

@@ -46,7 +46,7 @@ export default function DebuggingSettings(_: {||}) {
               setEnableDoubleLogging(currentTarget.checked)
             }
           />{' '}
-          Enable double logging
+          Suppress console during development-only second render pass
         </label>
       </div>
       <div className={styles.Setting}>

packages/react-devtools-shared/src/devtools/views/Settings/SettingsContext.js

@@ -89,7 +89,7 @@ function SettingsContextController({
     setEnableDoubleLogging,
   ] = useLocalStorage<boolean>(
     LOCAL_STORAGE_SHOULD_ENABLE_DOUBLE_LOGGING,
-    false,
+    true,
   );
 
   const [

packages/shared/ConsolePatchingDev.js

@@ -11,6 +11,7 @@
 // replaying on render function. This currently only patches the object
 // lazily which won't cover if the log function was extracted eagerly.
 // We could also eagerly patch the method.
+import {canUseDOM} from './ExecutionEnvironment';
 
 let disabledDepth = 0;
 let prevLog;
@@ -20,13 +21,16 @@ let prevError;
 let prevGroup;
 let prevGroupCollapsed;
 let prevGroupEnd;
+const {
+  __REACT_DEVTOOLS_ENABLE_DOUBLE_LOGGING__: enableDoubleLogging,
+} = canUseDOM ? window : global;
 
 function disabledLog() {}
 disabledLog.__reactDisabledLog = true;
 
 export function disableLogs(): void {
   if (__DEV__) {
-    if (disabledDepth === 0) {
+    if (disabledDepth === 0 && !enableDoubleLogging) {
       /* eslint-disable react-internal/no-production-logging */
       prevLog = console.log;
       prevInfo = console.info;
@@ -61,7 +65,7 @@ export function disableLogs(): void {
 export function reenableLogs(): void {
   if (__DEV__) {
     disabledDepth--;
-    if (disabledDepth === 0) {
+    if (disabledDepth === 0 && !enableDoubleLogging) {
       /* eslint-disable react-internal/no-production-logging */
       const props = {
         configurable: true,