D18583: [WIP] Init ShellClient only when commited to the buffer

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


  New simpler approach.
  
  We init on startup, but put a geometry blocker over everything until the shell is 
  committed.
  
  All signals/slots work without changes, client still only gets one correct configure.
  
  We might need to still re-evaluate some rules in finishInit.
  And we can maybe speed things up by adding some early returns if 
  m_initialSetupBlocker is set.
  
  Still WIP as I have some cleanup to do and 2 more unit tests to fix.

REPOSITORY
  R108 KWin

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D18583?vs=50449&id=51243

BRANCH
  master

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

AFFECTED FILES
  autotests/integration/kwin_wayland_test.h
  autotests/integration/quick_tiling_test.cpp
  autotests/integration/shell_client_test.cpp
  autotests/integration/test_helpers.cpp
  shell_client.cpp
  shell_client.h

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