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