D18157: Remove Client fullscreen hack
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 hack seems to be deactivated by default nowadays and I did not find an option in the KCMs to activate it. Git blame shows some last commits from around 2012 to the functions, but they only deal with code style. Also even if it would be enabled, in light of Wayland we don't want some roque XWayland client go bonkers via this hack. This patch removes the code path in order to decrease complexity and ease future changes to the fullscreen handling overall. TEST PLAN Manually and autotests pass. REPOSITORY R108 KWin BRANCH fullscreenHackRemoval REVISION DETAIL https://phabricator.kde.org/D18157 AFFECTED FILES client.h geometry.cpp manage.cpp options.cpp options.h To: romangg, #kwin Cc: kwin, kvanton, jraleigh, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, ngraham, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart