Bug 47409 - SVG feImage needs absolute subregion for preserveAspectRatio
Summary: SVG feImage needs absolute subregion for preserveAspectRatio
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Nobody
URL: http://dev.w3.org/SVG/profiles/1.1F2/...
Keywords:
Depends on:
Blocks: 68469 26389
  Show dependency treegraph
 
Reported: 2010-10-08 05:09 PDT by Dirk Schulze
Modified: 2014-05-12 05:54 PDT (History)
3 users (show)

See Also:


Attachments
Patch (64.76 KB, patch)
2010-10-08 05:28 PDT, Dirk Schulze
zimmermann: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dirk Schulze 2010-10-08 05:09:14 PDT
SVG feImage needs the subregion in absolute coordinates to determine the correct destination position and size. I have a fix for this.
Comment 1 Dirk Schulze 2010-10-08 05:28:31 PDT
Created attachment 70228 [details]
Patch
Comment 2 Nikolas Zimmermann 2010-10-08 06:04:05 PDT
Comment on attachment 70228 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=70228&action=review

> WebCore/platform/graphics/filters/FilterEffect.h:46
> +enum FilterEffectType {
> +    unknownEffectType,
> +    feImage,
> +    feTile,
> +    sourceInput

This is ugly and doesn't conform to the style guide (style bot doesn't check this).
"Enum members should user InterCaps with an initial capital letter." Also these names are confusing.
FilterEffectTypeUnknown,
FilterEffectTypeImage,
FilterEffectTypeTile,
FilterEffectTypeSourceInput

The rest looks good. Except the name for the test directory, I'd prefer LayoutTest/svg/W3C-SVG-1.1-SE/ (capital SE).
Please fix before landing.
Comment 3 Dirk Schulze 2010-10-08 12:41:29 PDT
Committed r69416: <http://trac.webkit.org/changeset/69416>