Merge lp://staging/~mterry/unity8/mouse-cursor-speed into lp://staging/unity8

Proposed by Michael Terry
Status: Merged
Approved by: Michael Zanetti
Approved revision: 2138
Merged at revision: 2161
Proposed branch: lp://staging/~mterry/unity8/mouse-cursor-speed
Merge into: lp://staging/unity8
Diff against target: 141 lines (+66/-1)
3 files modified
debian/control (+1/-1)
plugins/AccountsService/AccountsService.cpp (+61/-0)
plugins/AccountsService/AccountsService.h (+4/-0)
To merge this branch: bzr merge lp://staging/~mterry/unity8/mouse-cursor-speed
Reviewer Review Type Date Requested Status
Michael Zanetti (community) Approve
PS Jenkins bot (community) continuous-integration Needs Fixing
Unity8 CI Bot continuous-integration Needs Fixing
Review via email: mp+283547@code.staging.launchpad.net

Commit message

Synchronize the mouse-cursor-speed and touchpad-cursor-speed properties with USC.

Description of the change

Not all the properties from the new AS schema are used. I'm just starting with a few urgent ones. (I want to add some refactoring if we're going to be adding 8 new properties.)

You can test changing the property with:
sudo gdbus call --system --dest org.freedesktop.Accounts --object-path /org/freedesktop/Accounts/User32011 --method org.freedesktop.DBus.Properties.Set com.ubuntu.AccountsService.Input MouseCursorSpeed '<0.75>'

 * Are there any related MPs required for this MP to build/function as expected? Please list.
 Silo 45 (https://code.launchpad.net/~andreas-pokorny/unity-system-compositor/fix-1535919/+merge/283309)
 https://code.launchpad.net/~mterry/gsettings-ubuntu-touch-schemas/input-schema/+merge/283546

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

 * 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?
 I'm on the team.

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

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

FAILED: Continuous integration, rev:2138
https://unity8-jenkins.ubuntu.com/job/lp-unity8-1-ci/166/
Executed test runs:

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

review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:2138
http://jenkins.qa.ubuntu.com/job/unity8-ci/7127/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-vivid-touch/6130/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-xenial-touch/542/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-vivid/1832
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/unity8-qmluitest-xenial-amd64/535
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-vivid-amd64-ci/1727
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-vivid-i386-ci/1727
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-xenial-amd64-ci/534
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-xenial-i386-ci/533
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-vivid-touch/4729/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/6141
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/6141/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/26914
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-xenial-touch/278/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-xenial-armhf/540
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-xenial-armhf/540/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/26915

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

review: Needs Fixing (continuous-integration)
Revision history for this message
Michael Zanetti (mzanetti) wrote :

 * 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.

yip

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

yes

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