OSK does not appear in webapps

Bug #1179937 reported by Timo Jyrinki
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
touch-preview-images
Fix Released
Critical
Timo Jyrinki
qtbase-opensource-src (Ubuntu)
Fix Released
Critical
Thomas Moenicke

Bug Description

Maliit does not appear in webapps, like when launching Facebook app from the home page, when using qtbase 5.0.2 from qt5-proper PPA.

This may be because fix_maliit_activation.patch got dropped and a smaller inputmethod_fix_focusout.patch was added in its place (submitted to upstream).

Current status regarding images (getting the workaround back in): need testers on raring images on the device:
0. have latest phablet-flash, do apt-get dist-upgrade on the device
1. apt-add-repository ppa:canonical-qt5-edgers/qt5-beta2
2. apt-get update
3. apt-get upgrade (check that you get the 5.0.2+dfsg1-3ubuntu1~raring1~test4 version of packages)
4. reboot
5. Test OSK in eg. facebook webapp and elsewhere. Excepted result: works about like earlier (ie. known bugs with not disappearing in some cases, bug #1170674)

Tags: iso-testing
Changed in qtbase-opensource-src (Ubuntu):
assignee: nobody → Timo Jyrinki (timo-jyrinki)
Revision history for this message
Timo Jyrinki (timo-jyrinki) wrote :

While I'm doing an ARM build with fix_maliit_activation.patch (http://is.gd/A3NRTy) instead of inputmethod_fix_focusout.patch (http://is.gd/7xarAF), Thomas can you look at this since you're the author of both patches? They are being applied now on top of Qt 5.0.2, but both cannot be applied at the same time.

The ARM build with fix_maliit_activation.patch will start building at the test PPA https://launchpad.net/~canonical-qt5-edgers/+archive/qt5-beta2/+packages as soon as I've confirmed a local x86 build successful. I'm also doing a local arm build on an arm device.

Changed in qtbase-opensource-src (Ubuntu):
importance: Undecided → Critical
status: New → Confirmed
assignee: Timo Jyrinki (timo-jyrinki) → Thomas Moenicke (thomas-moenicke)
Changed in touch-preview-images:
importance: Undecided → Critical
status: New → Confirmed
Revision history for this message
Thomas Moenicke (thomas-moenicke) wrote :

 fix_maliit_activation.patch addresses issues seen in multiprocess-Webkit, so it could be very well the cause for the trouble.
Why can't they both be applied?

Revision history for this message
Timo Jyrinki (timo-jyrinki) wrote :

I tested a locally built Qt with the fix_maliit_activation.patch and it resolves this bug. https://launchpad.net/~canonical-qt5-edgers/+archive/qt5-beta2/+packages should have the PPA build in a couple of hours, please test from there (5.0.2+dfsg1-3ubuntu1~raring1~test4 ). If others can confirm the fix and we do some other smoke testing as well, I can copy that to qt5-proper PPA. It should match the behavior we had before.

Thomas: They seem to touch the same part of the code and it wasn't obvious to me how to apply the new oneliner after the fix_maliit_activation.patch. If you can wget https://launchpad.net/~canonical-qt5-edgers/+archive/qt5-beta-proper/+files/qtbase-opensource-src_5.0.2%2Bdfsg1.orig.tar.xz , bzr branch lp:~kubuntu-packagers/kubuntu-packaging/qtbase-opensource-src and then do a merge proposal where you modify the inputmethod_fix_focusout.patch (and uncomment it from debian/patches/series) so that bzr builddeb starts fine (all patches applied), that would fix the issue.

Another issue is of course that the fix_maliit_activation.patch was meant to go away at some point, but apparently we can't be without it still and it should be continued to be worked upon to be upstreamed, or a different work done in maliit itself?

description: updated
Changed in touch-preview-images:
assignee: nobody → Timo Jyrinki (timo-jyrinki)
description: updated
description: updated
description: updated
Revision history for this message
Thomas Moenicke (thomas-moenicke) wrote :

I tested it on N10 following the updated instructions and I can see the osk coming up in both FB and browser.
JENKINS_BUILD=123

Revision history for this message
Ubuntu QA Website (ubuntuqa) wrote :

This bug has been reported on the Ubuntu ISO testing tracker.

A list of all reports related to this bug can be found here:
http://iso.qa.ubuntu.com/qatracker/reports/bugs/1179937

tags: added: iso-testing
Revision history for this message
Timo Jyrinki (timo-jyrinki) wrote :

Please someone another still confirm the issue to be fixed with the instructions in the description, I'd like to have two independent confirmations before making a .deb copy.

Revision history for this message
Sergio Schvezov (sergiusens) wrote :

root@ubuntu-phablet:/# apt-get upgrade
Reading package lists... Done
Building dependency tree
Reading state information... Done
The following packages will be upgraded:
  libqt5concurrent5 libqt5core5 libqt5dbus5 libqt5gui5 libqt5network5
  libqt5opengl5 libqt5printsupport5 libqt5sql5 libqt5sql5-sqlite libqt5test5
  libqt5webkit5 libqt5webkit5-qmlwebkitplugin libqt5widgets5 libqt5xml5
  qt5-default qt5-qmake qtbase5-dev qtbase5-dev-tools

Confirming success for facebook and twitter

Revision history for this message
Timo Jyrinki (timo-jyrinki) wrote :

Fix copied to daily-build-next PPA.

Changed in touch-preview-images:
status: Confirmed → Fix Released
Revision history for this message
Timo Jyrinki (timo-jyrinki) wrote :

On another look, I don't find anything unclear anymore in the rebasing of the other patch. The other patch should be covered by other bug reports, so marking this bug completely as fixed.

Changed in qtbase-opensource-src (Ubuntu):
status: Confirmed → Fix Committed
Changed in qtbase-opensource-src (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.