Bug 77753

Summary: SVG parser allows white spaces between function name and opening brace
Product: WebKit Reporter: Jacob Goldstein <jacobg>
Component: SVGAssignee: Jacob Goldstein <jacobg>
Status: UNCONFIRMED    
Severity: Normal CC: ahmad.saleem792, jacobg, simon.fraser, WebkitBugTracker, zimmermann
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch ews-feeder: commit-queue-

Jacob Goldstein
Reported 2012-02-03 10:50:18 PST
The SVG parser allows white spaces between function name and first opening brace, CSS doesn't.
Attachments
Patch (2.04 KB, patch)
2012-02-03 13:35 PST, Jacob Goldstein 		ews-feeder: commit-queue-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Jacob Goldstein
Comment 1 2012-02-03 10:50:34 PST
SVG parser should change the behavior with the risk of losing a bit of its backward compatibility. WebKit had just one failing test after experimenting with this rule.
Jacob Goldstein
Comment 2 2012-02-03 10:51:28 PST
Adding this bug as a placeholder for future work and to attach tests
Jacob Goldstein
Comment 3 2012-02-03 13:35:38 PST
Created attachment 125401 [details] Patch Attaching patch that contains a test for this issue to be used in the future Not currently intended for review and/or commit
Jacob Goldstein
Comment 4 2012-02-03 13:37:23 PST
NOTE: - This test currently fails. Will pass when change is made such that the SVG parser does NOT allow a space after function name - May need to update location of test in WebKit directory structure
Ahmad Saleem
Comment 5 2022-10-28 04:51:31 PDT
I took the testcase from attached patch and changed it into JSFiddle: Link - https://jsfiddle.net/6m4yc7n5/show All browsers (Safari 16.1, Safari Technology Preview 156, Chrome Canary 109 and Firefox Nightly 108) show "red" rect only and does not show any "green" rect. Do we need to do anything more or now all browsers agree with each other? Thanks!
Note You need to log in before you can comment on or make changes to this bug.