Bug 57546 - [Qt][Symbian] Fix Api test failure -- tst_QWebFrame::inputFieldFocus
Summary: [Qt][Symbian] Fix Api test failure -- tst_QWebFrame::inputFieldFocus
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Qt (show other bugs)
Version: 528+ (Nightly build)
Hardware: S60 Hardware Other
: P2 Normal
Assignee: Yi Shen
URL:
Keywords: Qt, QtTriaged
Depends on:
Blocks: 50925
  Show dependency treegraph
 
Reported: 2011-03-31 07:14 PDT by Yi Shen
Modified: 2011-05-17 06:16 PDT (History)
4 users (show)

See Also:


Attachments
first try (1.76 KB, patch)
2011-03-31 07:55 PDT, Yi Shen
no flags Details | Formatted Diff | Diff
remove symbian tag (1.65 KB, patch)
2011-04-18 08:22 PDT, Yi Shen
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Yi Shen 2011-03-31 07:14:03 PDT
This test requires to count the paint events for the blinking caret in the input field, however, it fails on symbian because the fullscreen size platform Virtual Keyboard shows up when running this test, which covers the web page, and blocks the repaint for the caret in the input field.

One simple solution is to disable the VKB during this test. A patch will be provided soon.
Comment 1 Yi Shen 2011-03-31 07:55:18 PDT
Created attachment 87728 [details]
first try
Comment 2 Andreas Kling 2011-04-08 11:16:19 PDT
Do we have the same problem on Maemo? And does the fix really need to be Symbian-specific?
Comment 3 Yi Shen 2011-04-08 13:39:14 PDT
(In reply to comment #2)
> Do we have the same problem on Maemo? And does the fix really need to be Symbian-specific?

Thanks for the comments, Kling. I am not sure whether Maemo has the same problem (I guess it has, since N900 uses FullScreen VKB). The problem is that I am still struggling to setup a Meego/Maemo environment. Once I have the environment, I can work on the fix for Maemo. I am also thinking about creating a separated bug for Maemo, then people who has the env can test/submit a similar patch quickly.
Comment 4 Laszlo Gombos 2011-04-15 14:14:46 PDT
Can we just ignore the guards and Qt underneath would know which platforms have VKB ?
Comment 5 Yi Shen 2011-04-18 08:22:48 PDT
Created attachment 90040 [details]
remove symbian tag
Comment 6 Laszlo Gombos 2011-04-18 12:32:45 PDT
Comment on attachment 90040 [details]
remove symbian tag

r=me.
Comment 7 WebKit Commit Bot 2011-04-18 14:53:45 PDT
The commit-queue encountered the following flaky tests while processing attachment 90040 [details]:

storage/domstorage/localstorage/storagetracker/storage-tracker-4-create.html bug 58835
storage/domstorage/localstorage/storagetracker/storage-tracker-5-delete-one.html bug 58836
The commit-queue is continuing to process your patch.
Comment 8 WebKit Commit Bot 2011-04-18 14:56:37 PDT
Comment on attachment 90040 [details]
remove symbian tag

Clearing flags on attachment: 90040

Committed r84189: <http://trac.webkit.org/changeset/84189>
Comment 9 WebKit Commit Bot 2011-04-18 14:56:42 PDT
All reviewed patches have been landed.  Closing bug.
Comment 10 WebKit Commit Bot 2011-04-18 16:43:06 PDT
The commit-queue encountered the following flaky tests while processing attachment 90040 [details]:

security/block-test.html bug 55741 (authors: beidson@apple.com, mrowe@apple.com, and sam@webkit.org)
The commit-queue is continuing to process your patch.