Closed
Bug 1127507
Opened 11 years ago
Closed 11 years ago
Transform does not work on objects nested in a Mask
Categories
(Core :: SVG, defect)
Core
SVG
Tracking
()
RESOLVED
FIXED
mozilla38
People
(Reporter: shorlander, Assigned: jwatt)
References
Details
(Keywords: regression)
Attachments
(3 files)
816 bytes,
image/svg+xml
|
Details | |
4.43 KB,
image/png
|
Details | |
3.45 KB,
patch
|
dholbert
:
review+
Sylvestre
:
approval-mozilla-aurora+
Sylvestre
:
approval-mozilla-beta+
jwatt
:
checkin+
|
Details | Diff | Splinter Review |
I was trying to make a mask with transformed nested objects (see attachment), but it doesn't appear to work.
Works as expected in Webkit, Blink and IE.
Reporter | ||
Comment 1•11 years ago
|
||
Reporter | ||
Updated•11 years ago
|
OS: Mac OS X → All
![]() |
Assignee | |
Updated•11 years ago
|
Assignee: nobody → jwatt
![]() |
Assignee | |
Comment 2•11 years ago
|
||
Attachment #8556707 -
Flags: review?(dholbert)
Updated•11 years ago
|
Blocks: 932771
Keywords: regression
Updated•11 years ago
|
Attachment #8556707 -
Flags: review?(dholbert) → review+
![]() |
Assignee | |
Comment 3•11 years ago
|
||
Comment on attachment 8556707 [details] [diff] [review]
patch
https://hg.mozilla.org/integration/mozilla-inbound/rev/8fad1d7add62
Attachment #8556707 -
Flags: checkin+
Comment 4•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla38
Comment 5•11 years ago
|
||
Comment on attachment 8556707 [details] [diff] [review]
patch
Approval Request Comment
[Feature/regressing bug #]: bug 932771
[User impact if declined]: masks with transforms will be rendered incorrectly
[Describe test coverage new/current, TreeHerder]: patch includes tests
[Risks and why]: low given that we have tests for this.
[String/UUID change made/needed]: none
Attachment #8556707 -
Flags: approval-mozilla-beta?
Attachment #8556707 -
Flags: approval-mozilla-aurora?
Updated•11 years ago
|
Comment 6•11 years ago
|
||
Comment on attachment 8556707 [details] [diff] [review]
patch
Taking because we have test + one line change.
Attachment #8556707 -
Flags: approval-mozilla-beta?
Attachment #8556707 -
Flags: approval-mozilla-beta+
Attachment #8556707 -
Flags: approval-mozilla-aurora?
Attachment #8556707 -
Flags: approval-mozilla-aurora+
Comment 7•11 years ago
|
||
You need to log in
before you can comment on or make changes to this bug.
Description
•