Open Bug 1973380 Opened 1 year ago Updated 1 year ago

Content area shadow shows white shade behind the toolbar due to content shadow, if you use mica wih HDR on (Windows Explorer also affected)

Categories

(Core :: Graphics, defect, P3)

defect

Tracking

()

Tracking Status
firefox-esr128 --- unaffected
firefox-esr140 --- unaffected
firefox140 --- unaffected
firefox141 --- wontfix
firefox142 --- fix-optional

People

(Reporter: emilio, Unassigned)

References

(Blocks 1 open bug, Regression)

Details

(Keywords: regression)

Attachments

(6 files)

No description provided.

So at first I thought this was a styling issue, but if that was the case then a lot of themes with transparent toolbar colors would be affected. Looking further into it, this only reproduces on one of my machines (which uses an NVidia GPU), and the white probably comes from https://searchfox.org/mozilla-central/rev/10ecded0ef6e96a933b08d1cd946ab3bbcfe7a15/gfx/webrender_bindings/RenderCompositorANGLE.cpp#250 or so...

Ironically I can't even take an screenshot with the builtin tool because doing so "fixes" the rendering...

Attached image image.png

Ok the Game bar let me take a screenshot that shows the issue...

So I tracked it down to the HDR settings. If I turn HDR off then the rendering is the expected one. I also tried to change those backgrounds and I don't see any improvement...

Furthermore, I also see something very similar if not the same in some menus outside of Firefox, like "Open Start menu, search for "game bar", right-click on the result"...

Ashley, Sotaro, does this ring a bell? It feels like we're blending against white or something to me, but could be anything really...

Assignee: emilio → nobody
Blocks: windows-mica
Component: Theme → Graphics
Flags: needinfo?(sotaro.ikeda.g)
Flags: needinfo?(ahale)
Product: Firefox → Core
Summary: Content area shadow shows up behind the toolbar if you use mica → Content area shadow shows wrong color (white shade) behind the toolbar if you use mica in some HDR situations (nvidia?)

:emilio, the problem also happens with the following STR?

  • Set pref gfx.webrender.compositor = false
  • Restart Firefox.

OR

  • Set pref gfx.webrender.layer-compositor = true
  • Restart Firefox.
Flags: needinfo?(sotaro.ikeda.g) → needinfo?(emilio)
Attached file about:support info

Yeah both those tweaks still show the issue. It feels like if the blending Windows is doing with the native backdrop is wrong (thinking our content is hdr instead of sdr or vice versa...)

Flags: needinfo?(emilio)
Attached image HDR on

It seems Windows Explorer is also affected... So maybe not our bug? But the white shade looks very bad, maybe we can disable it when HDR mode is on at least...

Attachment #9496248 - Attachment description: Windows explorer HRD off → Windows explorer HDR off

I sent a feedback hub (here) since at the very least, if this is an app problem, this also affects Explorer :)

Set release status flags based on info from the regressing bug 1973001

I think this is probable either:

  • A windows bug (blending our SDR content with an HDR backdrop).
  • A graphics bug (not tagging our swapchain as SDR somehow?).
Summary: Content area shadow shows wrong color (white shade) behind the toolbar if you use mica in some HDR situations (nvidia?) → Content area shadow shows white shade behind the toolbar due to content shadow, if you use mica wih HDR on (Windows Explorer also affected)

My hunch from a first look is that I think this is probably related to linear blending of compositor swapchains in DWM, when HDR or ACM mode is on in Windows the entire desktop is rendered in scRGB which is a linear color space, so blending results between stacked transparent windows differ, but I'm not familiar enough to know why it affects this part of the window that way, perhaps Glenn has an idea?

Flags: needinfo?(ahale) → needinfo?(gwatson)

I don't think we explicitly tag our swapchains as SDR, but I assume that would be the default when creating a RGBA8 surface. I'm not very familiar with the HDR functionality in DC.

Flags: needinfo?(gwatson)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: