Fix bugs in PrependLocalTransformsTo implementations where we forget to prepend to the matrix that is passed in

RESOLVED FIXED in mozilla35

Status

()

Core
SVG
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: jwatt, Assigned: jwatt)

Tracking

Trunk
mozilla35
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Created attachment 8482987 [details] [diff] [review]
patch
Attachment #8482987 - Flags: review?(longsonr)
Hmm, SVGForeignObjectElement is wrong too isn't it? Please check any other cases.
Attachment #8482987 - Flags: review?(longsonr)
Flags: needinfo?(jwatt)
Good catch. SVGUseElement too. Those didn't trigger failures, but we should fix them.
Flags: needinfo?(jwatt)
Created attachment 8483096 [details] [diff] [review]
patch
Attachment #8482987 - Attachment is obsolete: true
Attachment #8483096 - Flags: review?(longsonr)
(Assignee)

Updated

3 years ago
Summary: Fix a bug in SVGSVGElement::PrependLocalTransformsTo where we forget to prepend to the past matrix → Fix bugs in PrependLocalTransformsTo implementations where we forget to prepend to the matrix that is passed in
Attachment #8483096 - Flags: review?(longsonr) → review+
https://hg.mozilla.org/mozilla-central/rev/ce99d180c6fb
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla35
You need to log in before you can comment on or make changes to this bug.