Re: [deviceorientation] Add Permissions API integration, start requiring requestPermission() usage (#123)

@reillyeon I've set this one as a draft because, like with #121, it's not clear to me whether these powerful feature names should be duplicated here or if we should use the definitions from the Accelerometer, Gyroscope and Magnetometer specs (which are already exported).

This also conflicts with #121 since https://w3c.github.io/permissions/#dfn-permission-state already performs Permissions Policy checks as well (TIL) so if this PR lands first the other one can be simplified a bit.

I've had to do some in parallel-queue a global task trickery that does not look great, but felt necessary.

The way the permission state checks were added to the parts that specify when/how to fire a device motion/orientation event are not pretty either, but I couldn't find a better way to also let event listeners added before permission is granted to start firing events once that happens (as discussed in #74).

-- 
GitHub Notification of comment by rakuco
Please view or discuss this issue at https://github.com/w3c/deviceorientation/pull/123#issuecomment-1811187276 using your GitHub account


-- 
Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config

Received on Tuesday, 14 November 2023 20:21:54 UTC