D16317: [effects/dimscreen] Make it work on Wayland
Vlad Zagorodniy <[email protected]>
| Newsgroups | gmane.comp.kde.devel.kwin |
|---|---|
| Message-ID | <[email protected]> |
zzag created this revision. zzag added a reviewer: KWin. Herald added a project: KWin. Herald added a subscriber: kwin. zzag requested review of this revision. REVISION SUMMARY There is no concept of managed windows in Wayland, so every time we call managed() on a Wayland client, it will return false. We need to call that method only for X11 clients. The weirdest part of this patch is missing resource names in window classes of authentication agents on Wayland. The reason for that is KWayland can't read the exe symlink(/proc/<pid>/exe) because the authentication agents run as root. CCBUG: 398100 TEST PLAN Before: F6338545: before.png <https://phabricator.kde.org/F6338545> After: F6338546: after.png <https://phabricator.kde.org/F6338546> REPOSITORY R108 KWin BRANCH effects-dimscreen-wayland REVISION DETAIL https://phabricator.kde.org/D16317 AFFECTED FILES effects/dimscreen/dimscreen.cpp To: zzag, #kwin Cc: kwin, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart