mirror of
https://github.com/nodejs/node.git
synced 2026-10-14 10:26:26 -04:00
worker: make MessagePort inherit from EventTarget
Use `NodeEventTarget` to provide a mixed `EventEmitter`/`EventTarget` API interface. PR-URL: https://github.com/nodejs/node/pull/34057 Refs: https://twitter.com/addaleax/status/1276289101671608320 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: David Carlier <devnexen@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
This commit is contained in:
8 files changed
+124
-69
No files matched your search
@@ -4,6 +4,7 @@ const common = require('../common.js');
|
||||
const { MessageChannel } = require('worker_threads');
|
||||
const bench = common.createBenchmark(main, {
|
||||
payload: ['string', 'object'],
|
||||
style: ['eventtarget', 'eventemitter'],
|
||||
n: [1e6]
|
||||
});
|
||||
|
||||
@@ -25,14 +26,26 @@ function main(conf) {
|
||||
const { port1, port2 } = new MessageChannel();
|
||||
|
||||
let messages = 0;
|
||||
port2.onmessage = () => {
|
||||
function listener() {
|
||||
if (messages++ === n) {
|
||||
bench.end(n);
|
||||
port1.close();
|
||||
} else {
|
||||
write();
|
||||
}
|
||||
};
|
||||
}
|
||||
|
||||
switch (conf.style) {
|
||||
case 'eventtarget':
|
||||
port2.onmessage = listener;
|
||||
break;
|
||||
case 'eventemitter':
|
||||
port2.on('message', listener);
|
||||
break;
|
||||
default:
|
||||
throw new Error('Unsupported listener type');
|
||||
}
|
||||
|
||||
bench.start();
|
||||
write();
|
||||
|
||||
|
||||
Reference in new issue
Block a user