Bug 191377
| Summary: | [SVG2]: Remove the interface SVGRect | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Said Abou-Hallawa <sabouhallawa> |
| Component: | SVG | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED INVALID | ||
| Severity: | Normal | CC: | ahmad.saleem792, krit, zimmermann |
| Priority: | P2 | ||
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Bug Depends on: | |||
| Bug Blocks: | 191292, 200143 | ||
Said Abou-Hallawa
This should affect the following methods:
SVGGraphicsElement::getBBox(): https://www.w3.org/TR/SVG2/types.html#InterfaceSVGGraphicsElement
SVGSVGElement::createSVGRect(): https://www.w3.org/TR/SVG2/struct.html#InterfaceSVGSVGElement
SVGTextContentElement::getExtentOfChar(): https://www.w3.org/TR/SVG2/text.html#InterfaceSVGTextContentElement
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Said Abou-Hallawa
It will also affect the interface SVGAnimatedRect: https://www.w3.org/TR/SVG2/types.html#InterfaceSVGAnimatedRect
Dirk Schulze
For all those interfaces, keep in mind that SVGRect, SVGMatrix and SVGPoint are aliases to DOMRect, DOMMatrix and DOMPoint now. So tests with the SVG* interfaces should still pass.
Said Abou-Hallawa
Related WPT tests:
https://wpt.fyi/results/svg/types/scripted/SVGAnimatedRect.html?label=master&label=experimental
Said Abou-Hallawa
https://wpt.fyi/results/svg/types/scripted/SVGGraphicsElement.svg?label=master&label=experimental
Ahmad Saleem
@Said - isn't consensus now to keep them - https://github.com/w3c/svgwg/issues/706
and we can close this bug?
Said Abou-Hallawa
Yes SVGRect should stay and the SVG2 specs for replacing SVGRect should be reverted. This bug should be resolved - Invalid.