Created attachment 76739 [details] Screenshots Most buttons, panels, and frames are rendering incorrectly. The vertical sides of the elements are not lining up with the corners. The vertical borders shift 1px to right for all elements, including the list panel. See the attached screen shots. Steps to reproduce the issue for a simple button: ------------------------ <html><body> <input type="button" value="Border Test Button"/> </body><html> ------------------------ 1. Save above html code in a file (say "e:/a.html"). 2. Open QtTestBrowser. 3. Enter "e:/a.html" in the address bar and press refresh button. Actual Result: HTML page with a button is displayed. Border of the button is not aligned properly. Additional Info: Qt - 4.7 QtWebkit - 2.1 OS - Sumbian^3 Issue has been moved from wrt team Boston defect 7315.
Related QtWRT Bugzilla error ID: http://bugs.nokia-boston.com/bugzilla/show_bug.cgi?id=7315
Fix required for Qtwebkit 2.2
Please follow http://trac.webkit.org/wiki/QtWebKitBugs when reporing bug here (missing the keyword Qt). (In reply to comment #2) > Fix required for Qtwebkit 2.2 Is this seriously a blocker for 2.1.x? Are you sure it is a bug in WebKit at all? I would think this is part of common code and the bug is in Qt.
> Is this seriously a blocker for 2.1.x? Are you sure it is a bug in WebKit at all? I would think this is part of common code and the bug is in Qt. This issue happens with mobile webkit style only if OpenVG backend is used, raster engine is fine. AFAIK Qt buttons don't have this issue.
I spent some time on this issue. It seems to happen only with Qt mobile theme enabled as Janne has pointed out. If we disable Qt mobile style, then everything seems to be ok. I think the themeQtMobile.css is not liked by symbian.
(In reply to comment #5) > I spent some time on this issue. It seems to happen only with Qt mobile theme enabled as Janne has pointed out. If we disable Qt mobile style, then everything seems to be ok. I think the themeQtMobile.css is not liked by symbian. That already some info... Maybe a property used by the theme is not well supported on Symbian. Have you tried with the raster graphic system and the mobile theme?
(In reply to comment #6) > (In reply to comment #5) > > I spent some time on this issue. It seems to happen only with Qt mobile theme enabled as Janne has pointed out. If we disable Qt mobile style, then everything seems to be ok. I think the themeQtMobile.css is not liked by symbian. > That already some info... > Maybe a property used by the theme is not well supported on Symbian. > Have you tried with the raster graphic system and the mobile theme? Raster graphic system is working fine.
(In reply to comment #7) > Raster graphic system is working fine. Could you make a reduction? That way you could create a bug for Qt and close the bug here.
This is actually bug/feature in the OpenVG hardware. Unfortunately we need to workaround this in QtWebkit. I'm preparing a patch.
Created attachment 81994 [details] patch for QtWebkit2.1 branch OpenVG backend cannot guarantee pixel rendering to be correct in non-integer boundaries unless antialising is used. Patch checks if the backend is OpenVg and forces lines to be drawn.
Adding to Qtwebkit 2.1 Master bug for cherry picking the fix to Qtwebkit 2.1
(In reply to comment #10) > Patch checks if the backend is OpenVg and forces lines to be drawn antialiased. Getting late again :)
Created attachment 81998 [details] Patch proposal for trunk There is no layout test for this as it would require pixel per pixel analysis of render.
Comment on attachment 81998 [details] Patch proposal for trunk View in context: https://bugs.webkit.org/attachment.cgi?id=81998&action=review > Source/WebCore/ChangeLog:12 > + No new tests. (OOPS!) You should remove that line.
Looks good to me, please fix ChangeLog.
Created attachment 82112 [details] patch proposal for trunk - changelog fixed
Created attachment 82114 [details] patch for QtWebkit2.1 branch - log fixed
Comment on attachment 82112 [details] patch proposal for trunk - changelog fixed r=me
Comment on attachment 82112 [details] patch proposal for trunk - changelog fixed Clearing flags on attachment: 82112 Committed r78332: <http://trac.webkit.org/changeset/78332>
All reviewed patches have been landed. Closing bug.
The commit-queue encountered the following flaky tests while processing attachment 82112 [details]: http/tests/websocket/tests/multiple-connections.html bug 53825 (author: abarth@webkit.org) The commit-queue is continuing to process your patch.
Revision r78332 cherry-picked into qtwebkit-2.1 with commit 30058fe <http://gitorious.org/webkit/qtwebkit/commit/30058fe>