Bug 54186

Summary: SVGFESpecularLightingElement doesn't support dynamic invalidation, when attributes change.
Product: WebKit Reporter: Renata Hodovan <rhodovan.u-szeged>
Component: SVGAssignee: Renata Hodovan <rhodovan.u-szeged>
Status: RESOLVED FIXED    
Severity: Normal CC: zimmermann
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Bug Depends on:    
Bug Blocks: 45453    
Attachments:
Description Flags
Patch none

Renata Hodovan
Reported 2011-02-10 02:23:25 PST
SVGFESpecularLightingElement doesn't support dynamic invalidation, when attributes change.
Attachments
Patch (561.25 KB, patch)
2011-02-10 02:59 PST, Renata Hodovan 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Renata Hodovan
Comment 1 2011-02-10 02:59:18 PST
Created attachment 81944 [details] Patch
Dirk Schulze
Comment 2 2011-02-10 03:05:38 PST
Comment on attachment 81944 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=81944&action=review r=me but with changes. > LayoutTests/svg/dynamic-updates/script-tests/SVGFESpecularLightingElement-dom-in-attr.js:12 > +specularElement.setAttribute("in", "BackgroundImage"); Why do you use backgroundImage here? We don't support it at the moment! Can you use SourceAlpha instead? Otherwise it could be confusing to compare the test on a manual run between browsers. > LayoutTests/svg/dynamic-updates/script-tests/SVGFESpecularLightingElement-svgdom-in-prop.js:12 > +specularElement.setAttribute("in", "BackgroundImage"); Ditto.
Renata Hodovan
Comment 3 2011-02-10 07:52:25 PST
Comment on attachment 81944 [details] Patch Closing bug. Committed r78222 http://trac.webkit.org/changeset/78222
Note You need to log in before you can comment on or make changes to this bug.