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

[Feature Policy] Confirm whether WebXR features can/should reuse existing feature names #765

Closed
ddorwin opened this issue Jul 15, 2019 · 1 comment
Labels
feature policy All things related to Feature Policy fixed by pending PR A PR that is in review will resolve this issue.

Comments

@ddorwin
Copy link
Contributor

ddorwin commented Jul 15, 2019

I, at least, have been assuming that XR features should reuse existing policy-controlled feature names. For example, "camera" for frame-aligned camera frame access in AR and maybe "ambient-light-sensor" for lighting estimation. We should confirm whether this is the case and maybe document some guidelines because it might affect the Feature Policy solution for WebXR and will definitely affect future XR features.

On the one hand:

  • The existing feature names read more like capabilities than specification names or APIs (i.e., methods).
  • It seems undesirable to duplicate features.
  • Reusing feature names makes it easier for polyfills to also work.

On the other hand:

  • The specification is called Feature Policy, not capability policy.
  • The specification says, "A policy-controlled feature is an API or behaviour which can be enabled or disabled in a document by referring to it in a feature policy" (emphasis added).
  • Each feature name is defined by a specification, and the Policy Controlled Features doc is just a non-normative collection of those.
    • Note that the default allowlist for each such feature is defined in the individual specifications as well.
  • What would it mean for another specification to come along and redefine the meaning of a string from another specification (even if it is in a consistent way)?

/cc @clelland

@probot-label
Copy link

probot-label bot commented Sep 21, 2019

This issue is fixed by PR #842

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature policy All things related to Feature Policy fixed by pending PR A PR that is in review will resolve this issue.
Projects
None yet
Development

No branches or pull requests

4 participants