Bug 52724

Summary: [Qt] svg/css/text-gradient-shadow.svg needs expected results
Product: WebKit Reporter: Csaba Osztrogonác <ossy>
Component: SVGAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, helder, kling, krit, rhodovan.u-szeged, zherczeg, zimmermann
Priority: P2 Keywords: Qt, QtTriaged
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on:    
Bug Blocks: 52531    
Attachments:
Description Flags
fail - png
none
Patch none

Csaba Osztrogonác
Reported 2011-01-19 09:56:51 PST
Created attachment 79440 [details] fail - png svg/css/text-gradient-shadow.svg introduced in http://trac.webkit.org/changeset/76029, but fails on Qt, because the shadow should be color, but it is gray. expected png: http://trac.webkit.org/browser/trunk/LayoutTests/platform/mac/svg/css/text-gradient-shadow-expected.png?rev=76029 fail png: <attached>
Attachments
fail - png (39.19 KB, image/png)
2011-01-19 09:56 PST, Csaba Osztrogonác
no flags
Patch (49.33 KB, patch)
2011-01-20 19:17 PST, Helder Correia
no flags
Helder Correia
Comment 1 2011-01-19 17:38:57 PST
(In reply to comment #0) > svg/css/text-gradient-shadow.svg introduced in http://trac.webkit.org/changeset/76029, > but fails on Qt, because the shadow should be color, but it is gray. The Qt result image is actually correct (shadow must be black). I'll upload a patch with the Qt-specific pixel test soon.
Helder Correia
Comment 2 2011-01-20 19:12:39 PST
Can you please change the title of this bug to "[Qt] svg/css/text-gradient-shadow.svg needs expected results"?
Helder Correia
Comment 3 2011-01-20 19:17:12 PST
Andreas Kling
Comment 4 2011-01-21 02:49:08 PST
Comment on attachment 79682 [details] Patch rs=me
WebKit Commit Bot
Comment 5 2011-01-21 03:14:59 PST
Comment on attachment 79682 [details] Patch Clearing flags on attachment: 79682 Committed r76338: <http://trac.webkit.org/changeset/76338>
WebKit Commit Bot
Comment 6 2011-01-21 03:15:05 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.