[web-animations-2] Web IDL validation errors

[autokagami]

🤖 This is an automatic issue report for Web IDL validation error. 🤖

The bot found validation errors but couldn't fix them automatically, since there is no proper autofix for them. Please check the following messages and fix them.

• Validation error at line 6 in web-animations-2,1, inside `interface Animation -> attribute rangeStart`:
      attribute (TimelineRangeOffset or CSSNumericValue or CSSKeywordValue
                 ^
  

  Error: Attributes cannot accept dictionary types.

• Validation error at line 7 in web-animations-2,1, inside `interface Animation -> attribute rangeEnd`:
      attribute (TimelineRangeOffset or CSSNumericValue or CSSKeywordValue
                 ^
  

  Error: Attributes cannot accept dictionary types.

Please file an issue at https://github.com/saschanaz/webidl-updater/issues/new if you think this is invalid or should be enhanced.

[tabatkins]

Note: this is for KeyframeAnimationOptions.

Indeed, dictionaries aren't allowed as attribute types (because an attribute type gives both the setter's input type and the getter's output type, and dictionaries aren't allowed as output types). There's an issue to separate getters and setters which would allow solving this, but for now I think, unfortunately, the only correct answer is to make the attribute any type and do the type-conversion manually in the setter algo.

[saschanaz]

Somehow the bot messed it up as reported in saschanaz/webidl-updater#190, rangeStart is not even an attribute but a dictionary member, right? I guess in that case there's no issue.

[saschanaz]

No actually, the bot was right and #9181 added those actual attributes yesterday:

csswg-drafts/web-animations-2/Overview.bs

Lines 2397 to 2403 in 6ac1985

	[Exposed=Window]
	partial interface Animation {
	attribute CSSNumberish? startTime;
	attribute CSSNumberish? currentTime;
	attribute (TimelineRangeOffset or CSSNumericValue or CSSKeywordValue or DOMString) rangeStart;
	attribute (TimelineRangeOffset or CSSNumericValue or CSSKeywordValue or DOMString) rangeEnd;
	};

But then the build failed and was not applied to the published editor's draft.

@kevers-google, do you want to take a look? (I wonder how the CI build never happened in the PR but only the IPR check)

[kevers-google]

I'll take a look.