Closed Bug 512501 (svg11tests) Opened 15 years ago Closed 6 years ago

We should pass all the W3C SVG 1.1 Full tests

Categories

(Core :: SVG, defect)

defect
Not set
normal

Tracking

()

RESOLVED WONTFIX

People

(Reporter: jwatt, Unassigned)

References

(Depends on 10 open bugs, )

Details

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.
Depends on: 427713, 292498
Depends on: 512514
Depends on: 474049, 437554
Depends on: 455986, 119490
Depends on: 391966
Depends on: 512525, 298281
Depends on: 371728
Depends on: 272288
Depends on: 512537
Depends on: 308338
Depends on: 503075
Depends on: 319163
Depends on: 317196
Depends on: svg-bidi
Depends on: 371787
Depends on: 273171
Depends on: enablesmil
Depends on: 512603
Version: unspecified → Trunk
Depends on: 388547
Depends on: 433345
Alias: svg11tests
Depends on: 525363
Depends on: 483584
Marking dependent on bug 275196 as some tests (e.g. animate-elem-201-t) appear to rely on xml:id support.
Depends on: 275196
(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.
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?
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#
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.)
No longer depends on: 275196
(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.
Depends on: 548795
Depends on: 557885
Depends on: 570354
Depends on: 570555
Depends on: 569722
Depends on: 52282
Depends on: 617737
No longer depends on: 503075
Depends on: 619492
Depends on: 619498
Depends on: 619503
Depends on: 619509
Depends on: 589648
Depends on: 619630
Depends on: 619632
Depends on: 619955
Depends on: 619959
Depends on: 619964
Depends on: 619967
Depends on: 619968
Depends on: 619971
Depends on: 619972
Depends on: 619992
Depends on: 620002
Depends on: 620008
Depends on: 620009
Depends on: 376027
Depends on: 620019
Depends on: 620026
Depends on: 602759
Depends on: 277955
Depends on: 620144
Depends on: 620278
Depends on: 620286
Depends on: 607854
Depends on: 621824
Depends on: 622686
Depends on: 454970
Depends on: 508561
(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
Depends on: 716347
Depends on: 398825
These tests are now obsolete, a conformant SVG 2 implementation would not pass them all.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.