Bug 30297

Summary: SVG - crash on feMerge when input not available
Product: WebKit Reporter: Dirk Schulze <krit>
Component: SVGAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: jeffschiller
Priority: P2    
Version: 525.x (Safari 3.1)   
Hardware: PC   
OS: OS X 10.5   
Bug Depends on:    
Bug Blocks: 68469, 26389    
Attachments:
Description Flags
fix for feMerge if one effect doesn't exist eric: review+, eric: commit-queue-

Description Dirk Schulze 2009-10-12 00:04:55 PDT
SVG - crash on feMerge when input not available
Comment 1 Dirk Schulze 2009-10-12 00:11:02 PDT
Created attachment 41020 [details]
fix for feMerge if one effect doesn't exist

fix for feMerge if one effect doesn't exist. I'll create test results at home.
Comment 2 Eric Seidel (no email) 2009-10-12 07:06:59 PDT
Comment on attachment 41020 [details]
fix for feMerge if one effect doesn't exist

The change and test looks fine.  But it looks like you need layout test results.  Please add results and update the ChangeLog accordingly before landing.
Comment 3 Eric Seidel (no email) 2009-10-12 07:08:06 PDT
Comment on attachment 41020 [details]
fix for feMerge if one effect doesn't exist

Also, ideally the test case woudl make clear that it's just not supposed to crash.  Probably a comment in the SVG file woudl help.  Adding text to the SVG file would make the results platform specific.  You could also add a 100x100 green square to the SVG, so that people looking at it visually would know that it passes.
Comment 4 Dirk Schulze 2009-10-12 14:20:50 PDT
landed in r49472.