Bug 81699 - qt-4.8.0 : background image applied on HTML table cells is not displayed correctly
Summary: qt-4.8.0 : background image applied on HTML table cells is not displayed corr...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Qt (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Linux
: P2 Major
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-03-20 14:23 PDT by rahmanih
Modified: 2012-04-20 12:12 PDT (History)
3 users (show)

See Also:


Attachments
html page showing the issue. (181.04 KB, application/octet-stream)
2012-03-20 14:24 PDT, rahmanih
no flags Details
screenshot showing the bug with qt-4.8.0 (116.15 KB, image/png)
2012-03-20 14:25 PDT, rahmanih
no flags Details
screenshot showing the correct behaviour with qt-4.7.4 (97.88 KB, image/png)
2012-03-20 14:26 PDT, rahmanih
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description rahmanih 2012-03-20 14:23:19 PDT
Hi,

the issue was met originally found on my Set-top box running qt-4.8.0 on top of directfb.

So I've built both qt-4.8.0 and qt-4.7.4 for x86 enabling the directfb qws plugin.
with the qt-4.8.0 the background image is not displayed correctly but with qt-4.7.4 it is.

have someone met this before?
what is the part of webkit is responsible to manage the background image for HTML tables?

attached are the simple html page reproducing the issue, the screenshot done with the "demos/browser"
run with qt-4.8.0 and qt-4.7.4

regards
Haithem
Comment 1 rahmanih 2012-03-20 14:24:03 PDT
Created attachment 132897 [details]
html page showing the issue.
Comment 2 rahmanih 2012-03-20 14:25:14 PDT
Created attachment 132899 [details]
screenshot  showing the bug with qt-4.8.0
Comment 3 rahmanih 2012-03-20 14:26:00 PDT
Created attachment 132900 [details]
screenshot  showing the correct behaviour with qt-4.7.4
Comment 4 rahmanih 2012-04-17 14:48:09 PDT
Hi,

After debugging, I have found that the issue is in the method:

void Image::drawPattern(GraphicsContext* ctxt, const FloatRect& tileRect, const AffineTransform& patternTransform,
                        const FloatPoint& phase, ColorSpace, CompositeOperator op, const FloatRect& destRect)

in"WebCore/platform/graphics/qt/ImageQt.cpp", because I used the implementation provided in the QtWebKit within qt-4.7.4 and it worked correctly, but I was not able to find why the new implementation fails.

As you have modified that method in the bug#50527, I assumed that you are the right persons to whom I should request help. 

any idea please?

regards
Haithem.
Comment 5 Noam Rosenthal 2012-04-17 15:22:44 PDT
(In reply to comment #4) 
> any idea please?
I haven't touched that code in a long time... the original fix was about scaling the image once and then drawing it several times instead of scaling it for each tile.
Comment 6 rahmanih 2012-04-20 12:12:36 PDT
Hi,
I fixed the issue

details are here:
https://bugzilla.stlinux.com/show_bug.cgi?id=17212

regards
Haithem.