Summary: | Solid inhibit fails the next suspend if inhibit outlasts suspend countdown while application is still open | ||
---|---|---|---|
Product: | [Plasma] Powerdevil | Reporter: | Jamie Smith <smithjd15> |
Component: | general | Assignee: | Plasma Development Mailing List <plasma-devel> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | kde, matej, oliver.henshaw, smithjd15 |
Priority: | NOR | ||
Version: | 5.1.95 | ||
Target Milestone: | --- | ||
Platform: | unspecified | ||
OS: | Linux | ||
URL: | http://paste.kde.org/675494/ | ||
Latest Commit: | http://commits.kde.org/powerdevil/bf9bf79c92a3d77df863d15c543f7f5329a564dc | Version Fixed In: | 5.4.0 |
Bug Depends on: | 318461 | ||
Bug Blocks: |
Description
Jamie Smith
2013-02-19 10:18:31 UTC
Reproduced here. Another way to reproduce this/related bug: 1. run Amarok git that does suspend inhibition 2. configure Solid to suspend on lid close 3. add one song to Amarok playlist, play it 4. Close lid, Amarok playback inhibits suspend 5. The playback ends, Amarok stops inhibiting 6. Nothing happens, while the system should've suspended as the lid is closed and no reason to inhibit suspend exists Still valid in 5.x, re-assigning there, also it qualifies as a bug imho. Git commit bf9bf79c92a3d77df863d15c543f7f5329a564dc by Kai Uwe Broulik. Committed on 06/06/2015 at 20:25. Pushed by broulik into branch 'master'. Simulate user activity when inhibitions change This way when an inhibition outlasts an idle action (eg. 10 minute timeout for auto-suspend but download takes 11 minutes) we make sure that 10 minutes after the download has finished, the system suspends, as well as vice-versa when a video stops after 9 minutes the system waits another 10 to suspend rather than suspending within the next minute. Since this patch has the potential for unwanted side-effects (screen waking up when download finishes, I could imagine) I'll leave this as 5.4 material and not part of the 5.3.x patch releases to give it more thorough testing. FIXED-IN: 5.4.0 M +6 -0 daemon/powerdevilcore.cpp http://commits.kde.org/powerdevil/bf9bf79c92a3d77df863d15c543f7f5329a564dc |