Bug 51198

Summary:	[Qt][Symbian] Regression after fix for bug 49787 (QtWebKit does not respond properly to QInputMethodEvents with replacement text in them)
Product:	WebKit	Reporter:	Ademar Reis <ademar>
Component:	WebKit Qt	Assignee:	Nobody <webkit-unassigned>
Status:	RESOLVED WORKSFORME
Severity:	Normal	CC:	benjamin, graham.henderson, koshuin, kristian.amlie, laszlo.gombos, menard
Priority:	P1	Keywords:	Qt, Regression
Version:	528+ (Nightly build)
Hardware:	S60 Hardware
OS:	All

Description Ademar Reis 2010-12-16 10:45:48 PST

From bug 49787:

"""
-- Comment #22 From Janne Koskinen 2010-12-02 05:08:29 PST (-) [reply] -------

> Pushed to qtwebkit-2.1 as http://gitorious.org/webkit/qtwebkit/commit/85a22e12280f290c4eb0ccb4aec2ec9f13ffb7af

Unfortunately this commit breaks text inputting on QtWebkit2.1 (least) on Symbian. Each character entered gets deleted right after entering. Commit needs to be reverted and backport redone and tested.
"""

The patch has landed on trunk, so the regression is *probably* there as well. I'm opening this ticket so that we can keep track of it (a fix has been added to qtwebkit-2.1, see bug 49787)

Comment 1 Janne Koskinen 2011-05-09 05:44:00 PDT

Hi, got contacted and asked about this issue.
I can't reproduce this with QtWebkit2.1.x anymore. IIRC Kristian did change somehing in Qt after that fix. Feel free to close if you can't reproduce this either.

Comment 2 Ademar Reis 2011-05-09 05:49:57 PDT

(In reply to comment #1)
> I can't reproduce this with QtWebkit2.1.x anymore. IIRC Kristian did change somehing in Qt after that fix. Feel free to close if you can't reproduce this either.

It was fixed in 2.1.x (Bug 49787), this one is about trunk.

Comment 3 Alexis Menard (darktears) 2011-05-26 09:08:35 PDT

What is the status with that bug??? Is the last attachment of https://bugs.webkit.org/show_bug.cgi?id=49787 just need a rebase?

Comment 4 Ademar Reis 2011-06-03 11:05:18 PDT

(In reply to comment #3)
> What is the status with that bug??? Is the last attachment of https://bugs.webkit.org/show_bug.cgi?id=49787 just need a rebase?

Janne: do you have a 2.2 or trunk environment there where you can test for this bug?

Comment 5 Ademar Reis 2011-08-05 13:18:58 PDT

I have the impression that this problem doesn't affect trunk or qtwebkit-2.2, but I don't have a symbian build to test and my phones are not included in the symbian buildbot (and therefore I can't install the .sis files generated there).

The testcase appears to be very simple: install qtwebkit and try to input something in a web form (for example, inside QtTestBrowser). This bug is valid if each character entered gets deleted right after being entered (see comment 1).

Comment 6 Ademar Reis 2011-09-12 14:34:26 PDT

Graham Henderson from Nokia/London tested this and couldn't reproduce it (although he couldn't find a physical keyboard).

Removing it from the blockers list of QtWebKit-2.2. It's been a long journey to find someone to test this... If this bug is real (and the problem is absolutely serious and easy to spot if it is), it will be reproduced soon. Either that or noone is using/testing Symbian, in which case this bug is not important anyway.

Comment 7 graham.henderson 2011-09-13 06:45:41 PDT

Tested Aug12th Symbian/ARM build of webkit2.2 on Nokia MCL wk35, text entry into various web forms using QtTestBrowser appears to work correctly. Tried USB keyboard, cut/paste of text, virtual keyboard text entry

Comment 8 Ademar Reis 2011-09-13 06:54:31 PDT

(In reply to comment #7)
> Tested Aug12th Symbian/ARM build of webkit2.2 on Nokia MCL wk35, text entry into various web forms using QtTestBrowser appears to work correctly. Tried USB keyboard, cut/paste of text, virtual keyboard text entry

Thanks a lot! Closing as WORKSFORME, finally. :-)