D15020: [xwl] Move Xwayland parts into separate class

davidedmundson added inline comments.

INLINE COMMENTS

> xwayland.cpp:236-237
> +
> +    m_app->startSession();
> +    m_app->createWorkspace();
> +

This isn't really within the scope of an XWayland class as it's handling startup, stuff app should be doing.

More correct to emit something and have it handled with the same   ApplicationWayland::continueStartupWithSceen

> xwayland.cpp:241
> +
> +    m_app->notifyKSplash();
> +}

This isn't needed on wayland, ksplash is changed to ignore "wm" startup signals

REPOSITORY
  R108 KWin

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

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