Merge lp://staging/~mterry/powerd/ussd-turn-on-screen-for-notification into lp://staging/powerd
Status: | Needs review |
---|---|
Proposed branch: | lp://staging/~mterry/powerd/ussd-turn-on-screen-for-notification |
Merge into: | lp://staging/powerd |
Diff against target: |
11 lines (+1/-1) 1 file modified
src/powerd.cpp (+1/-1) |
To merge this branch: | bzr merge lp://staging/~mterry/powerd/ussd-turn-on-screen-for-notification |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
PS Jenkins bot | continuous-integration | Approve | |
Ubuntu Phablet Team | Pending | ||
Review via email: mp+267197@code.staging.launchpad.net |
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:/
Description of the change
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:/
Unmerged revisions
- 177. By Michael Terry
-
Use NOTIFICATION reason when waking the screen for a USSD message
PASSED: Continuous integration, rev:177 jenkins. qa.ubuntu. com/job/ powerd- ci/184/ jenkins. qa.ubuntu. com/job/ powerd- vivid-armhf- ci/12 jenkins. qa.ubuntu. com/job/ powerd- vivid-armhf- ci/12/artifact/ work/output/ *zip*/output. zip
http://
Executed test runs:
SUCCESS: http://
deb: http://
Click here to trigger a rebuild: s-jenkins. ubuntu- ci:8080/ job/powerd- ci/184/ rebuild
http://