D20241: [wayland] Place all toplevels before the first configure

davidedmundson updated this revision to Diff 55437.
davidedmundson added a comment.


  Urgh, that cascade issue is a bit sucky. I assumed all placement operations were deterministic.
  
  I can do a "lazy" fix by just making it not place any invalid windows (this patch).
  Arguably we could do this in all placement strategies that only move the window for speed purposes.
  
  - Don't cascade invalid windows

REPOSITORY
  R108 KWin

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D20241?vs=55388&id=55437

BRANCH
  master

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

AFFECTED FILES
  autotests/integration/maximize_test.cpp
  placement.cpp
  shell_client.cpp

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