fix: missing extattr validation for arguments/operations (#567)

Author: saschanaz

lib/productions/argument.js

@@ -61,6 +61,7 @@ export class Argument extends Base {
    * @param {import("../validator.js").Definitions} defs
    */
   *validate(defs) {
+    yield* this.extAttrs.validate(defs);
     yield* this.idlType.validate(defs);
     const result = idlTypeIncludesDictionary(this.idlType, defs, {
       useNullableInner: true,

lib/productions/operation.js

@@ -63,6 +63,7 @@ export class Operation extends Base {
   }
 
   *validate(defs) {
+    yield* this.extAttrs.validate(defs);
     if (!this.name && ["", "static"].includes(this.special)) {
       const message = `Regular or static operations must have both a return type and an identifier.`;
       yield validationError(this.tokens.open, this, "incomplete-op", message);

test/autofix.js

@@ -299,7 +299,9 @@ describe("Writer template functions", () => {
         [LenientSetter] readonly attribute DOMString lenientSetter;
         [LenientThis] readonly attribute DOMString lenientThis;
         attribute [TreatNullAs] DOMString treatNullAs;
+        undefined treatNullAsOp([TreatNullAs] DOMString str);
         [Unforgeable] readonly attribute DOMString unforgeable;
+        [Unforgeable] DOMString unforgeableOp();
       };
 
       [TreatNonObjectAsNull]
@@ -314,7 +316,9 @@ describe("Writer template functions", () => {
         [LegacyLenientSetter] readonly attribute DOMString lenientSetter;
         [LegacyLenientThis] readonly attribute DOMString lenientThis;
         attribute [LegacyNullToEmptyString] DOMString treatNullAs;
+        undefined treatNullAsOp([LegacyNullToEmptyString] DOMString str);
         [LegacyUnforgeable] readonly attribute DOMString unforgeable;
+        [LegacyUnforgeable] DOMString unforgeableOp();
       };
 
       [LegacyTreatNonObjectAsNull]