47273 2010-10-06 09:19:59 -0700 [Qt][S60] Junk text is shown in textbox when text is highlighted 2010-11-01 06:37:22 -0700 1 1 1 Unclassified WebKit WebKit Qt 528+ (Nightly build) S60 Hardware S60 3rd edition RESOLVED DUPLICATE 39625 P2 Major --- 39625 0 ankur.baranwal webkit-qt-unassigned amit.rai ankur.baranwal kling neelkanth.nidgundi pragya.kaundal ramesh.hegde robert suresh.voruganti oldest_to_newest 290352 0 ankur.baranwal 2010-10-06 09:19:59 -0700 Qt: 4.7 QtWebkit: 2.1 OS: Symbian^3 Steps to reproduce the error - 1. Open www.google.com in Anomaly Browser. 2. Select the search text box. Virtual keypad opens. 3. Enter "hello" and highlight the word. 4. Now press ok button on virtual keypad. Expected Result - After step 4, "hello" word is displayed in text box. Actual Result - After step 4, "hellohello" is displayed in the text box. Additional Info - This bug is not reproducible on native applications and in sample Qt application. 291571 1 ramesh.hegde 2010-10-08 05:18:21 -0700 This is a Blocker issue for us.Can you raise this priority to P0-Blocker and work on it? 292496 2 suresh.voruganti 2010-10-11 08:38:55 -0700 This error is moved from wrt team Boston Defect -6537 and the analysis in 6537 doesnt prove that it is Qtwebkit error. Wrt team need to analyse and prove that it is Qtwebkit error, so ignoring the error as now. Re-open the error after analysis if the problem seems to be in QtWebkit. 294019 3 ankur.baranwal 2010-10-14 02:00:33 -0700 This bug is reproducible using QtTestBrowser. I debugged this further and found that following piece of code which, it seems, was introduced in Webkit2.1. Whenever control passes through this code, text is getting repeated. void QWebPagePrivate::inputMethodEvent(QInputMethodEvent *ev) { ... ... #if QT_VERSION >= 0x040600 if (hasSelection) { QString text = (renderTextControl) ? QString(renderTextControl->text()) : QString(); if (preedit.isEmpty() && selection.start + selection.length > 0) preedit = text; editor->setComposition(preedit, underlines, (selection.length < 0) ? selection.start + selection.length : selection.start, (selection.length < 0) ? selection.start : selection.start + selection.length); } else #endif ... ... } It seems text is getting appended to older string every time in QString(renderTextControl->text()). Also, this happens even if we simply click on empty area in the virtual keypad. This behavior is not seen with the Address bar text field. It happens only within the html text field. 294091 4 kling 2010-10-14 07:04:40 -0700 CC'ing Robert since he introduced this code block in <http://trac.webkit.org/changeset/58218> 294103 5 suresh.voruganti 2010-10-14 07:20:05 -0700 Adding dependency to 39121 as it is top issue for services. 294211 6 robert 2010-10-14 11:15:56 -0700 (In reply to comment #4) > CC'ing Robert since he introduced this code block in <http://trac.webkit.org/changeset/58218> What we need to do is outlined by Simon in: https://bugs.webkit.org/show_bug.cgi?id=39625#c15 Unfortunately I never got round to implementing it. I can look at it this weekend, but if the priority needs to be higher then someone should do it before me. 295230 7 robert 2010-10-17 10:10:30 -0700 There is a patch up for review at: https://bugs.webkit.org/show_bug.cgi?id=39625 Let me know if it works for you. Also, as someone who only has the unit tests to go on, can you provide a unit test for this case similar to those in tst_qwebpage? 295399 8 neelkanth.nidgundi 2010-10-18 01:36:35 -0700 Please provide the LE? 295428 9 kling 2010-10-18 03:41:15 -0700 (In reply to comment #8) > Please provide the LE? Please refrain from using Nokia jargon on public issue trackers. 298112 10 robert 2010-10-22 12:36:52 -0700 Hi Ankur, Is this fixed for you in trunk now? 302282 11 ankur.baranwal 2010-10-30 04:05:30 -0700 Hi Robert, the problem is fixed in trunk now. Thanks. 302638 12 suresh.voruganti 2010-11-01 06:36:15 -0700 Duplicating this error to 39625 as the patch for 39625 fixed this issue. *** This bug has been marked as a duplicate of bug 39625 ***