[Merge] lp:~mterry/powerd/ussd-turn-on-screen-for-notification into lp:powerd

Michael Terry michael.terry at canonical.com
Thu Aug 6 14:15:41 UTC 2015


Michael Terry has proposed merging lp:~mterry/powerd/ussd-turn-on-screen-for-notification into lp:powerd.

Commit message:
Use the NOTIFICATION reason when asking Unity System Compositor to wake up the screen for a UUSD message. This means the screen will only be on 15s instead of 60s.

This is a follow-on from using the NOTIFICATION reason for SMS messages [1]. But USSD messages should also be treated as notifications, right?

(I've left the CallAdded notification alone, since incoming rings last longer than 15s and it's a higher priority notification.)

[1] https://code.launchpad.net/~afrantzis/powerd/sms-turn-on-screen-for-notification/+merge/265271

Requested reviews:
  Ubuntu Phablet Team (phablet-team)
Related bugs:
  Bug #1426115 in unity-system-compositor (Ubuntu): "Screen timeout for system event (e.g. notification) needs to be a shorter duration than the standard inactivity timeout"
  https://bugs.launchpad.net/ubuntu/+source/unity-system-compositor/+bug/1426115

For more details, see:
https://code.launchpad.net/~mterry/powerd/ussd-turn-on-screen-for-notification/+merge/267197

Use the NOTIFICATION reason when asking Unity System Compositor to wake up the screen for a UUSD message. This means the screen will only be on 15s instead of 60s.

This is a follow-on from using the NOTIFICATION reason for SMS messages [1]. But USSD messages should also be treated as notifications, right?

(I've left the CallAdded notification alone, since incoming rings last longer than 15s and it's a higher priority notification.)

[1] https://code.launchpad.net/~afrantzis/powerd/sms-turn-on-screen-for-notification/+merge/265271
-- 
Your team Ubuntu Phablet Team is requested to review the proposed merge of lp:~mterry/powerd/ussd-turn-on-screen-for-notification into lp:powerd.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: review-diff.txt
Type: text/x-diff
Size: 446 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/ubuntu-reviews/attachments/20150806/e2e952e6/attachment.diff>


More information about the Ubuntu-reviews mailing list