Bug 43043
Summary: | [Qt] Issue with page display using QtWebKit 2.1 | ||
---|---|---|---|
Product: | WebKit | Reporter: | nokiabugz |
Component: | Layout and Rendering | Assignee: | Jay Tucker <jay.tucker> |
Status: | RESOLVED FIXED | ||
Severity: | Critical | CC: | ademar, benjamin, ext-prasad.adimulam, hausmann, jturcotte, markus, ostap73, suresh.voruganti, timothy.lu |
Priority: | P2 | Keywords: | PlatformOnly, Qt |
Version: | 528+ (Nightly build) | ||
Hardware: | S60 Hardware | ||
OS: | S60 3rd edition |
nokiabugz
STEPS TO REPRODUCE:
Installed webkit 2.1 on the device.
Launch the browser and load "http://iop1.nokia-boston.com/index.html".
Select the link 'Manual Testing Page' under 'Load Time Performance'.
ACTUAL RESULTS:
The page is loading with a "Black background and the urls are blue in color"
EXPECTED RESULTS:
The page should load with a white background.
Problem occurs on both Nokia browser and QtTestBrowser with WebKit 2.1 branch
Attachments | ||
---|---|---|
Add attachment proposed patch, testcase, etc. |
Timothy Lu
This is critical issue that is blocking Webkit 2.1 testing in Browser.
Simon Hausmann
Which Qt version are you using?
Suresh Voruganti
I think this issue is solved by getting latest Qt package.
Suresh Voruganti
This is not reproducible any more with latest Qt version. so resolving the bug.
Suresh Voruganti
I think still issue is seen interminently.
Does any one know which error from Qt has fixed this issue?
Jocelyn Turcotte
(In reply to comment #5)
I don't know how Qt handles platform color palettes on Symbian but this feels like bug #22123 to me.
Did you try changing the phone's theme to see if it changes the behavior?
Suresh Voruganti
Re opening the issue as it is blocking the testing.
Suresh Voruganti
Kennery, can you pls reply to comment 6?
Viatcheslav Ostapenko
(In reply to comment #6)
> (In reply to comment #5)
>
> I don't know how Qt handles platform color palettes on Symbian
Yes, it's theme issue.
There is workaround for this in QGraphicsWebView based classes:
QPalette pal = palette();
pal.setColor(QPalette::ButtonText, Qt::black);
pal.setColor(QPalette::Base, Qt::white);
setPalette(pal);
page()->setPalette(pal);
This also fixes the problem of white radio/checkbox buttons in case of phone themes with dark background.
> but this feels like bug #22123 to me.
Not exactly.
> Did you try changing the phone's theme to see if it changes the behavior?
Previously I've tried to figure out, should QGraphicsWebView by default pick up theme colors. Those colors can conflict with web colors used by most web pages. I've got different opinions about this.
Markus Goetz
It's a workaround for the nokia browser but no solution.
After a lot of discussion here in Oslo we saw that this needs a deeper fix in QtWebKit. More will follow..
Suresh Voruganti
Assigning the error to Jay as he is working on the issue together with others in the code camp
Jay Tucker
Worked together with Tor Arne on a patch that provides a principled solution to this issue. Need to finish patch in next day or two.
Suresh Voruganti
Removing the blocks 39121 as the fix is not required for Qtwebkit 2.1. Original issue reported by browser team is not reproducible, so removing the 39121
Benjamin Poulain
(In reply to comment #12)
> Need to finish patch in next day or two.
Any update? Should this task be closed?
Ademar Reis
(In reply to comment #14)
> (In reply to comment #12)
> > Need to finish patch in next day or two.
>
> Any update? Should this task be closed?
I'm removing the block for bug 44677, as 2.1 appears to not be affected anymore
(see comment #13) and nobody appears to be working on it anyway.
Jay: Please close this bug or update its status.
Jay Tucker
Per the comments above, this issue has been resolved.