[compiz] Bug with kde applications and damageWindowRect wrapping...

[Danny Baumann]

Hi,

> > > If someone can confirm this bug ...
> > > Run this in an xterm with smart placement method: 
> > > while true; do konsole & sleep 1; killall konsole; done
> > > 
> > > Look at the screen, you should randomly see the bug at konsole window
> > > mapping...
> > 
> > The placement plugin is broken. It shouldn't place windows when they
> > first appear but instead when we get the map request and before they are
> > mapped.
> > 
> > This bug is only visible in kde apps as kde doesn't support the sync
> > request protocol.
> 
> So you mean something like the attached patch?
> Let me know if it's correct and it works for you - it does for me ;-)

Actually, I missed one thing so the attached patch should work better.
The decoration plugin updates w->input in its handleEvent after calling
core's handleEvent which does the actual window mapping. Consequently,
place should also place the window after calling core and must
additionally be loaded after decoration to work properly.
However, I wonder if there is a special reason for decoration.c checking
the MapRequest after core. If not, we could place the window before the
actual mapping. David? ;-)

Regards,

Danny
-------------- next part --------------
A non-text attachment was scrubbed...
Name: place-fix.diff
Type: text/x-patch
Size: 3272 bytes
Desc: not available
Url : http://lists.freedesktop.org/archives/compiz/attachments/20070418/28d1edb7/place-fix.bin