https://bugs.kde.org/show_bug.cgi?id=495375

Michael <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |regression, wayland-only

--- Comment #6 from Michael <[email protected]> ---
I spent at least 20 min trying to search https://invent.kde.org/plasma/kwin for
a function that would ** return a list of all layers / surfaces ** (to then
test)  as I think the source of this bug might be:

https://invent.kde.org/plasma/kwin/-/blob/master/src/idle_inhibition.cpp#L78

if (visible && client->surface() && client->surface()->inhibitsIdle()) {

The test shouldn't only scan the top most layer (which is what surface()
grabs), but instead

** (more ideally) the inhibits idle attribute should be stored / tested on the
entire window object, irrespective of what layer, **

OR in this test, it should scan every layer.  Between the two options I'd
rather a single window-flag bit somewhere be used rather than walking every
layer every time.

+

Arguably this type of 'inhibits idle in a layer beneath a transparent (fully
occluding if not transparent) surface' issue is also a presently missing test
case:
https://invent.kde.org/plasma/kwin/-/blob/master/autotests/integration/idle_inhibition_test.cpp

I've also added regresion and wayland-only as just a day ago when I was using
x11 and the same Arch Linux software stack on my old PC I didn't have this
issue during video playback.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to