Bug 30297 - SVG - crash on feMerge when input not available
Summary: SVG - crash on feMerge when input not available
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: 525.x (Safari 3.1)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks: 68469 26389
  Show dependency treegraph
 
Reported: 2009-10-12 00:04 PDT by Dirk Schulze
Modified: 2014-05-12 05:54 PDT (History)
1 user (show)

See Also:


Attachments
fix for feMerge if one effect doesn't exist (2.40 KB, patch)
2009-10-12 00:11 PDT, Dirk Schulze
eric: review+
eric: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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.