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
