From 0df6c0b5f067806ddf821da7d3934472d8b21557 Mon Sep 17 00:00:00 2001
From: Anna Henningsen <anna@addaleax.net>
Date: Thu, 8 Jun 2017 23:13:11 +0200
Subject: [PATCH] Revert "readline: clean up event listener in onNewListener"

This reverts commit 81ddeb98f6a0f7a5165cd6e3b87cdaa550fdf3ec.

Ref: https://github.com/nodejs/node/pull/13266
PR-URL: https://github.com/nodejs/node/pull/13560
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
---
 lib/readline.js                             | 3 ---
 test/parallel/test-readline-set-raw-mode.js | 6 ++----
 2 files changed, 2 insertions(+), 7 deletions(-)

diff --git a/lib/readline.js b/lib/readline.js
index 6a1ed150d72cd2..60864f40afdbc1 100644
--- a/lib/readline.js
+++ b/lib/readline.js
@@ -1039,9 +1039,6 @@ function emitKeypressEvents(stream, iface) {
   } else {
     stream.on('newListener', onNewListener);
   }
-  if (iface) {
-    iface.once('close', () => { stream.removeListener('data', onData); });
-  }
 }
 
 /**
diff --git a/test/parallel/test-readline-set-raw-mode.js b/test/parallel/test-readline-set-raw-mode.js
index ddffd131d0baff..db42a5a9495a9e 100644
--- a/test/parallel/test-readline-set-raw-mode.js
+++ b/test/parallel/test-readline-set-raw-mode.js
@@ -74,8 +74,6 @@ assert(!rawModeCalled);
 assert(resumeCalled);
 assert(!pauseCalled);
 
-// One data listener for the keypress events.
-assert.strictEqual(stream.listeners('data').length, 1);
 
 // close() should call setRawMode(false)
 expectedRawMode = false;
@@ -88,5 +86,5 @@ assert(!resumeCalled);
 assert(pauseCalled);
 
 assert.deepStrictEqual(stream.listeners('keypress'), []);
-// Data listener is removed once interface is closed.
-assert.strictEqual(stream.listeners('data').length, 0);
+// one data listener for the keypress events.
+assert.strictEqual(stream.listeners('data').length, 1);