Bug 44111 - feTurbulence filter doesn't display properly
Summary: feTurbulence filter doesn't display properly
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Linux
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-08-17 08:39 PDT by Renata Hodovan
Modified: 2010-08-18 06:11 PDT (History)
4 users (show)

See Also:

Attachments
Fix displaying problems of feTurbulence and add a new test. (233.86 KB, patch)
2010-08-17 09:06 PDT, Renata Hodovan 		krit: review-
Details | Formatted Diff | Diff
Fix displaying problems of feTurbulence and add a new test. (234.04 KB, patch)
2010-08-17 12:22 PDT, Renata Hodovan 		no flags Details | Formatted Diff | Diff
Show Obsolete (1) 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 Renata Hodovan 2010-08-17 08:39:29 PDT 
feTurbulence filter doesn't display properly
Comment 1 Dirk Schulze 2010-08-17 08:52:20 PDT 
hm? :-P Can you provide some more details? :-)
Comment 2 Renata Hodovan 2010-08-17 09:06:30 PDT 
Created attachment 64598 [details]
Fix displaying problems of feTurbulence and add a new test.
Comment 3 Renata Hodovan 2010-08-17 09:12:24 PDT 
(In reply to comment #1)
> hm? :-P Can you provide some more details? :-)

We already talked about that by http://dev.w3.org/SVG/profiles/1.1F2/test/harness/htmlObject/filters-turb-02-f.html the two pics are not equivalent in WebKit. I've found the reason of this. It's fixed now and this test is added to WebKit as well.
Comment 4 Dirk Schulze 2010-08-17 09:29:07 PDT 
Comment on attachment 64598 [details]
Fix displaying problems of feTurbulence and add a new test.

Patch looks good. But discribe the bug and how you fixed it in the ChangeLog, please.

I'll give r+ on a better ChangeLog.
Comment 5 Dirk Schulze 2010-08-17 10:41:09 PDT 
(In reply to comment #3)
> (In reply to comment #1)
> > hm? :-P Can you provide some more details? :-)
> 
> We already talked about that by http://dev.w3.org/SVG/profiles/1.1F2/test/harness/htmlObject/filters-turb-02-f.html the two pics are not equivalent in WebKit. I've found the reason of this. It's fixed now and this test is added to WebKit as well.

Yes I know, but you should explain it in bug reports as well.
Comment 6 Renata Hodovan 2010-08-17 12:22:15 PDT 
Created attachment 64616 [details]
Fix displaying problems of feTurbulence and add a new test.
Comment 7 Dirk Schulze 2010-08-17 22:25:18 PDT 
Comment on attachment 64616 [details]
Fix displaying problems of feTurbulence and add a new test.

Great Patch! r=me
Comment 8 Dirk Schulze 2010-08-18 06:08:11 PDT 
Comment on attachment 64616 [details]
Fix displaying problems of feTurbulence and add a new test.

Sorry, forgot to set cq+.
Comment 9 Zoltan Herczeg 2010-08-18 06:11:36 PDT 
Landed in http://trac.webkit.org/changeset/65590
Forgot to close the bug.