Merge lp:~mzanetti/unity8/catch-osk into lp:unity8

Proposed by Michael Zanetti
Status: Merged
Approved by: Michał Sawicz
Approved revision: 2189
Merged at revision: 2245
Proposed branch: lp:~mzanetti/unity8/catch-osk
Merge into: lp:unity8
Diff against target: 198 lines (+25/-30)
10 files modified
qml/Components/InputMethod.qml (+1/-9)
tests/mocks/Unity/Application/MirSurface.cpp (+3/-2)
tests/mocks/Unity/Application/MirSurface.h (+2/-1)
tests/mocks/Unity/Application/SurfaceManager.cpp (+8/-9)
tests/mocks/Unity/Application/SurfaceManager.h (+3/-3)
tests/mocks/Unity/Application/VirtualKeyboard.cpp (+3/-2)
tests/mocks/Unity/Application/VirtualKeyboard.h (+1/-1)
tests/mocks/Unity/Application/plugin.cpp (+2/-1)
tests/qmltests/tst_OrientedShell.qml (+1/-1)
tests/qmltests/tst_Shell.qml (+1/-1)
To merge this branch: bzr merge lp:~mzanetti/unity8/catch-osk
Reviewer Review Type Date Requested Status
Michał Sawicz ci Approve
Unity8 CI Bot continuous-integration Approve
Lukáš Tinkl (community) Approve
Daniel d'Andrada (community) Approve
PS Jenkins bot (community) continuous-integration Needs Fixing
Review via email: mp+286076@code.launchpad.net

Commit message

Read inputMethod surface from the new property int QtMir

The previous signal had issues if the InputMethod in unity was
created after the surfaceAddedSignal() was emitted.

Description of the change

 * Are there any related MPs required for this MP to build/function as expected? Please list.

https://code.launchpad.net/~mzanetti/qtmir/surfacemanager-getters/+merge/286074

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

n/a

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

n/a

To post a comment you must log in.
Revision history for this message
Daniel d'Andrada (dandrader) wrote :

What about adding a simple property to SurfaceManager that contains the osk surface?

I mean, the general idea is that qtmir should provide us with a QML API that would simply unity8's life as much as possible.

review: Needs Information
Revision history for this message
Daniel d'Andrada (dandrader) wrote :

s/simply/simplify

Revision history for this message
Lukáš Tinkl (lukas-kde) wrote :

12 + surfaceItem.surface = surface;
13 + return;

This should probably "break", not "return", in case we might add some stuff below the loop in the future.

Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

FAILED: Continuous integration, rev:2182
https://unity8-jenkins.ubuntu.com/job/lp-unity8-1-ci/355/
Executed test runs:
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build/486/console
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/509
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/527
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial/527
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/523
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/523/artifact/output/*zip*/output.zip
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial/523/console
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/523
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/523/artifact/output/*zip*/output.zip
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial/523/console
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/523
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/523/artifact/output/*zip*/output.zip
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial/523/console

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

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

> What about adding a simple property to SurfaceManager that contains the osk
> surface?
>
> I mean, the general idea is that qtmir should provide us with a QML API that
> would simply unity8's life as much as possible.

done

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:2182
http://jenkins.qa.ubuntu.com/job/unity8-ci/7293/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-vivid-touch/6482
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-xenial-touch/708/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-vivid/1998
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-qmluitest-xenial-amd64/701
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-vivid-amd64-ci/1893
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-vivid-i386-ci/1893
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-xenial-amd64-ci/700
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-xenial-i386-ci/699
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-vivid-touch/4956
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/6493
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/6493/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/27608
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-xenial-touch/365/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-xenial-armhf/706
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-xenial-armhf/706/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/27607

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

review: Needs Fixing (continuous-integration)
Revision history for this message
Daniel d'Andrada (dandrader) wrote :

Please update the mock SurfaceManager.

review: Needs Fixing
Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Daniel d'Andrada (dandrader) wrote :

There's a conflict with the new trunk:

"""
Text conflict in tests/mocks/Unity/Application/SurfaceManager.h
1 conflicts encountered.
"""

review: Needs Fixing
Revision history for this message
Daniel d'Andrada (dandrader) wrote :

And please, in the SurfaceManager mock, also set m_virtualKeyboard back to nullptr when it's destroyed.

Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

