80882 - SVG does not render using correct xml namespace

Status: RESOLVED FIXED

(Core :: SVG, defect)

Description

[Eric Hodel]

When using the SVG namespace (http://www.w3.org/2000/svg) SVG will not render, 
but when using Mozilla's SVG namespace 
(http://www.mozilla.org/keymaster/gatekeeper/there.is.only.svg) SVG will render 
on an SVG-enabled build.

The file (line) to be modified may be 
http://lxr.mozilla.org/seamonkey/source/layout/svg/content/src/nsSVGAtoms.cpp#30

Will post a testcase when I get the time to write one.

Comment 1

[Eric Hodel]

Attachment: SVG using W3C namespace

Comment 2

[Eric Hodel]

Attachment: SVG using mozilla.org namespace

Comment 3

[Eric Hodel]

Expected results:
In the first attachment (SVG using W3C namespace) a black 10px by 10px box
should render in a blue border.  In the second attachment (SVG using mozilla.org
namespace) a collapsed blue border should render (and nothing else).

Actual results:
The first attachment renders as a collapsed blue border and the second renders
as a 10px by 10px black box in a blue border.

Comment 4

[Bradley Baetz]

The namespace stuff is fixed as part of bug 80142 (which will hopefully be
checked in in the next few days), but the testcases don't work anyway - I just
get a white box with a blue border for both of them. I'll look into this...

Comment 5

[Bradley Baetz]

actually, the reason you don't see anything is because you haven't specified a
fill, and the stylesheet you've referenced isn't findable from within bugzilla.
If I add style="fill: black" to the <polygon> then I see the box in the upper
left corner (with the patches)

The invalid namespace also works, but will probably be pulled out at some point
soon - its just there for backwards compatability.

Comment 6

[Eric Hodel]

Sorry, I forgot to mention that this was for Mozilla 0.8.1, the newest SVG build
I have access to. (Build ID: 2001032722)

Comment 7

[Bradley Baetz]

Right - its not checked in yet. Hopefully it will be by 0.9.1.

Comment 8

[basic]

this is fixed no?

Comment 9

[Daniele Nicolodi]

Fixed by SVG merge.

Comment 10

[Jonathan Watt [:jwatt]]

in-testsuite- since this will be covered by most other SVG tests.