[Webkit-unassigned] [Bug 27814] New: Qt Port does not obey color style in button tag with default background

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=27814

           Summary: Qt Port does not obey color style in button tag with
                    default background
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: PC
        OS/Version: All
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P2
         Component: New Bugs
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: mike.fenton at torchmobile.com
                CC: treat at kde.org


In the Qt Port, anytime the theme paints a button object, the buttons style
color property is ignored.

Ie.

<button style="color:red">  Does not draw a button with red text.
<button style="color:red;background-color:blue> Does draw a button with red
text and blue background.

This is happening because in adjustButtonStyle of RenderThemeQt, in addition to
setting expected values, there is the following line

style->setColor(QApplication::palette().text().color());

which resets the theme color to the default text color.

By removing this line the Qt Port correctly draws the text color including
effects for the button.

Note:  This was discovered while investigating the failure of
fast/dom/HTMLTableColElement/resize-table-using-col-width.html which was
failing due to a color mismatch in the expected vs actual results.

With no style provided default values were expected but the follow response
occurs.

-        RenderButton {BUTTON} at (2,2) size 393x28 [color=#1A1A1A]
[bgcolor=#C0C0C0]
+        RenderButton {BUTTON} at (2,2) size 393x28 [color=#141312]
[bgcolor=#C0C0C0]

With the fix the output is

         RenderButton {BUTTON} at (2,2) size 393x28 [bgcolor=#C0C0C0]

denoting that the color was not overridden.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.