FAILED: Continuous integration, rev:2185
https://unity8-jenkins.ubuntu.com/job/lp-unity8-1-ci/370/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build/506
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=vivid+overlay/153
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=xenial/153
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/529
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/547
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial/547
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/543
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/543/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial/543
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial/543/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/543
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/543/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial/543
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial/543/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/543
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/543/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial/543
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial/543/artifact/output/*zip*/output.zip

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

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

FAILED: Continuous integration, rev:2185
http://jenkins.qa.ubuntu.com/job/unity8-ci/7306/
Executed test runs:
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-vivid-touch/6497
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-xenial-touch/721/console
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-vivid/2011
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-qmluitest-xenial-amd64/714
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-vivid-amd64-ci/1906
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-vivid-i386-ci/1906
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-xenial-amd64-ci/713
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-xenial-i386-ci/712
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-vivid-touch/4961
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/6508
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/6508/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/27626
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-xenial-touch/370/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-xenial-armhf/719
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-xenial-armhf/719/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/27627

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

review: Needs Fixing (continuous-integration)
Revision history for this message
Daniel d'Andrada (dandrader) 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.
Mostly yeah. There's one failure I suspect will be fixed by lp:~dandrader/unity8/ownMockMirSurface

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

review: Approve
Revision history for this message
Lukáš Tinkl (lukas-kde) wrote :

It fixes the issue with missing OSK in the OOBE wizard too, yay

review: Approve
Revision history for this message
Michał Sawicz (saviq) wrote :

Unfortunately the test still is not stable:

FAIL! : qmltestrunner::Shell::test_showInputMethod() function returned unexpected result
   Actual (): false
   Expected (): true
   Loc: [/tmp/adt-run.dd2sFU/build.VbD/unity8-8.11+15.04.20160229/tests/qmltests/tst_Shell.qml(788)]

This is from a failed adt-run in britney:
https://objectstorage.prodstack4-5.canonical.com/v1/AUTH_77e2ada1e7a84929a74ba3b87153c0ac/autopkgtest-vivid-ci-train-ppa-service-landing-064/vivid/amd64/u/unity8/20160229_200055@/log.gz

review: Needs Fixing
Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

FAILED: Continuous integration, rev:2186
https://unity8-jenkins.ubuntu.com/job/lp-unity8-1-ci/540/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build/706
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/729
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/747
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial/747
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/743
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/743/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial/743
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial/743/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/743
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/743/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial/743
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial/743/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/743
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/743/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial/743
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial/743/artifact/output/*zip*/output.zip

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

review: Needs Fixing (continuous-integration)
Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

FAILED: Continuous integration, rev:2186
https://unity8-jenkins.ubuntu.com/job/lp-unity8-1-ci/543/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build/709
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/732
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/749
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial/749
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/745
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/745/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial/745
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial/745/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/745
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/745/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial/745
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial/745/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/745
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/745/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial/745
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial/745/artifact/output/*zip*/output.zip

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

review: Needs Fixing (continuous-integration)
lp:~mzanetti/unity8/catch-osk updated
2187. By Michael Zanetti

property parent the OSK surface in mocks

2188. By Michael Zanetti

register MirSurfaceInterface as a qml datatype

Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

FAILED: Continuous integration, rev:2188
https://unity8-jenkins.ubuntu.com/job/lp-unity8-1-ci/558/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/749
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/767
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial/767
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/763
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/763/artifact/output/*zip*/output.zip
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial/763/console
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/763
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/763/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial/763
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial/763/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/763
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/763/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial/763
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial/763/artifact/output/*zip*/output.zip

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

review: Needs Fixing (continuous-integration)
lp:~mzanetti/unity8/catch-osk updated
2189. By Michael Zanetti

don't do it properly... too late for that :(

Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

PASSED: Continuous integration, rev:2189
https://unity8-jenkins.ubuntu.com/job/lp-unity8-1-ci/567/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=vivid+overlay,testname=qmluitests.sh/319
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=xenial,testname=qmluitests.sh/319
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=phone-armhf,release=vivid+overlay,testname=autopilot.sh/319
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/758
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/776
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial/776
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/772
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/772/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial/772
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial/772/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/772
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/772/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial/772
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial/772/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/772
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/772/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial/772
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial/772/artifact/output/*zip*/output.zip

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

