59 bytes, text/x-review-board-request
A follow up according to Bug 1313276 comment 4: The context state management (what matrix is applied, what clips are applied) in the PaintMask function is getting a little hairy. It's fine for now but it seems easy to get wrong if somebody wants to add anything to the function. It would be nice to find a way to clean it up at some point.
Comment on attachment 8822477 [details] Bug 1324174 - Simplify context management logic in nsSVGIntegrationUtils::PaintMask. https://reviewboard.mozilla.org/r/101390/#review102042
Attachment #8822477 - Flags: review?(mstange) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/88ab9fde6a04 Simplify context management logic in nsSVGIntegrationUtils::PaintMask. r=mstange
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox53: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
Component: Layout: View Rendering → Layout: Web Painting
Product: Core → Core
You need to log in before you can comment on or make changes to this bug.