458068 - SVG fill="none" is broken - behaves like fill="black"

Status: VERIFIED FIXED

(Core :: SVG, defect, P1)

Description

[Duncan Loveday]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b1pre) Gecko/20081001 Minefield/3.1b1pre
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b1pre) Gecko/20081001 Minefield/3.1b1pre

A simple SVG element with fill="none" is being filled with black on the current trunk build. This is the case even if stroke is set to another colour.

Reproducible: Always

Steps to Reproduce:
1. Load the attached
2.
3.
Actual Results:  
The circle and rectangle are both filled with black.

Expected Results:  
The circle and rectangle should not be filled, one is stroked with green and the other blue.

Comment 1

[Duncan Loveday]

Attachment: SVG test case

Comment 2

[Mats Palmgren (inactive)]

Regression window: 2008-09-29-02 -- 2008-09-30-02, likely by bug 455984

Comment 3

[Robert Longson [:longsonr]]

Attachment: patch

Comment 4

[Robert Longson [:longsonr]]

Attachment: reftest

Comment 5

[Vladimir Vukicevic [:vlad] [:vladv] (needinfo me, slow to respond)]

This needs to go in for b1 -- it breaks google maps.  Testcase: http://tinyurl.com/3n489a

Comment 6

[Jesse Ruderman]

It looks like a reftest change for bug 455226 snuck into the reftest patch for this bug.

Comment 7

[Robert Longson [:longsonr]]

Part of it did but its innocuous. The part to enable that reftest did not sneak in so the reftest isn't run.

Comment 8

[Mike Beltzner [:beltzner, not reading bugmail]]

(In reply to comment #5)
> This needs to go in for b1 -- it breaks google maps.  Testcase:
> http://tinyurl.com/3n489a

Just FTR, you need to change general.useragent.extra.firefox to be Firefox/3.1b1pre to see this actually happen.

Comment 10

[Dão Gottwald [:dao]]

http://hg.mozilla.org/mozilla-central/rev/7a1870bcfef0
http://hg.mozilla.org/mozilla-central/rev/deb6ad2c79b5

Comment 11

[Marcia Knous [:marcia]]

verified fixed using Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1b2pre) Gecko/20081013 Minefield/3.1b2pre and Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.1b2pre) Gecko/20081013 Minefield/3.1b2pre. I verified using the testcase in Comment 1.