Merge lp:~michael-sheldon/ubuntu-keyboard/fix-1436076 into lp:ubuntu-keyboard

[Michael Sheldon]

Avoid reporting false visibility signal during startup

Avoid reporting false visibility signal during startup

[Michael Sheldon (michael-sheldon)]

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

 * No

Is your branch in sync with latest trunk (e.g. bzr pull lp:trunk -> no changes)

 * Yes

Did you perform an exploratory manual test run of your code change and any related functionality on device or emulator?

 * Yes

Did you successfully run all tests found in your component's Test Plan (https://wiki.ubuntu.com/Process/Merges/TestPlan/ubuntu-keyboard) on device or emulator?

 * Yes

If you changed the UI, was the change specified/approved by design?

 * No change

If you changed UI labels, did you update the pot file?

 * No change

If you changed the packaging (debian), did you add a core-dev as a reviewer to this MP?

 * No change

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:345
http://jenkins.qa.ubuntu.com/job/ubuntu-keyboard-ci/575/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-vivid-touch/1997
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-keyboard-vivid-amd64-ci/76
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-keyboard-vivid-armhf-ci/76
        deb: http://jenkins.qa.ubuntu.com/job/ubuntu-keyboard-vivid-armhf-ci/76/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-keyboard-vivid-i386-ci/76
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-vivid-mako/1758
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/1995
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/1995/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/19207

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

review: Approve (continuous-integration)

=== modified file 'qml/Keyboard.qml'
--- qml/Keyboard.qml	2015-02-18 14:31:24 +0000
+++ qml/Keyboard.qml	2015-03-26 17:17:51 +0000
@@ -86,7 +86,7 @@
             property bool extendedKeysShown: false
 
             property bool firstShow: true
-            property bool hidingComplete: true
+            property bool hidingComplete: false
 
             property string layoutId: "freetext"
 

=== modified file 'tests/autopilot/ubuntu_keyboard/tests/test_keyboard.py'
--- tests/autopilot/ubuntu_keyboard/tests/test_keyboard.py	2015-03-16 13:06:42 +0000
+++ tests/autopilot/ubuntu_keyboard/tests/test_keyboard.py	2015-03-26 17:17:51 +0000
@@ -363,6 +363,12 @@
         """The keyboard should only report visibility changes once.
 
         """
+
+        # Test bug #1436076 which only occurs after maliit is restarted
+        subprocess.check_call(['initctl', 'set-env', 'QT_LOAD_TESTABILITY=1'])
+        subprocess.check_call(['restart', 'maliit-server'])
+        sleep(10)
+
         qml = dedent("""
         import QtQuick 2.0
         import Ubuntu.Components 0.1