-
Notifications
You must be signed in to change notification settings - Fork 45
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
Remove GetCurrentTextureStatus_OutOfMemory #465
Remove GetCurrentTextureStatus_OutOfMemory #465
Conversation
This doesn't seem super likely to be handleable distinctly from Lost, but if it is the implementations (wgpu-native) can expose it as an extension. It doesn't exist in JS or Dawn. Issue: 401
I can't find any places where wgpu could return Tried looking for historical references but it's difficult because there have been so many different repositories over time. |
Loko and I discussed this a bit more, I think it would be pretty annoying to have a string that needs FreeMembers called on it but only in the error case which would make it really difficult to notice if you had a leak. It does seem like the "Error" case ( |
Oh I forgot to note, I realized this is not directly possible. There would need to be a way for the application to signal that it wants to know about OutOfMemory; an implementation can't just return it without being asked. |
Dec 12 meeting:
|
This doesn't seem super likely to be handleable distinctly from Lost (surface loss) or losing the device, but if it is, the implementation (wgpu-native) can expose it as an extension. It isn't used in JS or Dawn.
@lokokung to confirm because I forgot to take notes
Issue: #401
(dawn bug https://crbug.com/367401498)