-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/107983/
-----------------------------------------------------------
(Updated Jan. 5, 2013, 2:38 p.m.)
Review request for kdelibs, kwin, Plasma, Aaron J. Seigo, Marco Martin, Martin
Gräßlin, and Fredrik Höglund.
Changes
-------
Attaching Fredrik, since git log say: the signal is rather new.
@Fredrik - what was the intention behind comparing with winId instead of the
app root (since that is where the event is sent by kselectionowner)
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
Thanks,
Thomas Lübking
