Bug 204968 - wnck-applet gets woken up for every keypress when workspace-switcher is running
Summary: wnck-applet gets woken up for every keypress when workspace-switcher is running
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: Fedora
Classification: Fedora
Component: metacity
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Søren Sandmann Pedersen
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: FC7Target wakeup
TreeView+ depends on / blocked
 
Reported: 2006-09-01 19:27 UTC by Peter Jones
Modified: 2014-06-18 09:08 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2007-04-02 06:03:47 UTC


Attachments (Terms of Use)

Description Peter Jones 2006-09-01 19:27:00 UTC
Currently wnck-applet gets woken up for every keypress when workspace-switcher
is running.  This means we're taking 6 context switches per keypress, and (at
least) 3 tasks are being woken up.  That's bad.

Comment 1 Matthias Clasen 2006-09-03 01:49:42 UTC
The problem here is that 

a) libwnck listens for property changes on all toplevel windows, to catch
   icon changes, name changes, etc

b) GTK+ updates _NET_WM_USER_TIME property for every user interaction for
   focus stealing stuff

Comment 2 Matthias Clasen 2006-09-03 02:02:43 UTC
about the only fix I can see would be to add an indirection to the user time
property, by not storing it on the toplevel, but on a separate, dedicated window,
and only put the id of that window in a property on the toplevel. 

Comment 3 Matthias Clasen 2006-09-03 02:03:37 UTC
This also affects metacity, btw

Comment 4 Matthias Clasen 2006-09-03 03:33:04 UTC
...and Owen pointed this problem out back in May 2003:

http://mail.gnome.org/archives/wm-spec-list/2003-May/msg00047.html

Comment 5 Matthias Clasen 2006-09-04 03:19:30 UTC
I proposed an EWMH change to fix this issue:

http://mail.gnome.org/archives/wm-spec-list/2006-September/msg00000.html

Comment 6 Søren Sandmann Pedersen 2007-03-23 17:03:21 UTC
To fix this we need to add a patch to metacity to support the new spec. This
will cause gtk+ to put the updates on a separate window, which libwnck then
won't listen to.

Moving to metacity and putting on f7 target.


Comment 7 Matthias Clasen 2007-03-27 16:54:44 UTC
The EWMH addition is in the spec now, the metacity/gtk+ patches are being
developed here: http://bugzilla.gnome.org/show_bug.cgi?id=354213

Comment 8 Matthias Clasen 2007-04-02 06:03:47 UTC
No need to keep this bug open; the gtk and metacity changes have been committed
upstream and will find its way into F8 with gnome 2.20


Note You need to log in before you can comment on or make changes to this bug.