D7475: Make EffectsHandlerImpl::announceSupportProperty work without X11

[Thomas Lübking]

luebking added a comment.


  Pretty uninformed consideration:
  When dealing with X11/wayland hybrids, won't one end up with two property indexes which are hard -if not impossible- to align (because one will be assigned by kwin internally and the other by the casual xwayland server)?
  Might the xwayland property even change over the runtime of kwin (is the xwayland server conditionally removed)?
  
  Would it in this case not be required to abstract the property query with a mapper in general (or abort x11 properties and always ask kwin via dbus resp. the effects API for a purely internally kept property)?
  
  Random sidenote:
  registerPropertyType() has a hash trap (unregistering a non-existing atom will insert it forever)

REPOSITORY
  R108 KWin

BRANCH
  effects-announce-support-property-no-x11

REVISION DETAIL
  https://phabricator.kde.org/D7475

To: graesslin, #kwin, #plasma, davidedmundson
Cc: luebking, davidedmundson, plasma-devel, kwin, ZrenBot, progwolff, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170823/797facc3/attachment.html>