D16704: [wayland] add enter/leave virtual desktop API

davidedmundson updated this revision to Diff 45183.
davidedmundson edited the summary of this revision.
davidedmundson edited the test plan for this revision.
davidedmundson removed subscribers: zzag, anthonyfieroni.
davidedmundson added a comment.


    As setDesktop was changed to "move" this left unSetDesktop non-symetric.
    
    This replaces it with explicit API to enter/leave.
    
    This also moves new API to the new object based API rather than still
    using ints.
    
    Where numbers are used it has been tidied up so that desktop IDs are
    uint, which should be used when we have a list of desktops.
    int is used only when we have either a desktop ID or NET::OnAllDesktops
    (-1)
    
    Effects API cleared up to use this and use a set of x11 IDs, which
    avoids any potential complications of handling add and removes in different orders
    and removes any ambiguity with what happens if you leave NET_ALL

REPOSITORY
  R108 KWin

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D16704?vs=44961&id=45183

BRANCH
  desktops

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

AFFECTED FILES
  abstract_client.cpp
  abstract_client.h
  autotests/integration/virtual_desktop_test.cpp
  autotests/mock_effectshandler.h
  effects.cpp
  effects.h
  effects/desktopgrid/desktopgrid.cpp
  libkwineffects/kwineffects.cpp
  libkwineffects/kwineffects.h
  useractions.cpp

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