Mir

Merge lp://qastaging/~afrantzis/mir/set-base-display-config-authorization into lp://qastaging/mir

  1. set-base-display-config-authorization
  2. Merge into development-branch
Proposed by Alexandros Frantzis
Status: Merged
Approved by: Alexandros Frantzis
Approved revision: no longer in the source branch.
Merged at revision: 3132
Proposed branch: lp://qastaging/~afrantzis/mir/set-base-display-config-authorization
Merge into: lp://qastaging/mir
Prerequisite: lp://qastaging/~afrantzis/mir/client-set-base-display-config
Diff against target: 306 lines (+127/-38)
10 files modified
include/server/mir/frontend/session_authorizer.h (+1/-0)
include/test/mir/test/doubles/stub_session_authorizer.h (+4/-0)
src/server/default_server_configuration.cpp (+5/-0)
src/server/frontend/default_configuration.cpp (+5/-0)
src/server/frontend/default_ipc_factory.cpp (+4/-8)
src/server/frontend/unauthorized_display_changer.cpp (+29/-9)
src/server/frontend/unauthorized_display_changer.h (+12/-3)
tests/acceptance-tests/test_client_authorization.cpp (+1/-0)
tests/acceptance-tests/test_display_configuration.cpp (+29/-4)
tests/unit-tests/frontend/test_unauthorized_display_changer.cpp (+37/-14)
To merge this branch: bzr merge lp://qastaging/~afrantzis/mir/set-base-display-config-authorization
Reviewer Review Type Date Requested Status
Alberto Aguirre (community) Approve
Kevin DuBois (community) Approve
Alan Griffiths Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+277235@code.qastaging.launchpad.net

Commit message

frontend: Allow independent authorization of set base display configuration requests

Description of the change

frontend: Allow independent authorization of set base display configuration requests

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:3100
http://jenkins.qa.ubuntu.com/job/mir-ci/5538/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mir-android-vivid-i386-build/4852
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mir-clang-vivid-amd64-build/3759
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mir-mediumtests-vivid-touch/4794
    FAILURE: http://jenkins.qa.ubuntu.com/job/mir-mediumtests-wily-touch/752/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mir-wily-amd64-ci/1692
        deb: http://jenkins.qa.ubuntu.com/job/mir-wily-amd64-ci/1692/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mir-wily-i386-ci/752
        deb: http://jenkins.qa.ubuntu.com/job/mir-wily-i386-ci/752/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mir-mediumtests-builder-vivid-armhf/4795
        deb: http://jenkins.qa.ubuntu.com/job/mir-mediumtests-builder-vivid-armhf/4795/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mir-mediumtests-runner-touch/7392
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/25096
    FAILURE: http://jenkins.qa.ubuntu.com/job/mir-mediumtests-builder-wily-armhf/753/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/mir-ci/5538/rebuild

review: Approve (continuous-integration)
Revision history for this message
Alan Griffiths (alan-griffiths) wrote :

LGTM

review: Approve
Revision history for this message
Kevin DuBois (kdub) wrote :

looks good

review: Approve
Revision history for this message
Alberto Aguirre (albaguirre) 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