Bug 15280 – SVG Subresources will not be saved when creating WebArchives

• Status: RESOLVED
• Resolution: FIXED
• Component: SVG

[Eric Seidel (no email)]

SVG Subresources will not be saved when creating WebArchives

SVG elements (actually the Obj-C DOM wrappers for the elements) do not implement the _subresources, internal method to expose the urls of their subresources.  I had to implement this for SVGImageElement, but I have not done so for any of the other SVG Elements.  Mostly because the first two other elements I looked at (foreignObject and script) did not expose their xlink:href attribute via a binding method in their idls.  So there is some piping which would need to be put in place in order to get the actual href values from each of these SVG Elements and made available to the _subresources call.

See:
http://trac.webkit.org/projects/webkit/browser/trunk/WebKit/DOM/WebDOMOperations.mm

for examples of the _subresources code necessary.

see:
http://www.w3.org/TR/SVG/attindex.html
for a list of elements which use the xlink:href property (as well as any other property which might make reference to an external file).

[Eric Seidel (no email)]

Created attachment 16455 [details]
the fix

[Alexey Proskuryakov]

Comment on attachment 16455 [details]
the fix

r=me

[Eric Seidel (no email)]

I should note that this patch does not implement *all* of the _subresources methods that SVG might ever need.  If we ever support external references with <use> or <tref>, etc. those will all need _subresources methods in order to be correctly archived.  But I believe all of the SVGElement subclasses which do support external references now have proper _subresources methods.

Landed on feature-branch as r25809.

[Oliver Hunt]

This was skipped when rebasing and will need to be updated for the new featurebranch

[Eric Seidel (no email)]

relanded (same patch) as r25908.