D16703: [wayland] add explict AbstractClient::setDesktops(QList)

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

REVISION SUMMARY
  Currently setDesktop and unsetDesktop were out of sync, with the latter
  missing several important signals and updating of transients.
  
  By using a a shared implementation we avoid that, it also allows for an
  atomic move of a window between desktops.
  
  setDesktop is changed back to be a moval of desktop as it currently
  broke several unit tests as well as changing the behaviour of the move
  to desktop shortcut on wayland.

TEST PLAN
  testBindings now passes
  Moved windows with the context menu on X11

REPOSITORY
  R108 KWin

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

AFFECTED FILES
  abstract_client.cpp
  abstract_client.h

To: davidedmundson, #kwin
Cc: kwin, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, ngraham, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart