1069931 - Implement 'discard' element from SVG Animations

Status: ASSIGNED

(Core :: SVG, enhancement)

Description

[err]

Attachment: svgt12_discard.svg

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:35.0) Gecko/20100101 Firefox/35.0
Build ID: 20140918030202

Steps to reproduce:

See this test from Erik Dahlström, Opera Software:
http://xn--dahlstrm-t4a.net/svg/smil/svgt12_discard.svg
(also attached in case the link breaks)


Actual results:

Test fails: after 10 seconds still says "Loading..."


Expected results:

At the end of animation <discard> should remove its parent <g> so that the first element (<text>Load complete</text>) should become visible.

It already works in old Opera (Presto) and current Chrome, so it would be great to see this feature implemented in Firefox in order to start using it in practice.

Spec:
http://www.w3.org/TR/SVG2/struct.html#DiscardElement

Corresponding Chrome bug:
http://code.google.com/p/chromium/issues/detail?id=327339

Test from SVG 1.2 Tiny Test Suite, that works in Opera but doesn't work in Chrome for some reason:
http://www.w3.org/Graphics/SVG/Test/20080912/htmlObjectHarness/struct-discard-207-t.html

Comment 1

[Sebastian Zartner [:sebo]]

The <discard> element was moved to the SVG Animations spec. in the meantime.

Sebastian

Comment 2

[Kuba Niewiarowski]

Partially implemented in Chromium and marked as fixed, but still incomplete...

Comment 3

[Robert Longson [:longsonr]]

Attachment: Bug 1069931 - Implement the discard element from SVG Animations r=dholbert

Comment 4

[Robert Longson [:longsonr]]

Attachment: parser.txt

Comment 5

[Robert Longson [:longsonr]]

https://groups.google.com/a/mozilla.org/g/dev-platform/c/U4nufgUTdAk

Comment 6

[Henri Sivonen (:hsivonen)]

Comment on attachment 9325445 [details] [diff] [review]
parser.txt

Review of attachment 9325445 [details] [diff] [review]:
-----------------------------------------------------------------

r+ Thanks.