Merge lp:~afrantzis/powerd/sms-turn-on-screen-for-notification into lp:powerd

Proposed by Alexandros Frantzis
Status: Merged
Approved by: Alexandros Frantzis
Approved revision: 173
Merged at revision: 173
Proposed branch: lp:~afrantzis/powerd/sms-turn-on-screen-for-notification
Merge into: lp:powerd
Diff against target: 23 lines (+2/-1)
2 files modified
src/powerd-internal.h (+1/-0)
src/powerd.cpp (+1/-1)
To merge this branch: bzr merge lp:~afrantzis/powerd/sms-turn-on-screen-for-notification
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Kevin DuBois (community) Approve
Review via email: mp+265271@code.launchpad.net

Commit message

Turn the screen on using reason 'notification' when an SMS arrives

Description of the change

Turn the screen on using reason 'notification' when an SMS arrives

'notification' has been added as a reason in the latest unity-system-compositor.

To post a comment you must log in.
Revision history for this message
Kevin DuBois (kdub) wrote :

lgtm

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/powerd-internal.h'
2--- src/powerd-internal.h 2015-05-14 23:53:12 +0000
3+++ src/powerd-internal.h 2015-07-20 14:58:10 +0000
4@@ -37,6 +37,7 @@
5 {
6 UNITY_SCREEN_REASON_NORMAL = 0,
7 UNITY_SCREEN_REASON_PROXIMITY = 3,
8+ UNITY_SCREEN_REASON_NOTIFICATION = 4,
9 };
10
11 struct DbusNameWatch {
12
13=== modified file 'src/powerd.cpp'
14--- src/powerd.cpp 2015-05-07 20:32:30 +0000
15+++ src/powerd.cpp 2015-07-20 14:58:10 +0000
16@@ -284,7 +284,7 @@
17 (!strcmp(signal_name, "ImmediateMessage"))) &&
18 (call_added == 0)) {
19 powerd_debug("Waking up the device - Incoming SMS");
20- turn_display_on(TRUE, UNITY_SCREEN_REASON_NORMAL);
21+ turn_display_on(TRUE, UNITY_SCREEN_REASON_NOTIFICATION);
22 }
23 }
24

Subscribers

People subscribed via source and target branches