Repository navigation
Rename PermissionsPolicy interface back to FeaturePolicy - #582
marcoscaceres wants to merge 2 commits into
Conversation
There was a problem hiding this comment.
Pull request overview
Updates the Permissions Policy specโs script introspection API surface to use the legacy FeaturePolicy naming (matching current web-compat usage like document.featurePolicy) instead of PermissionsPolicy.
Changes:
- Rename the script-facing policy object from
PermissionsPolicytoFeaturePolicyin prose and examples. - Update WebIDL to expose
Document.featurePolicy/HTMLIFrameElement.featurePolicyreturningFeaturePolicy.
๐ก Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Since this API does not have agreement and only one implementation I think it would be better to remove it. |
a4047f3 to
4593ff6
Compare
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
Might be ok to remove it, but we would need to transplant it to the Web Compat spec. |
|
Why? There's only one implementation and nobody else wants to implement it. |
|
Ok, sure (and I agree), but we have the "37% of sites tested are using" it Chrome situation... this is a transitional path. This at least matches Chrome and we can then hopefully get agreement at TPAC to remove it and document the path to using It's the least bad option right now, IMO. As a compromise, should we maybe add a note/issue that says that this is a Feature at Risk? |
|
I don't see why that is a reason to keep it in the specification? Also page loads doesn't translate to websites. |
|
@miketaylr suggested the usage might go away as privacy sandbox APIs are retired, agree we should discuss at TPAC |
Chrome shows that ~37% of sites tested are using
document.featurePolicy:https://chromestatus.com/metrics/feature/timeline/popularity/2511
We should consider switching this back.
Related web platform tests:
web-platform-tests/wpt#58463
Preview | Diff