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

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


  Hopefully address all review comments
  
  I got rid of the RAII blocker, and mod the value manually
  
  testMinimised is added but uncovers an existing bug! (so that was a good suggestion)
  
  testing fullscreen configure events was merged into the newly added test about fullscreen restore

REPOSITORY
  R108 KWin

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

BRANCH
  master

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

AFFECTED FILES
  autotests/integration/effects/maximize_animation_test.cpp
  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