Summary: | SVG - crash on feMerge when input not available | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Dirk Schulze <krit> | ||||
Component: | SVG | Assignee: | 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
Dirk Schulze
2009-10-12 00:04:55 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 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 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.
|