[Wayland] Use PQ/HLG TF instead of Gamma 2.2 for HDR color spaces [chromium/src : main]
0 views
Skip to first unread message
Thomas Anderson (Gerrit)
Nov 3, 2025, 3:09:34 PM (3 days ago) Nov 3
to Kramer Ge, chromium...@chromium.org, max+watc...@igalia.com, nickdiego+wa...@igalia.com, ozone-...@chromium.org
Attention needed from Kramer Ge
Thomas Anderson voted Commit-Queue+1![]( "Open in Gerrit")
| Commit-Queue | +1 |
Related details
Attention is currently required from:
- Kramer Ge
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Kramer Ge (Gerrit)
Nov 4, 2025, 4:37:50 PM (2 days ago) Nov 4
to Thomas Anderson, Chromium LUCI CQ, chromium...@chromium.org, max+watc...@igalia.com, nickdiego+wa...@igalia.com, ozone-...@chromium.org
Attention needed from Thomas Anderson
Kramer Ge voted and added 1 comment![]( "Open in Gerrit")
Attention is currently required from:
- Thomas Anderson
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Thomas Anderson (Gerrit)
Nov 4, 2025, 5:08:21 PM (2 days ago) Nov 4
to Chromium LUCI CQ, chromium...@chromium.org, max+watc...@igalia.com, nickdiego+wa...@igalia.com, ozone-...@chromium.org
Thomas Anderson voted Commit-Queue+2![]( "Open in Gerrit")
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Chromium LUCI CQ (Gerrit)
Nov 4, 2025, 6:41:25 PM (2 days ago) Nov 4
to Thomas Anderson, Kramer Ge, chromium...@chromium.org, max+watc...@igalia.com, nickdiego+wa...@igalia.com, ozone-...@chromium.org
Chromium LUCI CQ submitted the change![]( "Open in Gerrit")
Change information
Commit message:
[Wayland] Use PQ/HLG TF instead of Gamma 2.2 for HDR color spaces
* If the compositor reports HDR headroom but suggests a TF of gamma
2.2, fallback to an HDR TF like PQ or HLG. This is required because
gfx::ColorSpace::IsHDR() will return false for gamma 2.2.
gfx::ColorSpace::GetAsHDR() cannot always be used because that sets
a custom TF, which is not supported by any compositors.
* Call wp_image_description_creator_params_v1_set_luminances for HDR
content. This is required for correct compositor tonemapping.
* Remove WaylandWpImageDescription::color_space() and
WaylandWpImageDescription::hdr_metadata(). These were never used.
* Properly report output reference luminance and headroom.
R=fangzhoug
Fixed: 446254087
Change-Id: If10548e32b01ef702b89291182e6bda77eb096ac
Reviewed-by: Kramer Ge <fang...@chromium.org>
Commit-Queue: Thomas Anderson <thomasa...@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1540360}
Files:
- M ui/ozone/platform/wayland/host/wayland_wp_color_manager.cc
- M ui/ozone/platform/wayland/host/wayland_wp_image_description.cc
- M ui/ozone/platform/wayland/host/wayland_wp_image_description.h
Change size: M
Delta: 3 files changed, 168 insertions(+), 78 deletions(-)
Branch: refs/heads/main
Submit Requirements:
Code-Review: +1 by Kramer Ge
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Reply all
Reply to author
Forward
0 new messages