bootstrap: bind properties of the global console in the default snapshot

lib/internal/console/constructor.js

@@ -90,7 +90,6 @@ const kIsConsole = Symbol('kIsConsole');
 const kWriteToConsole = Symbol('kWriteToConsole');
 const kBindProperties = Symbol('kBindProperties');
 const kBindStreamsEager = Symbol('kBindStreamsEager');
-const kBindStreamsLazy = Symbol('kBindStreamsLazy');
 const kUseStdout = Symbol('kUseStdout');
 const kUseStderr = Symbol('kUseStderr');
 
@@ -192,38 +191,6 @@ ObjectDefineProperties(Console.prototype, {
       });
     },
   },
-  [kBindStreamsLazy]: {
-    __proto__: null,
-    ...consolePropAttributes,
-    // Lazily load the stdout and stderr from an object so we don't
-    // create the stdio streams when they are not even accessed
-    value: function(object) {
-      let stdout;
-      let stderr;
-      ObjectDefineProperties(this, {
-        '_stdout': {
-          __proto__: null,
-          enumerable: false,
-          configurable: true,
-          get() {
-            if (!stdout) stdout = object.stdout;
-            return stdout;
-          },
-          set(value) { stdout = value; },
-        },
-        '_stderr': {
-          __proto__: null,
-          enumerable: false,
-          configurable: true,
-          get() {
-            if (!stderr) { stderr = object.stderr; }
-            return stderr;
-          },
-          set(value) { stderr = value; },
-        },
-      });
-    },
-  },
   [kBindProperties]: {
     __proto__: null,
     ...consolePropAttributes,
@@ -685,9 +652,6 @@ Console.prototype.error = Console.prototype.warn;
 Console.prototype.groupCollapsed = Console.prototype.group;
 
 function initializeGlobalConsole(globalConsole) {
-  globalConsole[kBindStreamsLazy](process);
-  globalConsole[kBindProperties](true, 'auto');
-
   const {
     namespace: {
       addSerializeCallback,
@@ -714,12 +678,15 @@ function initializeGlobalConsole(globalConsole) {
     for (const key of inspectorConsoleKeys) {
       globalConsole[key] = undefined;
     }
+    // Reset the streams so they are re-initialized after the user-land
+    // snapshot is deserialized.
+    globalConsole._stdout = undefined;
+    globalConsole._stderr = undefined;
   });
 }
 
 module.exports = {
   Console,
-  kBindStreamsLazy,
   kBindProperties,
   initializeGlobalConsole,
   formatTime, // exported for tests

lib/internal/console/global.js

@@ -14,13 +14,15 @@
 
 const {
   FunctionPrototypeBind,
+  ObjectDefineProperties,
   ReflectDefineProperty,
   ReflectGetOwnPropertyDescriptor,
   ReflectOwnKeys,
 } = primordials;
 
 const {
   Console,
+  kBindProperties,
 } = require('internal/console/constructor');
 
 const globalConsole = { __proto__: {} };
@@ -41,6 +43,36 @@ for (const prop of ReflectOwnKeys(Console.prototype)) {
   ReflectDefineProperty(globalConsole, prop, desc);
 }
 
+let stdout;
+let stderr;
+
+// Lazily load the stdout and stderr from the process object so we don't
+// create the stdio streams when they are not even accessed.
+ObjectDefineProperties(globalConsole, {
+  '_stdout': {
+    __proto__: null,
+    enumerable: false,
+    configurable: true,
+    get() {
+      if (!stdout) stdout = process.stdout;
+      return stdout;
+    },
+    set(value) { stdout = value; },
+  },
+  '_stderr': {
+    __proto__: null,
+    enumerable: false,
+    configurable: true,
+    get() {
+      if (!stderr) { stderr = process.stderr; }
+      return stderr;
+    },
+    set(value) { stderr = value; },
+  },
+});
+
+globalConsole[kBindProperties](true, 'auto');
+
 // This is a legacy feature - the Console constructor is exposed on
 // the global console instance.
 globalConsole.Console = Console;