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
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.