-
Notifications
You must be signed in to change notification settings - Fork 329
Compat: remove GPUAdapter.featureLevel #5037
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
Conversation
Issue: gpuweb#4266, gpuweb#4987 (Partially subsumes old PR gpuweb#4898)
**DRAFT!!** Depends on gpuweb#5036 Issue: gpuweb#4266, gpuweb#4987 (With gpuweb#5036, subsumes old PR gpuweb#4898)
One use of |
IIRC the CTS assumes if you pass in the "compatibility" flag to the CTS, then it requests compat adapters and expects it got compat adapters. It doesn't check the adapter itself. It just assumes via I'm not sure that can really change. The CTS needs to test devices that are actually running in compatibility mode. |
ps: if that wasn't clear. I meant to say that |
And, I'm wrong. I guess I don't remember this part of the code 😅 |
Np, but maybe we can change it to use that strategy instead. Depends on exactly how we use that helper function. |
For the record, linking related point: #5012 (comment)
|
I'm going to flatten this into #5036 while updating it, since the WG discussed all together. |
DRAFT!! - Depends on #5036, skip first two commits, see third: 665cbb8
This is a proposal.
Issue: #4266, #4987, #5012 (comment)
(Together with #5036, subsumes old PR #4898)