Bug 243808
| Summary: | Changes to the filter property are sometimes ignored | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Cameron McCormack (:heycam) <heycam> |
| Component: | SVG | Assignee: | Cameron McCormack (:heycam) <heycam> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | sabouhallawa, tanaka.toshiyuki, webkit-bug-importer, zach.silversmith, zimmermann |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Cameron McCormack (:heycam)
Before bug 242497, we would rebuild the SVGResources for a client SVG element whenever we call into its RenderObject's layout function. This was wasteful if we know that they layout change doesn't affect the resource, so in bug 242497 (and in the bugs I'm working under bug 242420), I am making the various RenderSVGResource handle caching data based on the layout information of the client.
When a RenderSVGResource is removed (e.g. because its render tree is being re-created, or the element is removed), we remove it from all clients' SVGResources and mark the clients as needing layout. Now that we no longer unconditionally rebuild the SVGResources for the client under its layout function, we need another way to indicate that it needs to be rebuilt.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Cameron McCormack (:heycam)
rdar://97335496
Cameron McCormack (:heycam)
Pull request: https://github.com/WebKit/WebKit/pull/3209
Sammy Gill
*** Bug 183871 has been marked as a duplicate of this bug. ***
Cameron McCormack (:heycam)
rdar://97335496
Cameron McCormack (:heycam)
*** Bug 247411 has been marked as a duplicate of this bug. ***
EWS
Committed 256335@main (64eee924e808): <https://commits.webkit.org/256335@main>
Reviewed commits have been landed. Closing PR #3209 and removing active labels.