Bug 308520 – [svg2] comprehensive test cases for parsing path data of the d attribute in path element

• Status: RESOLVED
• Resolution: FIXED
• Component: SVG

[Karl Dubost]

The SVG2 Path Data for the d attribute of the path element is defined in 
9.3.2. Specifying path data: the ‘d’ property
https://w3c.github.io/svgwg/svg2-draft/paths.html#TheDProperty

the d attribute takes a string with a specific grammar defined in 

9.3.9. The grammar for path data
https://w3c.github.io/svgwg/svg2-draft/paths.html#PathDataBNF

and has error handling constraints:

9.5.4. Error handling in path data
https://w3c.github.io/svgwg/svg2-draft/paths.html#PathDataErrorHandling

The current tests in https://wpt.fyi/results/svg/path
has a partial coverage of the constraints. 

This is an attempt at making more modern tests than the current ones and add tests of cases not being tested currently.

[Ahmad Saleem]

https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/renderer/core/svg/svg_path_parser_test.cc;l=1;drc=31a49a7444fa5b451e621cadb12ba2ea186119d4

^ Blink has these API test cases. Might be good to see if these can be converted into testharness based test cases.

[Karl Dubost]

I have already a series of test case. But it would be good to compare mine with the others.

[Radar WebKit Bug Importer]

<rdar://problem/171586653>

[Karl Dubost]

Pull request: https://github.com/WebKit/WebKit/pull/59882

[EWS]

Committed 308687@main (9af8ffff919a): <https://commits.webkit.org/308687@main>

Reviewed commits have been landed. Closing PR #59882 and removing active labels.

[Karl Dubost]

Submitted web-platform-tests pull request: https://github.com/web-platform-tests/wpt/pull/58282