--===============3266989430739573227== Content-Type: multipart/alternative; boundary="===============4694687303040328182==" --===============4694687303040328182== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107983/ ----------------------------------------------------------- (Updated May 27, 2013, 6:14 p.m.) Status ------ This change has been marked as submitted. Review request for kdelibs, kwin, Plasma, Aaron J. Seigo, Fredrik Höglund, Martin Gräßlin, and Marco Martin. Description ------- It works fine here (tested so far KWindowSystem signal, KSelectionWatcher only with kwin) with kwin (shift+alt+f12), xcompmgr, compiz & "metacity -c" and e17. Didn't try xfce nor mutter. Technically: I do not at all understand why KWindowSystem is *not* watching the root window - KSelectionOwner for one is sending events to the root and this also seems the case for all other WMs (at least everything now starts to cause the signal to be emitted) The KSelectionWatcher failure seems to be kwin specific (wrote me a cleaner testcase), there'll be some X11 event processing on top that eats away the client messages. So this one can be scratched from the patch, the KWindowSystem issue remains. This addresses bug 179042. http://bugs.kde.org/show_bug.cgi?id=179042 Diffs ----- kdeui/windowmanagement/kwindowsystem_x11.cpp f9b3cc1 Diff: http://git.reviewboard.kde.org/r/107983/diff/ Testing ------- see summary File Attachments ---------------- testcase http://git.reviewboard.kde.org/media/uploaded/files/2013/01/04/selectionwatcher.cpp Thanks, Thomas Lübking --===============4694687303040328182== Content-Type: text/html; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit
This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107983/

This change has been marked as submitted.

Review request for kdelibs, kwin, Plasma, Aaron J. Seigo, Fredrik Höglund, Martin Gräßlin, and Marco Martin.
By Thomas Lübking.

Updated May 27, 2013, 6:14 p.m.

Description 

It works fine here (tested so far KWindowSystem signal, KSelectionWatcher only with kwin) with kwin (shift+alt+f12), xcompmgr, compiz & "metacity -c" and e17.
Didn't try xfce nor mutter.

Technically:
I do not at all understand why KWindowSystem is *not* watching the root window - KSelectionOwner for one is sending events to the root and this also seems the case for all other WMs (at least everything now starts to cause the signal to be emitted)

The KSelectionWatcher failure seems to be kwin specific (wrote me a cleaner testcase), there'll be some X11 event processing on top that eats away the client messages.
So this one can be scratched from the patch, the KWindowSystem issue remains.

Testing 

see summary
Bugs: 179042

Diffs

  • kdeui/windowmanagement/kwindowsystem_x11.cpp (f9b3cc1)

View Diff

File Attachments

--===============4694687303040328182==-- --===============3266989430739573227== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel --===============3266989430739573227==--