D5756: Set pid on the ClientConnection backing the PlasmaWindow surface.

[Martin Flöser]

graesslin requested changes to this revision.
graesslin added inline comments.
This revision now requires changes to proceed.

INLINE COMMENTS

> abstract_client.cpp:708
>      updateAppId();
> -    w->setSkipTaskbar(skipTaskbar());
> +    w->setPid(surface()->client()->processId());
> +    surface()->client()->processId();

Why is setSkipTaskbar removed?

> abstract_client.cpp:709
> +    w->setPid(surface()->client()->processId());
> +    surface()->client()->processId();
>      w->setShadeable(isShadeable());

What is this line supposed to do?

REPOSITORY
  R108 KWin

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

To: hein, #plasma, sebas, graesslin
Cc: kwin, plasma-devel, #kwin, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170508/9c19a713/attachment-0001.html>