node: deprecate process.EventEmitter

The comment stating it was deprecated was added in 2011 via
4ef8f06. It is time to
actually deprecate it.

PR-URL: #5049
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>

Author: evanlucas

src/node.js

@@ -20,7 +20,17 @@
     });
     EventEmitter.call(process);
 
-    process.EventEmitter = EventEmitter; // process.EventEmitter is deprecated
+    let eeWarned = false;
+    Object.defineProperty(process, 'EventEmitter', {
+      get() {
+        const internalUtil = NativeModule.require('internal/util');
+        eeWarned = internalUtil.printDeprecationMessage(
+          `process.EventEmitter is deprecated. Use require('events') instead.`,
+          eeWarned
+        );
+        return EventEmitter;
+      }
+    });
 
     startup.setupProcessObject();