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

Authored by graesslin on Sep 23 2017, 6:25 PM.

Details

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.

Diff Detail

Repository
R108 KWin
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
graesslin created this revision.Sep 23 2017, 6:25 PM
Restricted Application added a project: KWin. · View Herald TranscriptSep 23 2017, 6:25 PM
Restricted Application added subscribers: kwin, plasma-devel. · View Herald Transcript
This revision was automatically updated to reflect the committed changes.