Re: [compute-pressure] Clarify what supportedSources() should expose. (#266)

Looking at https://github.com/w3c/compute-pressure/issues/164#issuecomment-2056303587, https://developer.mozilla.org/en-US/docs/Web/API/XRSystem/isSessionSupported is the most similar to what we have here: it's asynchronous and needs to query the platform before sending a result. All the other methods are synchronous because the data they need comes from the engine itself AFAICS.

The approach we take depends on what we want to convey: in its current form, `supportedSources` is a hint that basically says "this implementation supports _at most_ these sources", but whether or not a source in `supportedSources` actually works depends on calling `observe()`. I don't know if this is a useful hint or not; perhaps calling `observe()` and checking for `NotSupportedError` is enough for developers, perhaps not.

If `supportedSources` is supposed to really indicate which sources are supported by the platform (meaning the results could even vary across calls), then it needs to become a method like `XRSystem.isSessionSupported()`.

With this, I throw the ball back to @kenchris and @elchi3's court :-)

-- 
GitHub Notification of comment by rakuco
Please view or discuss this issue at https://github.com/w3c/compute-pressure/issues/266#issuecomment-2076905637 using your GitHub account


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

Received on Thursday, 25 April 2024 10:53:14 UTC