Unity System Compositor

Merge lp://qastaging/~afrantzis/unity-system-compositor/turn-on-display-only-on-key-down into lp://qastaging/unity-system-compositor

  1. turn-on-display-only-on-key-down
  2. Merge into trunk
Proposed by Alexandros Frantzis
Status: Merged
Approved by: Alexandros Frantzis
Approved revision: 286
Merged at revision: 290
Proposed branch: lp://qastaging/~afrantzis/unity-system-compositor/turn-on-display-only-on-key-down
Merge into: lp://qastaging/unity-system-compositor
Diff against target: 125 lines (+48/-5)
3 files modified
src/screen_event_handler.cpp (+12/-3)
src/screen_event_handler.h (+1/-0)
tests/unit-tests/test_screen_event_handler.cpp (+35/-2)
To merge this branch: bzr merge lp://qastaging/~afrantzis/unity-system-compositor/turn-on-display-only-on-key-down
Reviewer Review Type Date Requested Status
Cemil Azizoglu (community) Approve
Alberto Aguirre (community) Approve
Andreas Pokorny (community) Approve
Mir CI Bot continuous-integration Approve
Review via email: mp+290056@code.qastaging.launchpad.net

Commit message

ScreenEventHandler: Allow only key down events to turn screen on

Description of the change

ScreenEventHandler: Allow only key down events to turn screen on

To post a comment you must log in.
Revision history for this message
Mir CI Bot (mir-ci-bot) wrote :

PASSED: Continuous integration, rev:286
https://mir-jenkins.ubuntu.com/job/usc-ci/22/
Executed test runs:
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-usc/27
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-0-fetch/630
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/622
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial/622
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-usc/arch=amd64,release=xenial/31
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-usc/arch=amd64,release=xenial/31/artifact/output/*zip*/output.zip
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-usc/arch=armhf,release=vivid+overlay/31
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-usc/arch=armhf,release=vivid+overlay/31/artifact/output/*zip*/output.zip
    SUCCESS: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-usc/arch=i386,release=xenial/31
        deb: https://mir-jenkins.ubuntu.com/job/build-2-binpkg-usc/arch=i386,release=xenial/31/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://mir-jenkins.ubuntu.com/job/usc-ci/22/rebuild

review: Approve (continuous-integration)
Revision history for this message
Andreas Pokorny (andreas-pokorny) wrote :

yes looks perfectly reasonable.

review: Approve
Revision history for this message
Alberto Aguirre (albaguirre) wrote :

LGTM.

review: Approve
Revision history for this message
Cemil Azizoglu (cemil-azizoglu) wrote :

Ok

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches