search

w3c / geolocation

W3C Geolocation API
https://www.w3.org/TR/geolocation/
81 stars 56 forks source link

Permissions Policy integration incomplete #75

Closed annevk closed 3 years ago

annevk commented 3 years ago

It doesn't perform the "is allowed to use" check anywhere.

marcoscaceres commented 3 years ago

so, we had it, but apparently this is covered by the Permission Spec's "request permission to use" algorithm:

https://github.com/w3c/geolocation-api/pull/68#discussion_r599940995

cc @jyasskin, in case we need to clarify things.

annevk commented 3 years ago

Oh I see, https://www.w3.org/TR/permissions/#reading-current-states ends up looking at it. Interesting.

marcoscaceres commented 3 years ago

Yeah, let us know if that works ok. We can update Permissions or here if needed.

annevk commented 3 years ago

I think that works. There might be some cleanup to be done around settings objects, but that's out-of-scope for this issue. Thanks!