lib: bind functions on EventEmitter

[himself65]

Further modifications from #27325

bind some functions on EventEmitter so that we no need to change two functions at the same time.

for example:

before

Socket.prototype.on = function(){};
Socket.prototype.addListener = Socket.prototype.on;

now only need to code one line

Socket.prototype.on = function(){};
// or
Socket.prototype.addListener = function(){};

Checklist

• make -j4 test (UNIX), or vcbuild test (Windows) passes
• tests and/or benchmarks are included
• commit message follows commit guidelines

[mscdex]

CITGM: https://ci.nodejs.org/view/Node.js-citgm/job/citgm-smoker/1834/

[Fishrock123]

A comparative benchmark run for the events benchmarks would probably be desirable.

[bnoordhuis]

Why would this ever be the desired behavior? It breaks the principle of least surprise.

[BridgeAR]

.on is just an alias to .addListener. This does prevent some issues where someone wants to change the functionality but forgets to update all functions involved. I guess in this specific case it's fine therefore?

[himself65]

close this pull request, because I think this may break the coding style of some people