fixup! lib: refactor internal webidl converters

Signed-off-by: Filip Skokan <[email protected]>

Author: panva

lib/internal/webidl.js

@@ -770,8 +770,8 @@ function createSequenceConverter(converter) {
           'cannot be converted to sequence.',
           options);
       }
-      // Step 3.2: return the sequence when the iterator is done.
-      if (next.done === true) {
+      // Step 3.2: IteratorComplete applies ToBoolean(done).
+      if (next.done) {
         break;
       }
       // Step 3.3: convert next to an IDL value of type T.

test/parallel/test-internal-webidl.js

@@ -395,6 +395,16 @@ assert.throws(() => webidl.requiredArguments(1, 2, opts), {
     message: 'Prefix: Context[0] is a Symbol and cannot be converted ' +
       'to a string.',
   });
+
+  assert.deepStrictEqual(converter({
+    [Symbol.iterator]() {
+      return {
+        next() {
+          return { done: 1, value: Symbol() };
+        },
+      };
+    },
+  }), []);
 }
 
 {