worker: add Symbol.toStringTag to messaging classes

lib/internal/worker/io.js

@@ -15,6 +15,7 @@ const {
   ReflectApply,
   Symbol,
   SymbolFor,
+  SymbolToStringTag,
 } = primordials;
 
 const {
@@ -208,6 +209,22 @@ ObjectDefineProperty(MessagePort.prototype, inspect.custom, {
   },
 });
 
+ObjectDefineProperty(MessagePort.prototype, SymbolToStringTag, {
+  __proto__: null,
+  writable: false,
+  enumerable: false,
+  configurable: true,
+  value: 'MessagePort',
+});
+
+ObjectDefineProperty(MessageChannel.prototype, SymbolToStringTag, {
+  __proto__: null,
+  writable: false,
+  enumerable: false,
+  configurable: true,
+  value: 'MessageChannel',
+});
+
 function setupPortReferencing(port, eventEmitter, eventName) {
   // Keep track of whether there are any workerMessage listeners:
   // If there are some, ref() the channel so it keeps the event loop alive.
@@ -461,6 +478,13 @@ class BroadcastChannel extends EventTarget {
 }
 
 ObjectDefineProperties(BroadcastChannel.prototype, {
+  [SymbolToStringTag]: {
+    __proto__: null,
+    writable: false,
+    enumerable: false,
+    configurable: true,
+    value: 'BroadcastChannel',
+  },
   name: kEnumerableProperty,
   close: kEnumerableProperty,
   postMessage: kEnumerableProperty,

test/parallel/test-eventtarget-memoryleakwarning.js

@@ -15,12 +15,12 @@ common.expectWarning({
         'EventTarget. MaxListeners is 2. Use events.setMaxListeners() ' +
      'to increase limit'],
     ['Possible EventTarget memory leak detected. 3 foo listeners added to ' +
-        '[MessagePort [EventTarget]]. ' +
+        '[MessagePort]. ' +
         'MaxListeners is 2. ' +
         'Use events.setMaxListeners() to increase ' +
      'limit'],
     ['Possible EventTarget memory leak detected. 3 foo listeners added to ' +
-        '[MessagePort [EventTarget]]. ' +
+        '[MessagePort]. ' +
         'MaxListeners is 2. ' +
         'Use events.setMaxListeners() to increase ' +
      'limit'],

test/parallel/test-messaging-string-tag.js

@@ -0,0 +1,21 @@
+'use strict';
+
+require('../common');
+
+const assert = require('assert');
+
+const classesToBeTested = [ MessageChannel, MessagePort, BroadcastChannel ];
+
+classesToBeTested.forEach((cls) => {
+  assert.strictEqual(cls.prototype[Symbol.toStringTag], cls.name);
+  assert.deepStrictEqual(Object.getOwnPropertyDescriptor(cls.prototype, Symbol.toStringTag),
+                         { configurable: true, enumerable: false, value: cls.name, writable: false });
+});
+
+const channel = new MessageChannel();
+assert.strictEqual(Object.prototype.toString.call(channel), '[object MessageChannel]');
+assert.strictEqual(Object.prototype.toString.call(channel.port1), '[object MessagePort]');
+
+const broadcastChannel = new BroadcastChannel('string-tag');
+assert.strictEqual(Object.prototype.toString.call(broadcastChannel), '[object BroadcastChannel]');
+broadcastChannel.close();

test/parallel/test-worker-message-port-inspect-during-init-hook.js

@@ -12,7 +12,7 @@ async_hooks.createHook({
   init: common.mustCall((id, type, triggerId, resource) => {
     assert.strictEqual(
       util.inspect(resource),
-      'MessagePort [EventTarget] { active: true, refed: false }');
+      'MessagePort { active: true, refed: false }');
   }, 2)
 }).enable();