Last Comment Bug 512501 - (svg11tests) We should pass all the W3C SVG 1.1 Full tests
(svg11tests)
: We should pass all the W3C SVG 1.1 Full tests
Status: NEW
:
Product: Core
Classification: Components
Component: SVG (show other bugs)
: Trunk
: All All
: -- normal with 41 votes (vote)
: ---
Assigned To: Nobody; OK to take it and work on it
:
Mentors:
http://www.w3.org/Graphics/SVG/Test/2...
Depends on: 52282 298281 308338 371728 371787 376027 427713 437554 455986 512537 619509 619632 619972 620002 620278 621824 119490 272288 273171 277955 292498 svg-bidi 317196 319163 388547 391966 398825 433345 436276 436296 animateMotion 454970 474049 474743 enablesmil 483584 485157 508561 512514 512525 512603 525363 548795 557885 569722 570354 570555 587910 589648 602759 607854 617737 619492 619498 619503 619630 619955 619959 619964 619967 619968 619971 619992 620008 620009 620019 620026 620144 620286 622686 716347
Blocks:
  Show dependency treegraph
 
Reported: 2009-08-25 11:03 PDT by Jonathan Watt [:jwatt]
Modified: 2015-11-29 20:17 PST (History)
36 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments

Description Jonathan Watt [:jwatt] 2009-08-25 11:03:21 PDT
We should work our way through the W3C testsuite for SVG 1.1 Full:

  http://www.w3.org/Graphics/SVG/Test/20061213/htmlObjectHarness/full-index.html

and fix all the tests we fail on. Relatively up to date status can be found at:

  http://www.codedread.com/svg-support-table.html

Currently we're passing roughly 67% of the tests with our current SMIL support turned on.
Comment 1 Brian Birtles (:birtles, high review load) 2009-11-25 23:17:46 PST
Marking dependent on bug 275196 as some tests (e.g. animate-elem-201-t) appear to rely on xml:id support.
Comment 2 Henri Sivonen (:hsivonen) 2009-11-25 23:40:53 PST
(In reply to comment #1)
> Marking dependent on bug 275196 as some tests (e.g. animate-elem-201-t) appear
> to rely on xml:id support.

It seems to me the right fix is for the W3C to make tests not depend on xml:id. IIRC, the SVG WG itself was of the opinion that xml:id isn't worth the trouble after the SVG 1.2 Tiny experience.

Also, bug 275196 didn't get into Gecko for perf issues, and WebKit devs have decided not to support xml:id.
Comment 3 Olli Pettay [:smaug] 2009-11-26 01:06:08 PST
The minor perf problems bug 275196 had could be fixed (and probably are in the
latest patch).
But sure, if xml:id won't be supported everywhere, perhaps it isn't worth to
implement it.

Jonathan, do you know if SVG WG could change the spec so that id attribute
could be used as an ID?
Comment 4 Jonathan Watt [:jwatt] 2009-11-26 05:40:40 PST
Brian, I suggest for the moment you "fix up" the tests for your testing using something like:

  find . \( -name '*.svg' -o -name '*.html' \) -exec sed -i -e 's/xml:id/id/' {} \;

Smaug, the "id" attribute is already used as an ID on elements in the SVG namespace. If SVG tests use xml:id gratuitously, they should probably be changed to just use id.

> (In reply to comment #1)
> IIRC, the SVG WG itself was of the opinion that xml:id isn't worth the trouble
> after the SVG 1.2 Tiny experience.

I don't remember there being any final decision on xml:id, and I believe there are members in favor of it. Personally I'd rather we just treat "id" in the null namespace as ID for all content, and preferably have a new version of Namespaces in XML that requires that, and has other fixes to make namespaces user friendly. E.g.

http://groups.google.com/group/mozilla.dev.tech.xml/browse_thread/thread/e7b034b21af81242#
Comment 5 Jonathan Watt [:jwatt] 2009-11-26 05:42:29 PST
Removing bug 275196 from the dep list since this bug is specifically for the SVG 1.1 test suite, and there appear to be no tests in that suite that use xml:id. (The test Brian mentions is in the 1.2 Tiny suite.)
Comment 6 Brian Birtles (:birtles, high review load) 2009-11-26 22:49:55 PST
(In reply to comment #5)
> (The test Brian mentions is in the 1.2 Tiny suite.)

Oops, my bad. I thought I checked that. Sorry for creating a fuss about nothing.
Comment 7 Robert Longson 2010-10-19 02:32:42 PDT
*** Bug 605416 has been marked as a duplicate of this bug. ***
Comment 8 Lars Sonchocky-Helldorf 2011-09-26 12:25:48 PDT
(In reply to Jonathan Watt [:jwatt] from comment #0)
> We should work our way through the W3C testsuite for SVG 1.1 Full:
> 
>  
> http://www.w3.org/Graphics/SVG/Test/20061213/htmlObjectHarness/full-index.
> html

There's new version of the test suite in the works, which fixes some bugs of the above one:

http://dev.w3.org/SVG/profiles/1.1F2/test/harness/htmlObject/index.html

Note You need to log in before you can comment on or make changes to this bug.