D7942: Create a dedicated X11Filter for Client sync events and move it to X11 platform

Martin Flöser noreply at phabricator.kde.org
Fri Sep 22 18:39:43 UTC 2017


graesslin created this revision.
graesslin added reviewers: KWin, Plasma.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  On Wayland we have the sync disabled as it doesn't work properly. This
  allows us to also move the sync event handling into the X11 standalone
  platform.
  
  The code is slightly refactored: instead of passing the event to each
  Client, we search for the matching Client. For that the SyncAlaram struct
  is added to public section of Client. The method to handle the sync
  doesn't need the event any more and is moved from events.cpp to
  client.cpp.

TEST PLAN
  Run Xephyr+kwin_x11, resized a window and verified through
  gdb breakpoint that the sync still works

BRANCH
  sync-filter

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

AFFECTED FILES
  client.cpp
  client.h
  events.cpp
  plugins/platforms/x11/standalone/CMakeLists.txt
  plugins/platforms/x11/standalone/sync_filter.cpp
  plugins/platforms/x11/standalone/sync_filter.h
  plugins/platforms/x11/standalone/x11_platform.cpp
  plugins/platforms/x11/standalone/x11_platform.h

To: graesslin, #kwin, #plasma
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170922/5f965860/attachment.html>


More information about the Plasma-devel mailing list