Bug 11017

Summary: Browsers disagree on behavior for an invalid fill
Product: WebKit Reporter: Eric Seidel (no email) <eric>
Component: SVGAssignee: Eric Seidel (no email) <eric>
Status: RESOLVED FIXED    
Severity: Normal CC: tor
Priority: P4 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
DetailsFormatted DiffDiff
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+
DetailsFormatted DiffDiff
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.