process: maintain constructor descriptor

bengl · MylesBorins · commit 2462fd800903 · 2017-03-28T19:12:50.000-04:00
Use the original property descriptor instead of just taking the value, which would, by default, be non-writable and non-configurable. PR-URL: #9306 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
diff --git a/lib/internal/bootstrap_node.js b/lib/internal/bootstrap_node.js
@@ -13,10 +13,9 @@
     const EventEmitter = NativeModule.require('events');
     process._eventsCount = 0;
 
+    const origProcProto = Object.getPrototypeOf(process);
     Object.setPrototypeOf(process, Object.create(EventEmitter.prototype, {
-      constructor: {
-        value: process.constructor
-      }
+      constructor: Object.getOwnPropertyDescriptor(origProcProto, 'constructor')
     }));
 
     EventEmitter.call(process);
diff --git a/test/parallel/test-process-prototype.js b/test/parallel/test-process-prototype.js
@@ -0,0 +1,15 @@
+'use strict';
+require('../common');
+const assert = require('assert');
+const EventEmitter = require('events');
+
+const proto = Object.getPrototypeOf(process);
+
+assert(proto instanceof EventEmitter);
+
+const desc = Object.getOwnPropertyDescriptor(proto, 'constructor');
+
+assert.strictEqual(desc.value, process.constructor);
+assert.strictEqual(desc.writable, true);
+assert.strictEqual(desc.enumerable, false);
+assert.strictEqual(desc.configurable, true);
