Bug 339957

Summary:	Battery monitor - incorrect time remaining in tooltip.
Product:	[Plasma] plasmashell	Reporter:	perrantrevan
Component:	Power and Battery widget	Assignee:	Kai Uwe Broulik <kde>
Status:	RESOLVED FIXED
Severity:	normal	CC:	aleixpol, hrvoje.senjan, rdieter
Priority:	NOR
Version:	master
Target Milestone:	1.0
Platform:	Arch Linux
OS:	Linux
Latest Commit:	http://commits.kde.org/powerdevil/b20ee2bd244db2620e84e4b71e77675036cb15bd	Version Fixed In:	5.1.2
Sentry Crash Report:

Description perrantrevan 2014-10-14 13:21:29 UTC

In the battery monitor tooltip the time remaining is  never updated as the battery is depleted (the % of charge remaining is updated however)

Reproducible: Always

Steps to Reproduce:
1. Unplug laptop power cable
2. Move cursor over battery monitor tooltip.
3. The time remaining is shown along with percentage of charge remaining.
4. Repeat steps 2 and 3. Observe time remaining.

Actual Results:  
The charge remaining is updated if you move the mouse over the battery monitor. However, the time remaining stays the same.

Expected Results:  
The time remaining should be updated as the battery depletes.

Comment 1 Kai Uwe Broulik 2014-10-14 13:26:38 UTC

*** Bug 339958 has been marked as a duplicate of this bug. ***

Comment 2 Kai Uwe Broulik 2014-10-14 13:27:56 UTC

I cannot reproduce this issue running master. Both the batterymonitor tooltip (hovering the tray icon) as well as the tooltip of an individual battery is updated properly.

Can you repeatedly check the output of upower -d whether the time updates there? If it does not, this is likely a driver/acpi issue.

Comment 3 perrantrevan 2014-10-14 13:41:24 UTC

Hi, the times are completely different.

Battery monitor tooltip = 1:49:20

1st upower -d = 4.8 hours
2nd upower -d = 2.4 hours
3rd upowerd -d = 2.8 hours
4th upowerd -d = 4.8 hours

P

Comment 4 Aleix Pol 2014-11-24 16:26:43 UTC

So what happens with this? is it an upstream bug? Kai?

Comment 5 Kai Uwe Broulik 2014-12-04 08:44:46 UTC

https://git.reviewboard.kde.org/r/121343/

Comment 6 Weng Xuetian 2014-12-05 16:05:17 UTC

Git commit 23ad576c3fdbd1498b88907f6c9ee0cd693fa834 by Weng Xuetian.
Committed on 05/12/2014 at 15:56.
Pushed by xuetianweng into branch 'Plasma/5.1'.

Fix battery remaining time update with upower >= 0.99

Existing device is not connected to PropertiesChanged signal, so it
never recevies update.

REVIEW: 121343
FIXED-IN: 5.1.2

M  +6    -4    daemon/backends/upower/powerdevilupowerbackend.cpp
M  +1    -0    daemon/backends/upower/powerdevilupowerbackend.h

http://commits.kde.org/powerdevil/23ad576c3fdbd1498b88907f6c9ee0cd693fa834

Comment 7 Weng Xuetian 2014-12-05 16:05:31 UTC

Git commit b20ee2bd244db2620e84e4b71e77675036cb15bd by Weng Xuetian.
Committed on 05/12/2014 at 15:56.
Pushed by xuetianweng into branch 'master'.

Fix battery remaining time update with upower >= 0.99

Existing device is not connected to PropertiesChanged signal, so it
never recevies update.

REVIEW: 121343
FIXED-IN: 5.1.2

M  +6    -4    daemon/backends/upower/powerdevilupowerbackend.cpp
M  +1    -0    daemon/backends/upower/powerdevilupowerbackend.h

http://commits.kde.org/powerdevil/b20ee2bd244db2620e84e4b71e77675036cb15bd

Comment 8 Hrvoje Senjan 2014-12-10 17:38:12 UTC

@Weng, i'm guessing this could/should be backported to kde-workspace also?