D22225: Partly remove Compositor restart functionality

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 removes the dbus method to reinit the compositor and the restart
  of the Compositor class.
  
  The dbus method seems to be of little use and also did not work when
  tried last on master.
  
  Instead of the restart function reinitialize can be called. Reading again
  the settings in this case is fine, since it is done rarely. This reduces
  the code complexity.

TEST PLAN
  Manually on Wayland and X. 100% autotests pass.

REPOSITORY
  R108 KWin

BRANCH
  rmReinitRestart

REVISION DETAIL
  https://phabricator.kde.org/D22225

AFFECTED FILES
  autotests/integration/dont_crash_reinitialize_compositor.cpp
  autotests/integration/generic_scene_opengl_test.cpp
  autotests/integration/scene_qpainter_test.cpp
  composite.cpp
  composite.h
  dbusinterface.cpp

To: romangg, #kwin
Cc: kwin, LeGast00n, sbergeron, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.