As a security precaution, we have turned on the setting "Require API key authentication for API requests" for everyone. If this has broken something, please contact
Last Comment Bug 769103 - Get SVG masks working for HTML elements under CSS transforms
: Get SVG masks working for HTML elements under CSS transforms
Product: Core
Classification: Components
Component: SVG (show other bugs)
: Trunk
: All All
: -- normal (vote)
: mozilla16
Assigned To: Jonathan Watt [:jwatt]
: Jet Villegas (:jet)
Depends on:
Blocks: 614732
  Show dependency treegraph
Reported: 2012-06-27 17:06 PDT by Jonathan Watt [:jwatt]
Modified: 2012-06-29 00:46 PDT (History)
2 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

patch (5.29 KB, patch)
2012-06-27 17:16 PDT, Jonathan Watt [:jwatt]
roc: review+
Details | Diff | Splinter Review
patch (6.81 KB, patch)
2012-06-28 03:14 PDT, Jonathan Watt [:jwatt]
roc: review+
Details | Diff | Splinter Review

Description User image Jonathan Watt [:jwatt] 2012-06-27 17:06:18 PDT
nsSVGMaskFrame::ComputeMaskAlpha doesn't account for transforms that are already applied to the gfxContext that is being painting to. For elements that are under a CSS transform that's scales the element up, this means that it creates surfaces that are too small (or, more to the point, surfaces that don't have enough resolution) when creating the mask pattern it returns. As a result, the more the element is scaled up, the more pixelated the masking becomes.

This isn't generally a problem for SVG, since the transforms aren't accumulated on the gfxMatrix as we descend down the tree. However, when switching to display list painting of SVG, we do accumulate the transforms as we descend the tree, and this bug causes various SVG reftests to fail.
Comment 1 User image Jonathan Watt [:jwatt] 2012-06-27 17:16:35 PDT
Created attachment 637318 [details] [diff] [review]
Comment 2 User image Jonathan Watt [:jwatt] 2012-06-28 03:14:33 PDT
Created attachment 637440 [details] [diff] [review]

Actually, that's not right. While falling asleep last night I got to wondering why I could use SetDeviceOffset() at all, and this morning I came up with a test that proves that I shouldn't be. Here's a new patch to fix that, along with that test.
Comment 3 User image Ed Morley [:emorley] 2012-06-29 00:46:08 PDT

Note You need to log in before you can comment on or make changes to this bug.