Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Should browsingContext.contextDestroyed event payload maybe also contain children? #573

Closed
lutien opened this issue Oct 19, 2023 · 0 comments · Fixed by #578
Closed

Should browsingContext.contextDestroyed event payload maybe also contain children? #573

lutien opened this issue Oct 19, 2023 · 0 comments · Fixed by #578
Labels
module-browsingContext Browsing Context module needs-discussion Issues to be discussed by the working group

Comments

@lutien
Copy link
Contributor

lutien commented Oct 19, 2023

At the moment, the spec says that we should get the browsing context info with maxDepth = 0, which means that we should omit children. But since we want to send only one event in case of removing the whole tree of contexts, wouldn't it maybe be useful for the clients to have then information about removed children in the event, since there will be no separate event for the child contexts? Otherwise, I guess the clients would have to keep the tree of browsing contexts in the memory to know which child contexts should be removed (but maybe they have to do it anyway).

@whimboo whimboo added module-browsingContext Browsing Context module needs-discussion Issues to be discussed by the working group labels Oct 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
module-browsingContext Browsing Context module needs-discussion Issues to be discussed by the working group
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants