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