Bug 44761 – [Qt] tst_QWebView::focusInputTypes() fails on MeeGo Handset with WebKit 2.1

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Benjamin Poulain]

The test tst_QWebView::focusInputTypes() is not reliable on Meego Handset. Output of the bot:

FAIL! : tst_QWebView::focusInputTypes() 'webView->inputMethodHints() == Qt::ImhHiddenText' returned FALSE. ()

[Benjamin Poulain]

Created attachment 65703 [details]
Make the test more robust

On my hardware, this fixes the problems with webView.inputMethodHints().

However, I still have random failures out of QVERIFY(webView.testAttribute(Qt::WA_InputMethodEnabled)). Those cannot be reproduced in gdb so I suspect a race condition. The test bot has a different environment, we'll see if this patch fixes the failure.

[Antonio Gomes]

Comment on attachment 65703 [details]
Make the test more robust

Good work!

[WebKit Commit Bot]

Comment on attachment 65703 [details]
Make the test more robust

Clearing flags on attachment: 65703

Committed r66298: <http://trac.webkit.org/changeset/66298>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Ademar Reis]

Revision r66298 cherry-picked into qtwebkit-2.1 with commit 7c8ea628dd2c7d5c05e1682059d6178b2f8bc7e6

[Benjamin Poulain]

Created attachment 68911 [details]
Improve the test further

The test still fails on the test bots for an unknown reason. I hope to get more info from the output with this patch while still improving the test.

Using QCOMPARE() instead of QVERIFY() will output the return value so will help identifying the issue. Adding the "hasFocus()" on each element will help identifying if the element got the focus at all on the device.