Bug 11017

Summary:

Browsers disagree on behavior for an invalid fill

Product:

WebKit

Reporter:

Eric Seidel (no email) <eric>

Component:

SVG

Assignee:

Eric Seidel (no email) <eric>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

tor

Priority:

Keywords:

HasReduction

Version:

420+

Hardware:

Mac

OS:

OS X 10.4

URL:

http://hixie.ch/tests/adhoc/svg/error/003.xml

Bug Depends on:

Bug Blocks:

6018

Attachments:

Description	Flags
A test case to document our behavior	none
New test case (with differnet expected results)	none
Change to match opera's behavior, with updated testcase	darin: review+

Eric Seidel (no email)

Reported 2006-09-24 19:25:50 PDT

Browsers disagree on behavior for an invalid fill Hixie wrote a test (http://hixie.ch/tests/adhoc/svg/error/003.xml) which assumes fill should revert to black (default) when encountering an invalid fill. The spec says there should be a highly perceivable error message. No browser vendor (to my knowledge) has any plans to implement these error messages. Current behavior: Opera: fill=black Firefox: fill=none Safar: fill=none I've attached a test which will at least allow us to track this behavior.

Attachments
A test case to document our behavior (30.61 KB, patch) 2006-09-24 19:31 PDT, Eric Seidel (no email)	no flags	Details Formatted Diff Diff
New test case (with differnet expected results) (695 bytes, image/svg+xml) 2006-09-25 14:55 PDT, Eric Seidel (no email)	no flags	Details
Change to match opera's behavior, with updated testcase (42.48 KB, patch) 2006-09-26 02:29 PDT, Eric Seidel (no email)	darin: review+	Details Formatted Diff Diff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.

Eric Seidel (no email)

Comment 1 2006-09-24 19:31:43 PDT

Created attachment 10746 [details] A test case to document our behavior

Eric Seidel (no email)

Comment 2 2006-09-24 19:34:13 PDT

Comment on attachment 10746 [details] A test case to document our behavior beth reviewed.

Eric Seidel (no email)

Comment 3 2006-09-24 19:36:42 PDT

r16553

Eric Seidel (no email)

Comment 4 2006-09-25 14:55:50 PDT

Created attachment 10774 [details] New test case (with differnet expected results)

Eric Seidel (no email)

Comment 5 2006-09-25 14:56:09 PDT

re-opening. Firefox has agreed to join us in matching Opera.

Darin Adler

Comment 6 2006-09-25 15:46:02 PDT

Comment on attachment 10746 [details] A test case to document our behavior Clearing the review flag on this so the bug doesn't show up in the commit queue.

Eric Seidel (no email)

Comment 7 2006-09-26 02:29:50 PDT

Created attachment 10780 [details] Change to match opera's behavior, with updated testcase Firefox (tor) has agreed to match Opera's (and now our) behavior in this way. I'm about to file a firefox bug on the subject.

Darin Adler

Comment 8 2006-09-26 07:46:18 PDT

Comment on attachment 10780 [details] Change to match opera's behavior, with updated testcase r=me

Mark Rowe (bdash)

Comment 9 2006-10-05 15:04:07 PDT

This was landed in r16599.

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