D5747: add pid to plasma window management protocol

Martin Flöser noreply at phabricator.kde.org
Fri May 12 16:39:42 UTC 2017


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

INLINE COMMENTS

> plasmawindowmodel.cpp:84
>  
> +
>      QObject::connect(window, &PlasmaWindow::activeChanged, q,

Unrelated newline

> plasma-window-management.xml:263
> +      <description summary="process id of application owning the window has changed">
> +        This event will be sent when the process id of the application owning the window has changed.
> +        The pid will initially be 0, meaning it hasn't been set. As the compositor sets it, the client

The pid will never change.

REPOSITORY
  R127 KWayland

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

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


More information about the Kde-frameworks-devel mailing list