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
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.