Closed Bug 483765 Opened 13 years ago Closed 13 years ago

Filter effects applied to HTML do not zoom properly

Categories

(Core :: SVG, defect)

defect
Not set
normal

Tracking

()

VERIFIED FIXED

People

(Reporter: roc, Assigned: roc)

Details

(Keywords: verified1.9.1)

Attachments

(2 files, 1 obsolete file)

Attached file testcase
Zooming the attached testcase does not affect the size of the green square, which is subject to a filter. It should. (The filter is the identity operation.)
Flags: wanted1.9.1+
Attached patch fix (obsolete) — Splinter Review
The comment in nsSVGIntegrationUtils was just wrong, nsSVGFilterInstance expects us to paint with 1 gfxContext unit equal to one device pixel.
Attachment #367878 - Flags: review?(jwatt)
Whiteboard: [needs review]
Attached patch fix v1.1Splinter Review
Oops, can remove an unneeded line.
Attachment #367878 - Attachment is obsolete: true
Attachment #367879 - Flags: review?(jwatt)
Attachment #367878 - Flags: review?(jwatt)
Attachment #367879 - Attachment is patch: true
Attachment #367879 - Attachment mime type: application/text → text/plain
Comment on attachment 367879 [details] [diff] [review]
fix v1.1

r=jwatt
Attachment #367879 - Flags: review?(jwatt) → review+
Whiteboard: [needs review] → [needs landing]
http://hg.mozilla.org/mozilla-central/rev/8fd3690a1a22
Status: NEW → RESOLVED
Closed: 13 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Whiteboard: [needs landing] → [needs 191 approval]
Comment on attachment 367879 [details] [diff] [review]
fix v1.1

Low-risk patch, fixes a bug in a new feature
Comment on attachment 367879 [details] [diff] [review]
fix v1.1

a191=drivers
Attachment #367879 - Flags: approval1.9.1? → approval1.9.1+
Whiteboard: [needs 191 approval] → [needs 191 landing]
Verified fixed on the 1.9.1 branch using Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1b5pre) Gecko/20090515 Shiretoko/3.5b5pre and Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b5pre) Gecko/20090515 Shiretoko/3.5b5pre.

Verified fixed on the trunk using Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.2a1pre) Gecko/20090515 Minefield/3.6a1pre.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.