Unify code that turns screen on when a notification happens
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
powerd (Ubuntu) |
In Progress
|
Undecided
|
Unassigned | ||
telephony-service (Ubuntu) |
In Progress
|
Undecided
|
Unassigned | ||
unity8 (Ubuntu) |
Fix Released
|
Undecided
|
Michael Terry |
Bug Description
Right now, there are several places we ask USC to turn the screen on when a notification happens:
- powerd for SMS, USSD, and incoming calls
- telephony-service for MMS and telephony messages
And we don't seem to be turning them on for ubuntu-push notifications, but we should, right? Following the pattern above, we'd have ubuntu-push itself turn the screen on, for a third location.
We don't turn the screen on at all for notifications that don't come via those sources. For example, the following command line doesn't turn the screen on, but I think it should:
gdbus call --session --dest org.freedesktop
It feels weird to have telephony-service turning the screen on. Logically, it is just emitting a notification. The presentation of that notification is up to a higher layer (maybe we're on a raspberry pi2 and we present notifications as LED morse code or whatever). It just makes sense that unity8, which is drawing the notification, is the one that knows that the screen needs to be on. It also knows what form factor we are in (I assume we only want to turn screen on for phones/tablets, not desktops).
So I'm proposing that we unify all those places into one code path in unity8 that can turn the screen on when a notification is received.
(This is a breakout bug from bug 1426115.)
Related branches
- Albert Astals Cid (community): Approve
- PS Jenkins bot (community): Needs Fixing (continuous-integration)
- Unity Team: Pending requested
-
Diff: 69 lines (+21/-1)3 files modifiedplugins/Powerd/Powerd.cpp (+7/-0)
plugins/Powerd/Powerd.h (+4/-0)
qml/Notifications/Notification.qml (+10/-1)
- PS Jenkins bot: Approve (continuous-integration)
- Ubuntu Phablet Team: Pending requested
-
Diff: 145 lines (+0/-75)3 files modifiedsrc/powerd-internal.h (+0/-4)
src/powerd-object.c (+0/-8)
src/powerd.cpp (+0/-63)
- PS Jenkins bot: Approve (continuous-integration)
- Ubuntu Phablet Team: Pending requested
-
Diff: 24 lines (+0/-15)1 file modifiedindicator/textchannelobserver.cpp (+0/-15)
description: | updated |
Changed in unity8 (Ubuntu): | |
status: | Confirmed → Fix Released |
Changed in telephony-service (Ubuntu): | |
assignee: | nobody → Michael Terry (mterry) |
status: | New → In Progress |
Changed in powerd (Ubuntu): | |
assignee: | nobody → Michael Terry (mterry) |
status: | New → In Progress |
Changed in telephony-service (Ubuntu): | |
assignee: | Michael Terry (mterry) → nobody |
Changed in powerd (Ubuntu): | |
assignee: | Michael Terry (mterry) → nobody |
Humph... Unity8 is not running when the device is suspended, is it? That might put a damper on this plan.