RESOLVED FIXED 185600
Make all SVG shape interfaces inherit from SVGGeometryElement 
https://bugs.webkit.org/show_bug.cgi?id=185600
Summary Make all SVG shape interfaces inherit from SVGGeometryElement
Dirk Schulze
Reported 2018-05-13 20:55:48 PDT
Make all SVG shape interfaces inherit from SVGGeometryElement as defined by SVG2. https://svgwg.org/svg2-draft/shapes.html#DOMInterfaces
Attachments
Patch (50.75 KB, patch)
2018-05-13 21:17 PDT, Dirk Schulze 		no flags
DetailsFormatted DiffDiff
Patch for landing (52.92 KB, patch)
2018-05-18 07:38 PDT, Dirk Schulze 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Dirk Schulze
Comment 1 2018-05-13 21:17:41 PDT
Created attachment 340288 [details] Patch
Antti Koivisto
Comment 2 2018-05-16 23:35:38 PDT
Comment on attachment 340288 [details] Patch r=me
Antti Koivisto
Comment 3 2018-05-16 23:37:40 PDT
Comment on attachment 340288 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=340288&action=review > Source/WebCore/rendering/svg/RenderSVGShape.cpp:362 > + bool isValid; > + point = m_path->pointAtLength(distance, isValid); What happens when the point is not valid? Is this covered by the tests?
Dirk Schulze
Comment 4 2018-05-18 07:38:07 PDT
Created attachment 340694 [details] Patch for landing
WebKit Commit Bot
Comment 5 2018-05-18 08:23:01 PDT
Comment on attachment 340694 [details] Patch for landing Clearing flags on attachment: 340694 Committed r231955: <https://trac.webkit.org/changeset/231955>
WebKit Commit Bot
Comment 6 2018-05-18 08:23:02 PDT
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 7 2018-05-18 08:24:17 PDT
<rdar://problem/40365807>
Note You need to log in before you can comment on or make changes to this bug.