Skip to content

Update 'ancestor origins list' xref#1812

Open
zcorpan wants to merge 2 commits intow3c:mainfrom
zcorpan:ancestorOrigins-on-document
Open

Update 'ancestor origins list' xref#1812
zcorpan wants to merge 2 commits intow3c:mainfrom
zcorpan:ancestorOrigins-on-document

Conversation

@zcorpan
Copy link

@zcorpan zcorpan commented Jan 22, 2026

This term is now a list (not a DOMTokenList) on Document (not Location).

See whatwg/html#12071


Preview | Diff

This term is now a list (not a DOMTokenList) on Document (not Location).

See whatwg/html#12071
The authors would also like to thank Dimitri Glazkov for his scripts and formatting tools which have been essential in the production of this specification. The authors are also grateful for his considerable guidance.

Thanks also to Vivian Cromwell, Greg Simon, Alex Komoroske, Wonsuk Lee, and Seojin Kim for their considerable professional support.
</section> No newline at end of file
Copy link
Collaborator

Choose a reason for hiding this comment

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

nit: newline?

Copy link
Author

Choose a reason for hiding this comment

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

I added a newline (or my editor did automatically).

@monica-ch
Copy link
Collaborator

LGTM

@yoshisatoyanagisawa
Copy link
Collaborator

Maybe ancestor origin lists are moved from Location to Document, this might be a follow up for that, right?
https://whatpr.org/html/12071/90c74b7...6bf7fe6/nav-history-apis.html
https://whatpr.org/html/12071/90c74b7...6bf7fe6/dom.html#ancestor-origins

Please resolve the newline as Monica asked.

index.bs Outdated
1. Let |focusState| be the result of running the [=has focus steps=] with |browsingContext|'s [=active document=] as the argument.
1. Let |ancestorOriginsList| be the empty list.
1. If |client| is a [=window client=], set |ancestorOriginsList| to |browsingContext|'s [=active document=]'s [=relevant global object=]'s {{Location}} object's [=Location/ancestor origins list=]'s associated list.
1. If |client| is a [=window client=], set |ancestorOriginsList| to |browsingContext|'s [=active document=]'s [=Document/ancestor origins list=].
Copy link
Author

Choose a reason for hiding this comment

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

I've switched it back to being a DOMStringList in whatwg/html#12071

Suggested change
1. If |client| is a [=window client=], set |ancestorOriginsList| to |browsingContext|'s [=active document=]'s [=Document/ancestor origins list=].
1. If |client| is a [=window client=], set |ancestorOriginsList| to |browsingContext|'s [=active document=]'s [=Document/ancestor origins list=]'s associated list.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants