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

zzag added inline comments.

INLINE COMMENTS

> quick_tiling_test.cpp:474
>      QVERIFY(configureRequestedSpy.isValid());
> +    surface->commit();
> +    QVERIFY(configureRequestedSpy.wait());

We don't want frame callback, sorry for this mistake.

> test_helpers.cpp:529
> +    QVERIFY(configureRequestedSpy.isValid());
> +    surface->commit();
> +    configureRequestedSpy.wait();

We don't want the frame callback.

REPOSITORY
  R108 KWin

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

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