D23545: Lift output enablement into Platform
Roman Gilg
noreply at phabricator.kde.org
Wed Aug 28 21:12:22 BST 2019
romangg created this revision.
romangg added a reviewer: KWin.
Herald added a project: KWin.
Herald added a subscriber: kwin.
romangg requested review of this revision.
REVISION SUMMARY
This lifts the enablement code for outputs from the DRM backend to Platform
allowing other Wayland backends in the future to use this interface as well.
To do that we also create some helper functions on Platform level and have to
spill some KWayland classes into AbstractOutput what motivates a further split
of Platform into a Wayland child class like for AbstractOutput.
This is just a port. There are some inconsistencies with the current logic and
in contrast to before the image of a disabled output is not cleared. I assume
this is not an error in the porting but was before rather accidentally since
there was no actual modeset requested after disabling.
TEST PLAN
Disabled and enabled an output in DRM session.
REPOSITORY
R108 KWin
BRANCH
platformOutputEnablement
REVISION DETAIL
https://phabricator.kde.org/D23545
AFFECTED FILES
abstract_output.cpp
abstract_output.h
abstract_wayland_output.cpp
abstract_wayland_output.h
platform.cpp
platform.h
plugins/platforms/drm/drm_backend.cpp
plugins/platforms/drm/drm_backend.h
plugins/platforms/drm/drm_output.h
wayland_server.cpp
To: romangg, #kwin
Cc: kwin, LeGast00n, The-Feren-OS-Dev, sbergeron, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwin/attachments/20190828/92011aee/attachment.html>
More information about the kwin
mailing list