Unity 8

Merge lp://qastaging/~mterry/unity8/new-input-proxies into lp://qastaging/unity8

new-input-proxies
Merge into trunk

Proposed by Michael Terry on 2016-02-08

Status:

Superseded

Proposed branch:

lp://qastaging/~mterry/unity8/new-input-proxies

Merge into:

lp://qastaging/unity8

Diff against target:

375 lines (+204/-5)

9 files modified

plugins/AccountsService/AccountsService.cpp (+54/-2)
plugins/AccountsService/AccountsService.h (+4/-1)
tests/plugins/AccountsService/CMakeLists.txt (+3/-0)
tests/plugins/AccountsService/PropertiesServer.cpp (+1/-2)
tests/plugins/AccountsService/UscServer.cpp (+28/-0)
tests/plugins/AccountsService/UscServer.h (+39/-0)
tests/plugins/AccountsService/client.cpp (+55/-0)
tests/plugins/AccountsService/interfaces.xml (+10/-0)
tests/plugins/AccountsService/server.cpp (+10/-0)

To merge this branch:

bzr merge lp://qastaging/~mterry/unity8/new-input-proxies

High

Fix Released

Link a bug report

Reviewer	Review Type	Date Requested	Status
Michał Sawicz			Abstain on 2016-02-22
Unity8 CI Bot	continuous-integration		Needs Fixing on 2016-02-17
PS Jenkins bot (community)	continuous-integration		Needs Fixing on 2016-02-17
Lukáš Tinkl (community)		2016-02-08	Approve on 2016-02-09
Review via email: mp+285364@code.qastaging.launchpad.net

This proposal has been superseded by a proposal from 2016-02-25.

Commit message

Proxy more mouse and touchpad properties to USC (LP: #1540398)

Description of the change

Mostly simple, but I added a converter capacity to our AccountsService proxy relay. And added some tests for proxy support in general (I realized it had been missed!) and specifically the converter support.

Note that MouseDoubleClickSpeed and TouchpadDoubleClickSpeed are not supported on the USC side yet. The properties exist in AccountsService, but USC/Mir haven't caught up. I added them anyway, so that there's one less thing to do to support them. It won't hurt us any to try and fail to proxy them until USC adds support.

= Checklist =

* Are there any related MPs required for this MP to build/function as expected? Please list.
Yes, just the pre-requisite.

* Did you perform an exploratory manual test run of your code change and any related functionality?
No, it's just adding properties.

* Did you make sure that your branch does not contain spurious tags?
Yes

* If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?
NA

* If you changed the UI, has there been a design review?
NA

Revision history for this message

Unity8 CI Bot (unity8-ci-bot) wrote on 2016-02-08:

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-1-ci/292/rebuild

review: Needs Fixing (continuous-integration)

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2016-02-08:

FAILED: Continuous integration, rev:2142
http://jenkins.qa.ubuntu.com/job/unity8-ci/7234/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-vivid-touch/6381
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-xenial-touch/649/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-vivid/1939
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-qmluitest-xenial-amd64/642/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-vivid-amd64-ci/1834
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-vivid-i386-ci/1834
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-xenial-amd64-ci/641/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-xenial-i386-ci/640/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-vivid-touch/4893
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/6392
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/6392/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/27391
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-xenial-armhf/647/console

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

review: Needs Fixing (continuous-integration)

Revision history for this message

Lukáš Tinkl (lukas-kde) wrote on 2016-02-08:

Some minor stuff in inline comments, otherwise looks great

review: Needs Fixing (code-review)

Revision history for this message

Michael Terry (mterry) wrote on 2016-02-09:

Commented, updated.

Revision history for this message

Unity8 CI Bot (unity8-ci-bot) wrote on 2016-02-09:

FAILED: Continuous integration, rev:2144
https://unity8-jenkins.ubuntu.com/job/lp-unity8-1-ci/302/
Executed test runs:
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build/423/console
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/446
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/464
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial/464
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/460/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial/460/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/460/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial/460/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/460/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial/460/console

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-1-ci/302/rebuild

review: Needs Fixing (continuous-integration)

Revision history for this message

Lukáš Tinkl (lukas-kde) wrote on 2016-02-09:

Cool, I like it

* Did you perform an exploratory manual test run of the code change and any related functionality?

Yes

* Did CI run pass? If not, please explain why.

No, missing explicit dependency on https://code.launchpad.net/~unity-team/unity-api/surfaceItemSizeHints/+merge/284890 in the chain

* Did you make sure that the branch does not contain spurious tags?

Yes

review: Approve

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2016-02-09:

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

review: Needs Fixing (continuous-integration)

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2016-02-12:

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

review: Needs Fixing (continuous-integration)

Revision history for this message

Michael Terry (mterry) wrote on 2016-02-16:

I had to muck with the bzr history due to some pre-req shenanigans. But the diff is the same.

Revision history for this message

Michał Sawicz (saviq) wrote on 2016-02-17:

CMake Error at tests/plugins/AccountsService/CMakeLists.txt:28 (add_executable):
Cannot find source file:

UscServer.cpp

review: Needs Fixing

Revision history for this message

Unity8 CI Bot (unity8-ci-bot) wrote on 2016-02-17:

FAILED: Continuous integration, rev:2145
https://unity8-jenkins.ubuntu.com/job/lp-unity8-1-ci/399/
Executed test runs:
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build/537/console
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/560
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/578
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial/578
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/574/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial/574/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/574/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial/574/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/574/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial/574/console

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-1-ci/399/rebuild

review: Needs Fixing (continuous-integration)

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2016-02-17:

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

review: Needs Fixing (continuous-integration)

lp://qastaging/~mterry/unity8/new-input-proxies updated on 2016-02-17

2146. By Michael Terry on 2016-02-17: Whoops, add missing files

Revision history for this message

Michael Terry (mterry) wrote on 2016-02-17:

Whoops, forgot to bzr add UscServer.{cpp,h} when I was rebasing this branch. Those files were included in the original MP and review. I just dropped them when futzing with bzr history. :-/

Fixed now.

Revision history for this message

Unity8 CI Bot (unity8-ci-bot) wrote on 2016-02-17:

FAILED: Continuous integration, rev:2146
https://unity8-jenkins.ubuntu.com/job/lp-unity8-1-ci/400/
Executed test runs:
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build/539/console
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/562
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/580
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial/580
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/576/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial/576/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/576/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial/576/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/576/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial/576/console