Bug 278551
| Summary: | [SVG] SVG Mask code needs to distinguish repaint rect and bounding box | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Yusuke Suzuki <ysuzuki> |
| Component: | SVG | Assignee: | Taher <taher_ali> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | sabouhallawa, simon.fraser, webkit-bug-importer, zimmermann |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| See Also: |
https://bugs.webkit.org/show_bug.cgi?id=263351 https://bugs.webkit.org/show_bug.cgi?id=277624 |
||
Yusuke Suzuki
There are places mixing repaint rect and bounding box. But repaint rect is used for damage computation and it does not need to be accurate, but bounding box needs to be.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/134573463>
Said Abou-Hallawa
This is related to bug 263351 and bug 277624. See the FIXME comments in RenderSVGResourceMasker.cpp and LegacyRenderSVGResourceMasker.cpp.
Taher
Pull request: https://github.com/WebKit/WebKit/pull/55846
EWS
Committed 304945@main (0a40ee094081): <https://commits.webkit.org/304945@main>
Reviewed commits have been landed. Closing PR #55846 and removing active labels.
Nikolas Zimmermann
NOTE: there is a new class in LBSE code that is supposed to handle decorated bounding boxes already. I will look at unifying this once LBSE work resumes in january