Diffusion KWin 339a08de3883

Acknowledge the server side deco mode in WaylandServer not ShellClient

Authored by graesslin on Jan 18 2018, 8:17 PM.

Description

Acknowledge the server side deco mode in WaylandServer not ShellClient

Summary:
With GTK applications our code showed issues. With GTK the sequence for
a menu or dialog window is:

  1. create surface
  2. create server side decoration
  3. request csd
  4. create xdg_shell_surface

Which result in KWin to have the ServerSideDecoration object being
created before the ShellClient is created. The connect for mode changed
happens in ShellClient, which is too late to catch that the window
requested client side decoration. Thus KWin created a decoration about
things such as a menu.

This change moves the acknowledge of the mode to WaylandServer. As we
anyway always acknowledge the requested mode we can also do it there and
don't require ShellClient to be present. Thus when the ShellClient is
created the mode is properly reflected and no decoration is created.

BUG: 389117
FIXED-IN: 5.12.0

Test Plan: Played a lot with evince

Reviewers: KWin, Plasma

Subscribers: plasma-devel, kwin

Tags: Plasma

Differential Revision: https://phabricator.kde.org/D9967