[css-counter-styles-3] Setting `CSSCounterStyleRule.name` should ignore symbolic counter styles

[cdoublev]

If I am not mistaken, CSSCounterStyleRule.name should ignore symbolic counter styles, because they are invalid.

The keywords decimal, disc, square, circle, disclosure-open, and disclosure-closed are valid <counter-style-name>s, but are invalid when used here to name a counter style rule; doing so makes the rule invalid (related issue: #3584).

https://drafts.csswg.org/css-counter-styles-3/#the-counter-style-rule

-  If the value is not "decimal", "disc", or "none", replace the associated rule’s name with an identifier equal to the value.
+  If the value is not "decimal", "disc", "square", "circle", "disclosure-open", "disclosure-closed", or "none", replace the associated rule’s name with an identifier equal to the value.

[Loirooriol]

Seems that Blink and Gecko already check that.
Would be better to define the list in a single place, and refer to it from the other place.

[tabatkins]

Agreed, that was an oversight in updating. I've switched to defining the list as a term, and referencing it in both places.

Agenda+ to confirm the edit.

[vitorroriz]

Seems that Blink and Gecko already check that. Would be better to define the list in a single place, and refer to it from the other place.

Sounds good. WebKit also checks it.

[css-meeting-bot]

The CSS Working Group just discussed [css-counter-styles-3] Setting `CSSCounterStyleRule.name` should ignore symbolic counter styles, and agreed to the following:

• RESOLVED: Accept edits

The full IRC log of that discussion

<emilio> TabAtkins: again obvious bug-fix, I had two places which reference the special counter-styles
<emilio> ... didn't keep them in sync
<emilio> ... so cssom had a shorter version of the list
<emilio> ... I've made the list a defined term and reference in both places
<emilio> ... only needs sign-off
<emilio> ... also impls do this correctly
<emilio> RESOLVED: Accept edits