music-app blocks system suspend even when not playing any song
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Canonical System Image |
Fix Released
|
High
|
Unassigned | ||
| qtmir (Ubuntu) |
Fix Released
|
High
|
Gerry Boland | ||
| qtmir (Ubuntu RTM) |
Fix Released
|
High
|
Gerry Boland | ||
| unity8 (Ubuntu RTM) |
Invalid
|
Undecided
|
Unassigned | ||
Bug Description
current build number: 241
device name: krillin
channel: ubuntu-
last update: 2015-02-20 00:41:40
version version: 241
version ubuntu: 20150219
version device: 20150216-fe747ac
version custom: 20150216-561-29-186
I know we currently have an exception in the lifecycle for music-app, and it seems that there is a quite critical bug with it at the current image.
Opening up music-app is already enough for unity8 to hold a system state request from powerd, blocking suspend until you close the app (having it in the background doesn't really help here).
Ideally the system state request (wakelock) should just be enabled and active when the music-app is also active, playing songs.
This bug can drastically affect the battery life for the user of the phone.
You can check this by running 'powerd-cli list' after opening music-app.
phablet@
powerd-cli: Running as user is not fully supported.
System State Requests:
Name: active, Owner: :1.77, State: 1
Related branches
- Albert Astals Cid (community): Approve
- PS Jenkins bot (community): Approve (continuous-integration)
- Michał Sawicz: Abstain
- Gerry Boland (community): Abstain
- Michael Zanetti (community): Approve
-
Diff: 1165 lines (+646/-246)13 files modifiedCMakeLists.txt (+2/-0)
debian/control (+2/-0)
src/common/abstractdbusservicemonitor.cpp (+2/-3)
src/common/abstractdbusservicemonitor.h (+1/-7)
src/modules/Unity/Application/application.cpp (+3/-1)
src/modules/Unity/Application/application_manager.cpp (+5/-2)
src/modules/Unity/Application/sharedwakelock.cpp (+120/-57)
src/modules/Unity/Application/sharedwakelock.h (+14/-9)
tests/modules/Application/application_test.cpp (+7/-7)
tests/modules/ApplicationManager/application_manager_test.cpp (+78/-0)
tests/modules/SharedWakelock/CMakeLists.txt (+6/-0)
tests/modules/SharedWakelock/sharedwakelock_test.cpp (+365/-146)
tests/modules/common/mock_shared_wakelock.h (+41/-14)
- Albert Astals Cid (community): Approve
- Gerry Boland (community): Abstain
-
Diff: 1184 lines (+653/-249)15 files modifieddebian/control (+2/-0)
src/common/abstractdbusservicemonitor.cpp (+2/-3)
src/common/abstractdbusservicemonitor.h (+1/-7)
src/modules/Unity/Application/Application.pro (+1/-1)
src/modules/Unity/Application/application.cpp (+3/-1)
src/modules/Unity/Application/application_manager.cpp (+5/-2)
src/modules/Unity/Application/sharedwakelock.cpp (+120/-57)
src/modules/Unity/Application/sharedwakelock.h (+14/-9)
tests/modules/Application/application_test.cpp (+7/-7)
tests/modules/ApplicationManager/application_manager_test.cpp (+78/-0)
tests/modules/SharedWakelock/SharedWakelock.pro (+11/-0)
tests/modules/SharedWakelock/sharedwakelock_test.cpp (+365/-146)
tests/modules/common/common.pri (+2/-1)
tests/modules/common/mock_shared_wakelock.h (+41/-14)
tests/modules/modules.pro (+1/-1)
| affects: | unity8 (Ubuntu) → unity8 (Ubuntu RTM) |
| tags: | added: battery |
| Changed in canonical-devices-system-image: | |
| status: | New → Confirmed |
| importance: | Undecided → High |
| Changed in qtmir (Ubuntu RTM): | |
| assignee: | nobody → Gerry Boland (gerboland) |
| importance: | Undecided → High |
| status: | New → In Progress |
| Changed in unity8 (Ubuntu RTM): | |
| status: | New → Invalid |
| Changed in canonical-devices-system-image: | |
| milestone: | none → ww13-ota |
| Changed in canonical-devices-system-image: | |
| status: | Confirmed → In Progress |
| Launchpad Janitor (janitor) wrote | #1 |
| Changed in qtmir (Ubuntu): | |
| status: | New → Confirmed |
| Changed in qtmir (Ubuntu): | |
| status: | Confirmed → In Progress |
| assignee: | nobody → Gerry Boland (gerboland) |
| importance: | Undecided → High |
| Launchpad Janitor (janitor) wrote | #2 |
| Changed in qtmir (Ubuntu RTM): | |
| status: | In Progress → Fix Released |
| Changed in canonical-devices-system-image: | |
| status: | In Progress → Fix Released |
| Changed in qtmir (Ubuntu): | |
| assignee: | Gerry Boland (gerboland) → Michał Sawicz (saviq) |
| Changed in qtmir (Ubuntu): | |
| assignee: | Michał Sawicz (saviq) → Gerry Boland (gerboland) |
| Launchpad Janitor (janitor) wrote | #3 |
| Changed in qtmir (Ubuntu): | |
| status: | In Progress → Fix Released |
Status changed to 'Confirmed' because the bug affects multiple users.