[Merge] lp:~mterry/telephony-service/notification-timeout into lp:telephony-service
Michael Terry
michael.terry at canonical.com
Thu Aug 6 14:05:41 UTC 2015
Michael Terry has proposed merging lp:~mterry/telephony-service/notification-timeout into lp:telephony-service.
Commit message:
Use the NOTIFICATION reason when asking Unity System Compositor to wake up the screen for an MMS. This means the screen will only be on 15s instead of 60s.
This is a follow-on from USC adding support for the NOTIFICATION reason [1]. We don't need to bump our dependency version since USC will treat unknown reasons as normal reasons (from my look through the code).
[1] https://code.launchpad.net/~afrantzis/unity-system-compositor/notification-timeouts/+merge/264853
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/telephony-service/notification-timeout/+merge/267196
Use the NOTIFICATION reason when asking Unity System Compositor to wake up the screen for an MMS. This means the screen will only be on 15s instead of 60s.
This is a follow-on from USC adding support for the NOTIFICATION reason [1]. We don't need to bump our dependency version since USC will treat unknown reasons as normal reasons (from my look through the code).
[1] https://code.launchpad.net/~afrantzis/unity-system-compositor/notification-timeouts/+merge/264853
--
Your team Ubuntu Phablet Team is requested to review the proposed merge of lp:~mterry/telephony-service/notification-timeout into lp:telephony-service.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: review-diff.txt
Type: text/x-diff
Size: 489 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/ubuntu-reviews/attachments/20150806/15bfffd5/attachment.diff>
More information about the Ubuntu-reviews
mailing list