Skip to content

merge_pr_61359

Choose a tag to compare

View all tags
@github-actions github-actions released this 17 Jul 05:23
merge_pr_61359
36f4a33

Handle values of

attribute more consistently.

This tries to be more consistent at implementing the spec concepts that:

  • the missing value default is the None state
  • the empty value default is the Multiple state
  • the invalid value default is the Multiple state

This aligns with how I specified the checkable attribute's reflection on
the menu-elements branch in
whatwg/html@3373ac0

The tests were initially AI-generated but I then duplicated them to test
both empty string and invalid values. The empty string dynamic change
tests fail without this CL. The others help to test the spec prose
around missing value default and invalid value default.

Bug: 406566432
Change-Id: I59ded48c8973e6337efc75e5743e711d1eeb58a2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/8102282
Reviewed-by: Joey Arhar jarhar@chromium.org
Commit-Queue: David Baron dbaron@chromium.org
Cr-Commit-Position: refs/heads/main@{#1663573}

Assets 8
Loading