Bug 272702
| Summary: | [WebGPU] Add support for GPUCanvasToneMapping | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Mike Wyrzykowski <mwyrzykowski> |
| Component: | WebGPU | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED DUPLICATE | ||
| Severity: | Normal | CC: | adixon, gman, mark, possum.bracing0v, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Mike Wyrzykowski
[WebGPU] Add support for GPUCanvasToneMapping from https://github.com/gpuweb/gpuweb/pull/4500/
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/126507138>
Alex Dixon
This is something we're very interested in for Lightroom and Photoshop Web at Adobe to enable HDR editing in Safari.
mdrejhon
We need this for TestUFO 2.0 for both WCG and HDR support, too.
There's a version of www.testufo.com being launched that will add WCG and HDR support for Chrome/Edge, and I'd like to support HDR (not just WCG) in Safari.
Even if a permission request is needed to get access to the brighter-than-white, if Apple is concerned about UX or power considerations.
Mike Wyrzykowski
Please note this tracks adding extended range support only to WebGPU. It is not inclusive of HDR canvas support, WebGL, or similar web features. It will allow you to read back HDR values from a rgba16float canvas but it will still display as SDR in the HTMLCanvas or OffscreenCanvas itself.
Gregg Tavares
You probably already know this, but just in case, configuring the canvas with `{ toneMapping: 'extended'}` and then calling `canvas.getConfiguration()` returns `{ toneMapping: 'extended' }` on MacOS Release 226 (WebKit 20623.1.3.19.1) but the canvas is not actually being displayed with extended tone mapping. It is on Safari on iOS with WebGPU and WebGPU HDR turned on
Ideally MacOS would show the canvas in HDR. If it is not it should instead report `{}` even though the user requested `{ toneMapping: 'extended' }`
Mike Wyrzykowski
This was completed with 279816.
The issue is not known to us Gregg, thanks for reporting it.
*** This bug has been marked as a duplicate of bug 279816 ***