review: Approve (continuous-integration)
Revision history for this message
Michał Sawicz (saviq) :
review: Approve (ci)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'qml/Components/InputMethod.qml'
--- qml/Components/InputMethod.qml 2015-12-02 10:34:31 +0000
+++ qml/Components/InputMethod.qml 2016-03-02 23:42:46 +0000
@@ -22,15 +22,6 @@
22Item {22Item {
23 id: root23 id: root
2424
25 Connections {
26 target: SurfaceManager
27 onSurfaceCreated: {
28 if (surface.type == Mir.InputMethodType) {
29 surfaceItem.surface = surface;
30 }
31 }
32 }
33
34 property int transitionDuration: UbuntuAnimation.FastDuration25 property int transitionDuration: UbuntuAnimation.FastDuration
3526
36 MirSurfaceItem {27 MirSurfaceItem {
@@ -41,6 +32,7 @@
4132
42 surfaceWidth: width33 surfaceWidth: width
43 surfaceHeight: height34 surfaceHeight: height
35 surface: SurfaceManager.inputMethodSurface
4436
45 onLiveChanged: {37 onLiveChanged: {
46 if (surface !== null && !live) {38 if (surface !== null && !live) {
4739
=== modified file 'tests/mocks/Unity/Application/MirSurface.cpp'
--- tests/mocks/Unity/Application/MirSurface.cpp 2015-11-30 18:25:47 +0000
+++ tests/mocks/Unity/Application/MirSurface.cpp 2016-03-02 23:42:46 +0000
@@ -22,8 +22,9 @@
22 Mir::Type type,22 Mir::Type type,
23 Mir::State state,23 Mir::State state,
24 const QUrl& screenshot,24 const QUrl& screenshot,
25 const QUrl &qmlFilePath)25 const QUrl &qmlFilePath,
26 : unity::shell::application::MirSurfaceInterface(nullptr)26 QObject *parent)
27 : unity::shell::application::MirSurfaceInterface(parent)
27 , m_name(name)28 , m_name(name)
28 , m_type(type)29 , m_type(type)
29 , m_state(state)30 , m_state(state)
3031
=== modified file 'tests/mocks/Unity/Application/MirSurface.h'
--- tests/mocks/Unity/Application/MirSurface.h 2015-11-30 18:25:47 +0000
+++ tests/mocks/Unity/Application/MirSurface.h 2016-03-02 23:42:46 +0000
@@ -41,7 +41,8 @@
41 Mir::Type type,41 Mir::Type type,
42 Mir::State state,42 Mir::State state,
43 const QUrl& screenshot,43 const QUrl& screenshot,
44 const QUrl &qmlFilePath = QUrl());44 const QUrl &qmlFilePath = QUrl(),
45 QObject *parent = nullptr);
45 virtual ~MirSurface();46 virtual ~MirSurface();
4647
47 ////48 ////
4849
=== modified file 'tests/mocks/Unity/Application/SurfaceManager.cpp'
--- tests/mocks/Unity/Application/SurfaceManager.cpp 2015-11-30 18:25:47 +0000
+++ tests/mocks/Unity/Application/SurfaceManager.cpp 2016-03-02 23:42:46 +0000
@@ -34,6 +34,13 @@
34 QObject(parent)34 QObject(parent)
35 , m_virtualKeyboard(nullptr)35 , m_virtualKeyboard(nullptr)
36{36{
37 m_virtualKeyboard = new VirtualKeyboard(this);
38 connect(m_virtualKeyboard, &QObject::destroyed, this, [this](QObject *obj) {
39 MirSurface* surface = qobject_cast<MirSurface*>(obj);
40 m_virtualKeyboard = nullptr;
41 Q_EMIT inputMethodSurfaceChanged();
42 Q_EMIT surfaceDestroyed(surface);
43 });
37}44}
3845
39MirSurface *SurfaceManager::createSurface(const QString& name,46MirSurface *SurfaceManager::createSurface(const QString& name,
@@ -58,16 +65,8 @@
58 return surface;65 return surface;
59}66}
6067
61MirSurface *SurfaceManager::inputMethodSurface()68MirSurface *SurfaceManager::inputMethodSurface() const
62{69{
63 if (!m_virtualKeyboard) {
64 m_virtualKeyboard = new VirtualKeyboard;
65 connect(m_virtualKeyboard, &QObject::destroyed, this, [this](QObject *obj) {
66 MirSurface* surface = qobject_cast<MirSurface*>(obj);
67 Q_EMIT surfaceDestroyed(surface);
68 });
69 Q_EMIT surfaceCreated(m_virtualKeyboard);
70 }
71 return m_virtualKeyboard;70 return m_virtualKeyboard;
72}71}
7372
7473
=== modified file 'tests/mocks/Unity/Application/SurfaceManager.h'
--- tests/mocks/Unity/Application/SurfaceManager.h 2015-11-30 18:25:47 +0000
+++ tests/mocks/Unity/Application/SurfaceManager.h 2016-03-02 23:42:46 +0000
@@ -25,7 +25,7 @@
25class SurfaceManager : public QObject25class SurfaceManager : public QObject
26{26{
27 Q_OBJECT27 Q_OBJECT
2828 Q_PROPERTY(MirSurface* inputMethodSurface READ inputMethodSurface NOTIFY inputMethodSurfaceChanged)
29 Q_PROPERTY(int newSurfaceMinimumWidth READ newSurfaceMinimumWidth WRITE setNewSurfaceMinimumWidth NOTIFY newSurfaceMinimumWidthChanged)29 Q_PROPERTY(int newSurfaceMinimumWidth READ newSurfaceMinimumWidth WRITE setNewSurfaceMinimumWidth NOTIFY newSurfaceMinimumWidthChanged)
30 Q_PROPERTY(int newSurfaceMaximumWidth READ newSurfaceMaximumWidth WRITE setNewSurfaceMaximumWidth NOTIFY newSurfaceMaximumWidthChanged)30 Q_PROPERTY(int newSurfaceMaximumWidth READ newSurfaceMaximumWidth WRITE setNewSurfaceMaximumWidth NOTIFY newSurfaceMaximumWidthChanged)
31 Q_PROPERTY(int newSurfaceMinimumHeight READ newSurfaceMinimumHeight WRITE setNewSurfaceMinimumHeight NOTIFY newSurfaceMinimumHeightChanged)31 Q_PROPERTY(int newSurfaceMinimumHeight READ newSurfaceMinimumHeight WRITE setNewSurfaceMinimumHeight NOTIFY newSurfaceMinimumHeightChanged)
@@ -43,8 +43,7 @@
43 Mir::State state,43 Mir::State state,
44 const QUrl& screenshot);44 const QUrl& screenshot);
4545
46 // To be used in the tests46 MirSurface* inputMethodSurface() const;
47 Q_INVOKABLE MirSurface* inputMethodSurface();
4847
49 int newSurfaceMinimumWidth() const { return m_newSurfaceMinimumWidth; }48 int newSurfaceMinimumWidth() const { return m_newSurfaceMinimumWidth; }
50 void setNewSurfaceMinimumWidth(int value);49 void setNewSurfaceMinimumWidth(int value);
@@ -65,6 +64,7 @@
65 void setNewSurfaceHeightIncrement(int);64 void setNewSurfaceHeightIncrement(int);
6665
67Q_SIGNALS:66Q_SIGNALS:
67 void inputMethodSurfaceChanged();
68 void countChanged();68 void countChanged();
69 void surfaceCreated(MirSurface *surface);69 void surfaceCreated(MirSurface *surface);
70 void surfaceDestroyed(MirSurface*surface);70 void surfaceDestroyed(MirSurface*surface);
7171
=== modified file 'tests/mocks/Unity/Application/VirtualKeyboard.cpp'
--- tests/mocks/Unity/Application/VirtualKeyboard.cpp 2015-09-02 10:35:16 +0000
+++ tests/mocks/Unity/Application/VirtualKeyboard.cpp 2016-03-02 23:42:46 +0000
@@ -22,12 +22,13 @@
2222
23#include <QDebug>23#include <QDebug>
2424
25VirtualKeyboard::VirtualKeyboard()25VirtualKeyboard::VirtualKeyboard(QObject *parent)
26 : MirSurface("input-method",26 : MirSurface("input-method",
27 Mir::InputMethodType,27 Mir::InputMethodType,
28 Mir::MinimizedState,28 Mir::MinimizedState,
29 QUrl("qrc:///Unity/Application/vkb_portrait.png"),29 QUrl("qrc:///Unity/Application/vkb_portrait.png"),
30 QUrl("qrc:///Unity/Application/VirtualKeyboard.qml"))30 QUrl("qrc:///Unity/Application/VirtualKeyboard.qml"),
31 parent)
31{32{
32}33}
3334
3435
=== modified file 'tests/mocks/Unity/Application/VirtualKeyboard.h'
--- tests/mocks/Unity/Application/VirtualKeyboard.h 2015-08-19 20:30:09 +0000
+++ tests/mocks/Unity/Application/VirtualKeyboard.h 2016-03-02 23:42:46 +0000
@@ -25,7 +25,7 @@
25{25{
26 Q_OBJECT26 Q_OBJECT
27public:27public:
28 explicit VirtualKeyboard();28 explicit VirtualKeyboard(QObject *parent = nullptr);
29 ~VirtualKeyboard();29 ~VirtualKeyboard();
30};30};
3131
3232
=== modified file 'tests/mocks/Unity/Application/plugin.cpp'
--- tests/mocks/Unity/Application/plugin.cpp 2015-08-03 15:00:47 +0000
+++ tests/mocks/Unity/Application/plugin.cpp 2016-03-02 23:42:46 +0000
@@ -78,8 +78,9 @@
78 qmlRegisterUncreatableType<unity::shell::application::ApplicationInfoInterface>(uri, 0, 1, "ApplicationInfoInterface", "Abstract interface. Cannot be created in QML");78 qmlRegisterUncreatableType<unity::shell::application::ApplicationInfoInterface>(uri, 0, 1, "ApplicationInfoInterface", "Abstract interface. Cannot be created in QML");
79 qmlRegisterUncreatableType<Session>(uri, 0, 1, "Session", "Session can't be instantiated from QML");79 qmlRegisterUncreatableType<Session>(uri, 0, 1, "Session", "Session can't be instantiated from QML");
80 qmlRegisterUncreatableType<MirSurface>(uri, 0, 1, "MirSurface", "MirSurface can't be instantiated from QML");80 qmlRegisterUncreatableType<MirSurface>(uri, 0, 1, "MirSurface", "MirSurface can't be instantiated from QML");
81 qmlRegisterUncreatableType<unity::shell::application::MirSurfaceInterface>(
82 uri, 0, 1, "MirSurface", "MirSurface can't be instantiated from QML");
81 qmlRegisterType<MirSurfaceItem>(uri, 0, 1, "MirSurfaceItem");83 qmlRegisterType<MirSurfaceItem>(uri, 0, 1, "MirSurfaceItem");
82
83 qmlRegisterType<ApplicationInfo>(uri, 0, 1, "ApplicationInfo");84 qmlRegisterType<ApplicationInfo>(uri, 0, 1, "ApplicationInfo");
8485
85 qmlRegisterSingletonType<ApplicationManager>(uri, 0, 1, "ApplicationManager", applicationManagerSingleton);86 qmlRegisterSingletonType<ApplicationManager>(uri, 0, 1, "ApplicationManager", applicationManagerSingleton);
8687
=== modified file 'tests/qmltests/tst_OrientedShell.qml'
--- tests/qmltests/tst_OrientedShell.qml 2016-01-28 11:31:48 +0000
+++ tests/qmltests/tst_OrientedShell.qml 2016-03-02 23:42:46 +0000
@@ -314,7 +314,7 @@
314 checked: false314 checked: false
315 activeFocusOnPress: false315 activeFocusOnPress: false
316 onCheckedChanged: {316 onCheckedChanged: {
317 var surface = SurfaceManager.inputMethodSurface();317 var surface = SurfaceManager.inputMethodSurface;
318 if (checked) {318 if (checked) {
319 surface.setState(Mir.RestoredState);319 surface.setState(Mir.RestoredState);
320 } else {320 } else {
321321
=== modified file 'tests/qmltests/tst_Shell.qml'
--- tests/qmltests/tst_Shell.qml 2016-02-12 00:11:28 +0000
+++ tests/qmltests/tst_Shell.qml 2016-03-02 23:42:46 +0000
@@ -773,7 +773,7 @@
773 loadShell("phone");773 loadShell("phone");
774 swipeAwayGreeter();774 swipeAwayGreeter();
775 var item = findChild(shell, "inputMethod");775 var item = findChild(shell, "inputMethod");
776 var surface = SurfaceManager.inputMethodSurface();776 var surface = SurfaceManager.inputMethodSurface;
777777
778 surface.setState(Mir.MinimizedState);778 surface.setState(Mir.MinimizedState);
779 tryCompare(item, "visible", false);779 tryCompare(item, "visible", false);

Subscribers

People subscribed via source and target branches