Created attachment 48439 [details]
Certain elements will highlight with a black background instead of the default.
This can completely mask elements and make them unusable until unselected.
For a simple reproducible example, compile the "Fancy Browser Example" shipped with Qt.
Do a drag selection from the text above and go down.
Certain elements will highlight with black instead of the default.
It's also worth noting that this example works fine in Chrome and Safari.
Created attachment 48617 [details]
This naive patch solves the problem for me.
However, It seems to be indicative of a deeper problem where we have a brush with an invalid color that does not have a style == Qt::NoBrush.
Ah I remember that we had such a similar issue before. Do you remember the details Simon?
I guess this is because of my commit: http://trac.webkit.org/changeset/37421
The optimization does not take into account that the color could be bogus.
The patch from Nick looks good to solve this problem.
> I guess this is because of my commit: http://trac.webkit.org/changeset/37421
False alarm. Excluding the optimization still shows the same behaviour.
Nick: if you include some ChangeLog, then we can land your patch.
Please follow the QtWebKit bug reporting guidelines when reporting bugs.
- The 'QtWebKit' component should only be used for bugs/features in the
public QtWebKit API layer, not to signify that the bug is specific to
the Qt port of WebKit
- Add the keyword 'Qt' to signal that it's a Qt-related bug
Fixed by the commit http://trac.webkit.org/changeset/58949
*** This bug has been marked as a duplicate of bug 38523 ***