Bug 226391 - Hiding battery monitor embedded in SystemTray causes 100% CPU
Summary: Hiding battery monitor embedded in SystemTray causes 100% CPU
Status: RESOLVED FIXED
Alias: None
Product: plasma4
Classification: Unmaintained
Component: general (show other bugs)
Version: unspecified
Platform: openSUSE Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
: 219793 227591 (view as bug list)
Depends on:
Blocks:
 
Reported: 2010-02-11 21:30 UTC by Paul
Modified: 2010-02-26 13:58 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Paul 2010-02-11 21:30:09 UTC
Version:            (using KDE 4.4.0)
OS:                Linux
Installed from:    openSUSE RPMs

Steps to reproduce:

- System Tray Settings -> Plasma Widgets: check 'Battery Monitor'.
(fyi, this tab is greyed out for me unless the host panel is set to 'unlock widgets', which is itself a bug I believe)

- System Tray Settings -> Auto Hide:  set 'battery' to hidden

at this point plasma-desktop and xorg take up constant 100% CPU.
returns to 0% after unhiding battery monitor.
I think this is causing the same underlying behavior as in bug 207177 .

Hiding other icons does not cause this behavior, including other embedded plasma widgets, e.g. weather.

Environment:  KDE SC 4.4.0 / opensuse 11.2 / kernel 2.6.31.12
Comment 1 Marco Martin 2010-02-11 22:11:37 UTC
SVN commit 1088883 by mart:

don't relayout when the task is user hidden and attempts being shown
not sure if it fixes
CCBUG:226391


 M  +3 -0      taskarea.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1088883
Comment 2 Aaron J. Seigo 2010-02-11 23:19:31 UTC
this doesn't resolve it for me; will look further.
Comment 3 Aaron J. Seigo 2010-02-11 23:52:19 UTC
SVN commit 1088928 by aseigo:

we should only muck with the minimum sizes of the popup applet when iconified if it is actually using an icon; otherwise we leave it up to the applet itself to do this properly as it normally would have to
BUG:226391


 M  +7 -2      popupapplet.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1088928
Comment 4 Aaron J. Seigo 2010-02-11 23:53:45 UTC
SVN commit 1088931 by aseigo:

we should only muck with the minimum sizes of the popup applet when iconified if it is actually using an icon; otherwise we leave it up to the applet itself to do this properly as it normally would have to
CCBUG:226391


 M  +7 -2      popupapplet.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1088931
Comment 5 Dario Andres 2010-02-26 13:57:57 UTC
This should be fixed for KDE SC 4.4.1/4.5
Comment 6 Dario Andres 2010-02-26 13:58:06 UTC
*** Bug 219793 has been marked as a duplicate of this bug. ***
Comment 7 Dario Andres 2010-02-26 13:58:12 UTC
*** Bug 227591 has been marked as a duplicate of this bug. ***