Bug 55462 - FEMorphologyElement changes doesn't require relayout.
Summary: FEMorphologyElement changes doesn't require relayout.
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: Renata Hodovan
URL:
Keywords:
Depends on:
Blocks: 55075
  Show dependency treegraph
 
Reported: 2011-03-01 05:29 PST by Renata Hodovan
Modified: 2011-03-02 00:43 PST (History)
1 user (show)

See Also:


Attachments
Proposed patch (5.47 KB, patch)
2011-03-01 05:37 PST, Renata Hodovan
kling: review-
Details | Formatted Diff | Diff
Proposed patch (5.23 KB, patch)
2011-03-01 07:22 PST, Renata Hodovan
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Renata Hodovan 2011-03-01 05:29:16 PST
FEMorphologyElement changes doesn't require relayout.
Comment 1 Renata Hodovan 2011-03-01 05:37:11 PST
Created attachment 84218 [details]
Proposed patch

SVGFECompositeElement.cpp is in ChangeLog, but it has just a style fix.
Comment 2 Andreas Kling 2011-03-01 07:00:19 PST
Comment on attachment 84218 [details]
Proposed patch

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

r=me

> Source/WebCore/ChangeLog:18
> +        * svg/SVGFECompositeElement.cpp:

The whitespace change does not belong in this patch, please leave it out.
Comment 3 Andreas Kling 2011-03-01 07:02:31 PST
Comment on attachment 84218 [details]
Proposed patch

Oops, this is missing modifications to SVGFEMorphologyElement::svgAttributeChanged()!
Comment 4 Renata Hodovan 2011-03-01 07:22:26 PST
Created attachment 84227 [details]
Proposed patch
Comment 5 Renata Hodovan 2011-03-02 00:42:56 PST
Landed in <http://trac.webkit.org/changeset/80099>.