D26883: Turn AbstractClient::move() into a convenience method

zzag created this revision.
zzag added a reviewer: KWin.
Herald added a project: KWin.
Herald added a subscriber: kwin.
zzag requested review of this revision.

REVISION SUMMARY
  Maintaining two different ways of updating geometry can be challenging.
  For what it's worth, prior to 5.18, move() didn't move the decoration
  input window because someone forgot to update X11Client::doMove().
  
  This change turns move() into a convenience method to simplify geometry
  handling. It adds a little overhead, but on the other hand move() is
  not used frequently.

REPOSITORY
  R108 KWin

BRANCH
  move-wrap-set-frame-geometry

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

AFFECTED FILES
  abstract_client.cpp
  abstract_client.h
  internal_client.cpp
  internal_client.h
  x11client.cpp
  x11client.h
  xdgshellclient.cpp
  xdgshellclient.h

To: zzag, #kwin
Cc: kwin, Orage, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, ahiemstra, mart