D7955: Move X11-only specific part of EffectsHandlerImpl to x11 platform plugin

Martin Flöser noreply at phabricator.kde.org
Sat Sep 23 18:25:16 UTC 2017


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

REVISION SUMMARY
  The Platform API is extended by a call to create the EffectsHandler. In
  X11 standalone Platform a new EffectsHandlerImplX11 is added which
  contains the X11 only parts of the EffectsHandler, such as grabbing the
  X keyboard and the X11 mouse interception window.
  
  The EffectsHandlerImpl gains some virtual methods for the parts which
  are now done in the X11 specific implementation. In return we get rid of
  lots of if-else structures checking for the operation mode.

TEST PLAN
  Only compile tested.

REPOSITORY
  R108 KWin

BRANCH
  x11-effects-handler

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

AFFECTED FILES
  CMakeLists.txt
  composite.cpp
  effects.cpp
  effects.h
  effects_mouse_interception_x11_filter.cpp
  effects_mouse_interception_x11_filter.h
  platform.cpp
  platform.h
  plugins/platforms/wayland/wayland_backend.cpp
  plugins/platforms/x11/standalone/CMakeLists.txt
  plugins/platforms/x11/standalone/effects_mouse_interception_x11_filter.cpp
  plugins/platforms/x11/standalone/effects_mouse_interception_x11_filter.h
  plugins/platforms/x11/standalone/effects_x11.cpp
  plugins/platforms/x11/standalone/effects_x11.h
  plugins/platforms/x11/standalone/x11_platform.cpp
  plugins/platforms/x11/standalone/x11_platform.h
  utils.h

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


More information about the Plasma-devel mailing list