Bug 77650 - [Qt] green should be RGB(0x00,0x80,0x00)
Summary: [Qt] green should be RGB(0x00,0x80,0x00)
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords: Qt, QtTriaged
Depends on:
Blocks:
 
Reported: 2012-02-02 09:08 PST by Csaba Osztrogonác
Modified: 2014-02-03 03:19 PST (History)
8 users (show)

See Also:

Attachments
example for this fail: svg/filters/feImage-target-remove-from-document-expected.png (1.53 KB, patch)
2012-02-02 09:08 PST, Csaba Osztrogonác 		no flags Details | Formatted Diff | Diff
patch idea (606 bytes, patch)
2012-05-17 07:08 PDT, Zoltan Herczeg 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Csaba Osztrogonác 2012-02-02 09:08:50 PST 
Created attachment 125144 [details]
example for this fail: svg/filters/feImage-target-remove-from-document-expected.png

QtWebKit sometimes generates RGB(0x00,0xBE,0x00) green 
instead of the standardized RGB(0x00,0x80,0x00) green.

I see this kind of failures regularly in SVG tests, I attached one of them.

Have you got any idea what cause this bug and how can we fix it?
Comment 1 Csaba Osztrogonác 2012-02-02 09:10:59 PST 
source:
<rect id="green-rect" width="100" height="100" fill="green"/>

rendertree dump:
RenderSVGRect {rect} at (0,0) size 100x100 [fill={[type=SOLID] [color=#008000]}] [x=0.00] [y=0.00] [width=100.00] [height=100.00]

It's strange for me that the color is correct in rendertree dump,
but incorrect in the real world. (browser, png, etc.)
Comment 2 Csaba Osztrogonác 2012-02-02 09:12:12 PST 
Click on the "Review patch" to see the differences.
Comment 3 Alexey Proskuryakov 2012-02-02 20:44:32 PST 
See also bug 5972.
Comment 4 Zoltan Herczeg 2012-05-17 07:08:41 PDT 
Created attachment 142467 [details]
patch idea

Niko, could you check this patch?
Comment 5 Nikolas Zimmermann 2012-05-17 08:12:51 PDT 
Comment on attachment 142467 [details]
patch idea

A pity we have to do this :(
This will affect non-Qt platforms as well, so I guess this will result in lots of rebaselines?
Comment 6 Jocelyn Turcotte 2014-02-03 03:19:53 PST 
=== Bulk closing of Qt bugs ===

If you believe that this bug report is still relevant for a non-Qt port of webkit.org, please re-open it and remove [Qt] from the summary.

If you believe that this is still an important QtWebKit bug, please fill a new report at https://bugreports.qt-project.org and add a link to this issue. See http://qt-project.org/wiki/ReportingBugsInQt for additional guidelines.