events: preset usingDomains to false

The line setting this was removed in a previous commit. This potentially breaks code in the wild using this property. Refs: #17403 (comment) PR-URL: #18944 Reviewed-By: Vladimir de Turckheim <vlad2t@hotmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
nodejs · Feb 23, 2018 · 761caec · stenalpjolly · Mar 2, 2018 · richardlau
1 parent 927c1b1
commit 761caec
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/lib/events.js b/lib/events.js
@@ -31,6 +31,8 @@ module.exports = EventEmitter;
 // Backwards-compat with node 0.10.x
 EventEmitter.EventEmitter = EventEmitter;
 
+EventEmitter.usingDomains = false;
+
 EventEmitter.prototype._events = undefined;
 EventEmitter.prototype._eventsCount = 0;
 EventEmitter.prototype._maxListeners = undefined;

diff --git a/test/parallel/test-event-emitter-subclass.js b/test/parallel/test-event-emitter-subclass.js
@@ -36,6 +36,7 @@ function MyEE(cb) {
 
 const myee = new MyEE(common.mustCall());
 
+myee.hasOwnProperty('usingDomains');
 
 util.inherits(ErrorEE, EventEmitter);
 function ErrorEE() {