Skip to content

fix: make userContext mandatory in browsingContext.CreateResult#1122

Closed
juliandescottes wants to merge 1 commit into
w3c:mainfrom
juliandescottes:browsingContext-create-return-userContext
Closed

fix: make userContext mandatory in browsingContext.CreateResult#1122
juliandescottes wants to merge 1 commit into
w3c:mainfrom
juliandescottes:browsingContext-create-return-userContext

Conversation

@juliandescottes
Copy link
Copy Markdown
Contributor

@juliandescottes juliandescottes commented May 19, 2026

The algorithm for browsingContext.create reads as follows:

Let body be a map matching the browsingContext.CreateResult production, with the context field set to traversable’s navigable id and the userContext property set to the user context id of traversable’s associated user context.

Nothing here can lead userContext to be missing so I don't see why the property should be optional?


Preview | Diff

@juliandescottes juliandescottes requested review from OrKoN and whimboo May 19, 2026 12:45
@juliandescottes
Copy link
Copy Markdown
Contributor Author

juliandescottes commented May 19, 2026

Ok I see that this seems to be another follow up of #1046 which not only added userContext to events, but also to the return value of browsingContext.create.

I will close in favor of #1071, I guess we can make it mandatory at the same time as the rest (in the future we should add a note in the spec when we make fields temporarily optional, that would save time ...)

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.

1 participant