-
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
swapchain & presentation rework #203
Conversation
9e34065
to
7f211e8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM though I'd like @kainino0x to look at it as well.
It's going to be a pain to update Dawn ^^
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM but I did just think of one more thing to bikeshed
7f211e8
to
1717a23
Compare
30fb8eb
to
2249667
Compare
2249667
to
a76cd0f
Compare
Going to go ahead and land this; whoever ends up doing the next Dawn->webgpu-headers update (probably @lokokung) you'll probably want to put this into our generator (tagged "upstream") to preserve these. It'll move them around, but that's fine. |
The TextureUsage is a Bitmask, update the usage in SurfaceConfiguration to use the Flags typedef to be consistent with out uses of TextureUsage. Follow-up to webgpu-native#203
The TextureUsage is a Bitmask, update the usage in SurfaceConfiguration to use the Flags typedef to be consistent with out uses of TextureUsage. Follow-up to #203
This CL syncs dawn.json to the changes added in webgpu-native/webgpu-headers#203 and other minor differences to upstream webgpu.h. Change-Id: I214a74325e405a3165dbf92b2f455b78377b81b3 Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/147581 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Dan Sinclair <dsinclair@chromium.org> Reviewed-by: Kai Ninomiya <kainino@chromium.org>
closes: #197