D23545: Lift output enablement into Platform
Roman Gilg <[email protected]>
| Newsgroups | gmane.comp.kde.devel.kwin |
|---|---|
| Message-ID | <[email protected]> |
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