Implement restrictions on clipPath content

RESOLVED FIXED in mozilla1.9.3a3

Status

()

Core
SVG
RESOLVED FIXED
8 years ago
2 years ago

People

(Reporter: roc, Assigned: Robert Longson)

Tracking

Trunk
mozilla1.9.3a3
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

http://www.w3.org/TR/SVG/masking.html#EstablishingANewClippingPath

A 'clipPath' element can contain 'path' elements, 'text' elements, basic shapes (such as 'circle') or a 'use' element. If a 'use' element is a child of a 'clipPath' element, it must directly reference 'path', 'text' or basic shape elements. Indirect references are an error (see Error processing).

We don't implement these restrictions, but we should.
(Assignee)

Comment 1

8 years ago
Created attachment 429394 [details] [diff] [review]
patch

(In reply to comment #0)
> 
> We don't implement these restrictions, but we should.

We will when this lands :-)
Assignee: nobody → longsonr
Attachment #429394 - Flags: review?(roc)
Comment on attachment 429394 [details] [diff] [review]
patch

Excellent! Could you add a test for the <use> restrictions?
Attachment #429394 - Flags: review?(roc) → review+
(Assignee)

Comment 3

8 years ago
Created attachment 429432 [details] [diff] [review]
with use reftest
Attachment #429394 - Attachment is obsolete: true
(Assignee)

Comment 4

8 years ago
Created attachment 429438 [details] [diff] [review]
also fix existing invalid reftest
Attachment #429432 - Attachment is obsolete: true
(Assignee)

Comment 5

8 years ago
pushed http://hg.mozilla.org/mozilla-central/rev/ba77049941c3
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
(Assignee)

Updated

8 years ago
Target Milestone: --- → mozilla1.9.3a3
Depends on: 553053
(Assignee)

Updated

2 years ago
Depends on: 1165750
You need to log in before you can comment on or make changes to this bug.