1924328 - WebGPU: Support dual-source blending (`dual-source-blending` device feature)

Status: ASSIGNED

(Core :: Graphics: WebGPU, enhancement, P3)

Description

[Erich Gubler [:ErichDonGubler] (he/him)]

Upstream WGPU issue: wgpu#6402

Comment 1

[Erich Gubler [:ErichDonGubler] (he/him)]

Attachment: Bug 1924328 - feat(webgpu): recognize unimplemented `dual-source-blending` feature r=#webgpu-reviewers!

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/5fec1334c4d2
feat(webgpu): recognize unimplemented `dual-source-blending` feature r=webgpu-reviewers,webidl,teoxoy,saschanaz

Comment 3

[Serban Stanca [:SerbanS]]

https://hg.mozilla.org/mozilla-central/rev/5fec1334c4d2

Comment 4

[Erich Gubler [:ErichDonGubler] (he/him)]

I'm going to leave the full resolution of this bug for later. With bug 1933324, users will be directed to this bug to holler if they're blocked.

Comment 5

[Erich Gubler [:ErichDonGubler] (he/him)]

Attachment: WIP: Bug 1924328 - feat(webgpu): recognize `GPUBlendFactor`s that require `dual-source-blending` feature r=#webgpu-reviewers!

Coverage for blend factors is mostly at webgpu:api,operation,rendering,color_target_state:blending,*, but these tests skip testing these factors if the dual-source-blending feature isn't supported. This change is mostly intended to improve diagnostics while waiting for dual-source-blending is implemented.

Comment 6

[Erich Gubler [:ErichDonGubler] (he/him)]

Attachment: WIP: Bug 1924328 - chore(webgpu): document reliance on `enum` discriminants matching for blend factors r=#webgpu-reviewers

Comment 7

[Erich Gubler [:ErichDonGubler] (he/him)]

Attachment: WIP: Bug 1924328 - feat(webgpu): implement `dual-source-blending` r=#webgpu-reviewers!

Comment 8

[Erich Gubler [:ErichDonGubler] (he/him)]

I think the implementation should be trivial, now that upstream WGPU has an implementation that supposedly worked. Fired off a Try push with some easy changes at try:c6309e98d743. 🤞🏻