Bug 226391

Summary: Hiding battery monitor embedded in SystemTray causes 100% CPU
Product: [Unmaintained] plasma4 Reporter: Paul <paulhollensen>
Component: generalAssignee: Plasma Bugs List <plasma-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: andresbajotierra, aseigo, juha.tiensyrja, larvid
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: openSUSE   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:

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. ***