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

Set Permission vagueness #386

Open
annevk opened this issue Oct 20, 2022 · 1 comment
Open

Set Permission vagueness #386

annevk opened this issue Oct 20, 2022 · 1 comment
Assignees

Comments

@annevk
Copy link
Member

annevk commented Oct 20, 2022

It's not at all apparent what this means:

Interpret parameters.state as if it were the result of an invocation ...

Presumably this is meant to end up changing the permission, but how?

cc @johannhof

@jyasskin
Copy link
Member

Yes, by interacting with https://www.w3.org/TR/permissions/#dfn-permission-state step 5:

If there was a previous invocation of this algorithm with the same descriptor and settings, returning previousResult, and the user agent has not received new information about the user's intent since that invocation, return previousResult.

If you're migrating all this to a permission store anyway, you can do it the more straightforward way.

@johannhof johannhof self-assigned this Oct 20, 2022
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

No branches or pull requests

3 participants