Skip to content

NameFrom: Heading... WG preferred DFS over IDS/IDDFS search - #50507

Merged
cookiecrook merged 2 commits into
web-platform-tests:masterfrom
cookiecrook:namefrom-heading2
Mar 1, 2025
Merged

NameFrom: Heading... WG preferred DFS over IDS/IDDFS search#50507
cookiecrook merged 2 commits into
web-platform-tests:masterfrom
cookiecrook:namefrom-heading2

Conversation

@cookiecrook

@cookiecrook cookiecrook commented Feb 5, 2025

Copy link
Copy Markdown
Contributor

Comment on lines +49 to +50
<!-- Depth first search (DFS) tests (standard DFS rather than iterative-deepening depth-first search IDDFS/IDS) -->
<!-- Note: The WG wanted change this from the initially proposed IDS to DFS, but IDS is more performant, so we'll need multiple implementor buy-in for the selection. -->

@cookiecrook cookiecrook Feb 5, 2025

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewers... This test is the only substantive change since last year's approved PR... I updated to match the WG's preference for a depth-first-search rather than the alternative IDS/IDDFS.

@cookiecrook

Copy link
Copy Markdown
Contributor Author

The failing results are expected until implementations ship an update to add NameFrom: Heading... WPT is ready to merge with one reviewer.

Comment thread accname/name/comp_name_from_heading.tentative.html Outdated
@cookiecrook
cookiecrook merged commit 5bbb8c3 into web-platform-tests:master Mar 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add nameFrom: heading tests once the AccName change is reviewed

4 participants